Skip to content

[choreolib] Update Java and C++ to WPILib 2027.0.0-alpha-6#1481

Open
Daniel1464 wants to merge 44 commits into
SleipnirGroup:mainfrom
Daniel1464:2027-update
Open

[choreolib] Update Java and C++ to WPILib 2027.0.0-alpha-6#1481
Daniel1464 wants to merge 44 commits into
SleipnirGroup:mainfrom
Daniel1464:2027-update

Conversation

@Daniel1464

@Daniel1464 Daniel1464 commented May 9, 2026

Copy link
Copy Markdown
Contributor

In addition to updating to 2027, this PR also flips the coordinate system convention to be compatible with 2027 WPILib(namely, that (0,0) is at the center of the field). Note that the UI still generates coordinates in the 2026 WPILib coordinate system.

On the Java API side, AutoRoutine has been removed in favor of making AutoTrajectory an isolated class (since triggers can now be scoped per-opmode or, in the case of commands v3, per command). AutoChooser has also been removed in favor of using opmodes to lazily load trajectories.

@github-actions github-actions Bot added component: choreolib-java component: choreolib-cpp type: build Build, CI, and release artifacts component: backend Rust/Tauri backend component: cli Command-line interface labels May 9, 2026
@Daniel1464 Daniel1464 marked this pull request as draft May 9, 2026 16:48
Comment thread .github/workflows/choreolib.yml Outdated
Comment thread .github/workflows/choreolib.yml Outdated
Comment thread .github/workflows/choreolib.yml Outdated
Comment thread .github/workflows/choreolib.yml Outdated
Comment thread .github/workflows/choreolib.yml Outdated
Comment thread .github/workflows/choreolib.yml Outdated
Comment thread .github/workflows/choreolib.yml Outdated
Comment thread .github/workflows/choreolib.yml Outdated
Comment thread .github/workflows/choreolib.yml
Comment thread .github/workflows/choreolib.yml

@calcmogul calcmogul left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There should be Systemcore Docker containers you can use instead of manually installing the toolchain.

Comment thread .github/workflows/choreolib.yml Outdated
Comment thread .github/workflows/choreolib.yml Outdated
Comment thread .github/workflows/choreolib.yml Outdated
Comment thread .github/workflows/choreolib.yml Outdated
Comment thread .github/workflows/choreolib.yml Outdated
Comment thread .github/workflows/choreolib.yml
Comment thread .github/workflows/lint-format.yml Outdated
Comment thread .github/workflows/lint-format.yml Outdated
Comment thread src-tauri/Cargo.toml Outdated
Comment thread Cargo.lock
Co-authored-by: Tyler Veness <calcmogul@gmail.com>
Comment thread update_field_dimensions.py Outdated
Co-authored-by: Tyler Veness <calcmogul@gmail.com>
@Daniel1464 Daniel1464 marked this pull request as ready for review May 10, 2026 02:37
@github-actions github-actions Bot removed the component: ui UI frontend label May 10, 2026
@Daniel1464 Daniel1464 requested a review from calcmogul May 10, 2026 13:21
Comment thread .github/workflows/choreolib.yml Outdated
Comment thread .github/workflows/choreolib.yml Outdated
Comment thread .github/workflows/choreolib.yml Outdated
Comment thread .github/workflows/choreolib.yml Outdated
Comment thread choreolib/src/main/native/include/choreo/trajectory/DifferentialSample.hpp Outdated
Comment thread choreolib/src/main/native/include/choreo/util/AllianceFlipperUtil.hpp Outdated
Comment thread choreolib/src/main/native/include/choreo/util/FieldDimensions.hpp Outdated
Comment thread choreolib/src/test/native/cpp/SampleFlippingTests.cpp Outdated
Comment thread choreolib/src/test/native/cpp/TrajectoryFileTests.cpp Outdated
Comment thread choreolib/build.gradle Outdated
Co-authored-by: Tyler Veness <calcmogul@gmail.com>
Comment thread .github/workflows/choreolib.yml Outdated
Comment thread choreolib/build.gradle Outdated
Comment thread choreolib/build.gradle Outdated
Comment thread choreolib/build.gradle
Comment thread choreolib/build.gradle
Comment thread choreolib/build.gradle Outdated
Comment thread choreolib/config.gradle Outdated
@github-actions github-actions Bot added the component: documentation Improvements or additions to documentation label May 11, 2026
@Daniel1464 Daniel1464 requested a review from calcmogul May 11, 2026 20:05
Comment thread choreolib/build.gradle Outdated
Comment thread choreolib/build.gradle Outdated
Comment thread choreolib/build.gradle Outdated
Comment thread .github/workflows/choreolib.yml Outdated
Daniel1464 and others added 3 commits May 11, 2026 20:04
Co-authored-by: Tyler Veness <calcmogul@gmail.com>
Removed architecture specification from Java setup step.
@Daniel1464 Daniel1464 requested a review from calcmogul May 12, 2026 13:01

@calcmogul calcmogul left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We can probably rely on the Java architecture autodetection here. I'm not sure -Pbuildalldesktop does anything because I can't find any references in GradleRIO or native-utils. We could keep it anyway since the vendor template has it.

Otherwise, this PR looks good to me.

Comment thread .github/workflows/choreolib.yml Outdated
Comment thread .github/workflows/choreolib.yml Outdated
Comment thread .github/workflows/choreolib.yml Outdated
Co-authored-by: Tyler Veness <calcmogul@gmail.com>
@calcmogul calcmogul changed the title Update ChoreoLib java and c++ to WPILib 2027 alpha 6 Update ChoreoLib Java and C++ to WPILib 2027.0.0-alpha-6 May 13, 2026
@calcmogul calcmogul changed the title Update ChoreoLib Java and C++ to WPILib 2027.0.0-alpha-6 [choreolib] Update Java and C++ to WPILib 2027.0.0-alpha-6 May 13, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

component: backend Rust/Tauri backend component: choreolib-cpp component: choreolib-java component: cli Command-line interface component: documentation Improvements or additions to documentation type: build Build, CI, and release artifacts

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants