Skip to content

(#1) Updated to wpilib 27 alpha in order to build - #2

Open
thetaback wants to merge 9 commits into
mainfrom
1-update-for-syscore
Open

thetaback wants to merge 9 commits into
mainfrom
1-update-for-syscore

Conversation

@thetaback

Copy link
Copy Markdown
Contributor

No description provided.

@thetaback

Copy link
Copy Markdown
Contributor Author

After trying to run this in sim it seems like we may need more coppercore changes. Here is the error output:

Error at coppercore.parameter_tools.json.strategies.JSONPrimitiveCheckStrategy.handlePrimitive(JSONPrimitiveCheckStrategy.java:92): Unhandled exception instantiating robot coppercore.parameter_tools.json.strategies.JSONPrimitiveCheckStrategy java.lang.RuntimeException: Primitive Used: 
In Class: com.therekrab.autopilot.APConstraints
Declaring Class Super Class: java.lang.Object
Field Name: x0
Field Generic Type: double

@godmar Should we roll another release of coppercore with these changes if we can get the robot code to build?

@godmar

godmar commented Sep 8, 2026

Copy link
Copy Markdown
Contributor

This doesn't look like a coppercore issue, but rather autopilot being incompatible with coppercore's serialization mechanism. Specifically, we don't allow final variables that are primitive types like lowercase double, as here. I'm a bit confused though we this didn't trigger an error earlier, given that the old 2026 version of autopilot also has a double here.

Did we perhaps have a custom JSON serializer for APConstraints that was used earlier and isn't anymore?

@thetaback

Copy link
Copy Markdown
Contributor Author

I commented out BatteryVoltageAlert as it is used for an Elastic alert, as this was preventing us from running the code in simulation and may need to be changed anyway in alpha7. Build still seems to be failing on the GitHub side, but I was able to run the latest change successfully in simulation and run an auto routine.

@Samson560915 can you take a look at the normalizeHeading() as it is used here (currently, it does not compile as we removed the overload of this method, due to recent WPILib changes)? You can branch off of this branch in order to work on fixing this issue.

Godmar Back and others added 4 commits September 16, 2026 16:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants