-
Notifications
You must be signed in to change notification settings - Fork 613
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
AHRS crashes on startup due to RuntimeDetector removal #6823
Comments
There is no supported workaround. Vendor deps are often broken in development versions. You might be able to insert a copy of the old version of the RuntimeDetector class into your robot project, but that’s not guaranteed to work. |
ok, i'll figure it out. the curious thing is that it seems like nobody expected this particular thing to break. you: "Do any vendor libraries use the removed functionality?" Thad: "They shouldn't be doing so anyway if they are." |
The 2025 development artifacts have more breaking changes than just that change. I would not rely on any vendordeps working 100% until they update to build against development. |
:) anything in particular come to mind in terms of broken things? |
OK i can answer my own question here. I was able to work around the NavX use of RuntimeDetector, just by liberally copying the NavX java code. But Revlib uses RuntimeLoader and JNI. I half-heartedly tried duplicating all the revlib java code and removing the vendordep but it all seemed like a lot of work, because of JNI name mapping. So I gave up. I think these vendordep things tend to be updated really late, don't they? like after kick-off? i really like getting a head start on using the new parts of wpilib (i mean, come on! Ellipse2d !!), and these breaking changes will delay that. |
(if anyone knows a way to remap JNI calls, so i can use the old native code with an arbitrary caller package, (not com.revrobotics), i could take another crack at it.) |
Having looked at a few open source vendors this comments seems very wrong. Both URCL and libgrapplefrc were using RuntimeLoader.
For a stable release? yes. For a beta release you should have one soon after the first WPILib beta release, around the end of August I believe? @PeterJohnson would be the one to ask.
Often these breaking changes are the new features it would really slow down development to not merge breaking changes after season it appears |
Something like https://github.com/CurtinFRC/2024-Offseason/blob/master/src/main/java/frc/robot/URCL.java ? |
the issue with rev is that the class with all the native methods (CANSparkMaxJNI, I think) extends RevJNIWrapper, which consists only of the broken loader in its static initializer. so I need to avoid loading the jni class at all, yet id like to call the same native code that its signature refers to, e.g. something like com_revrobotics_jni_cansparkmaxjni_c_sparkmax_setpointcommand(), within REVLibDriver. it looks like maybe the "foreign function" thing in java 19 would work. are we moving to java 19 anytime soon? |
We're looking at moving to the Java 21 runtime on the Rio for 2025 (we've already built the JVM). However, my understanding is that the new FFI interface does not support arm32 platforms like the Rio. |
Another workaround is to build and publish allwpilib locally with the RuntimeLoader change reverted. |
Fixed by #6930. |
I'm using the 2025 development version, and find that the NavX vendordep is now broken, due to 0e4d2c4, which removed RuntimeDetector.
This is AHRS.java:225, in the NavX vendordep. It uses RuntimeDetector, so it crashes on startup.
Any ideas to work around this? I'd like to use 2025 and I'd like to use NavX.
the stack trace
The text was updated successfully, but these errors were encountered: