-
Notifications
You must be signed in to change notification settings - Fork 37
Add Romi Deploy #192
Comments
I really enjoy this tool compared to frc-characterization. Is there are any way to bypass this limitation (e.g. using https://github.com/bb-frc-workshops/romi-examples/tree/main/romi-characterization)? |
To anyone who is interested in this - I did a quick experiment and was able to connect Logger but Romi didn't respond to any test. Later I found that there are some changes in data-collection (see #131), so most likely the answer to my own question is no (unless if you would like to understand all the changes that was done and re-implement them) |
This is really a shame. The main reason I purchased a Romi was to give our students an easy way to learn about characterization and trajectory following commands. When I discovered a few days ago that sysid does not support Romi, I too thought about working around this dropped functionality by using a modified version of https://github.com/bb-frc-workshops/romi-examples/tree/main/romi-characterization but it sounds like that won't work. Is there any prospect for sysid to support Romi in the foreseeable future? Thanks. |
None of the existing contributors is currently working on that. Modifying the romi-characterization repo to use the new NT data format would be sufficient. Porting the logging parts of https://github.com/wpilibsuite/sysid/tree/main/sysid-library to Java is an option as well. |
Thanks, Tyler. |
If you'd like you can try this:
Not sure if this will work but it might be able to run the sysid code for the Romi |
Thanks everyone for suggestions. I've tried all of them but was able to make progress only in porting changes back to Java. I'm still trying to figure out how to do the Quasistatic analysis right and would really appreciate any comments that you may have in this PR -> bb-frc-workshops/romi-examples#15. Thank you! |
OBE by #518. |
Currently there's no way to deploy the Romi code.
The text was updated successfully, but these errors were encountered: