Skip to content

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

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

(#1) Updated to wpilib 27 alpha in order to build#2
thetaback wants to merge 3 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?

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