Skip to content
This repository has been archived by the owner on Oct 1, 2023. It is now read-only.

Add Romi Deploy #192

Closed
Piphi5 opened this issue Oct 15, 2021 · 8 comments
Closed

Add Romi Deploy #192

Piphi5 opened this issue Oct 15, 2021 · 8 comments
Assignees
Labels
component: generation Generation of robot projects

Comments

@Piphi5
Copy link
Collaborator

Piphi5 commented Oct 15, 2021

Currently there's no way to deploy the Romi code.

@veysby
Copy link

veysby commented Feb 2, 2022

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)?

@veysby
Copy link

veysby commented Feb 2, 2022

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)

@sickingp
Copy link

sickingp commented Feb 2, 2022

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.

@calcmogul
Copy link
Member

calcmogul commented Feb 2, 2022

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.

@sickingp
Copy link

sickingp commented Feb 3, 2022

Thanks, Tyler.

@Piphi5
Copy link
Collaborator Author

Piphi5 commented Feb 3, 2022

If you'd like you can try this:

  1. Cloning this repo
  2. Putting the attached config file in sysid-projects/deploy
  3. Running ./gradlew :sysid-projects:drive:simulateNativeRelease -Pintegration & to launch the simulation GUI of the Romi project

config.json.txt

Not sure if this will work but it might be able to run the sysid code for the Romi

@veysby
Copy link

veysby commented Feb 7, 2022

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!

@calcmogul calcmogul added the component: generation Generation of robot projects label Sep 20, 2023
@calcmogul
Copy link
Member

OBE by #518.

@calcmogul calcmogul closed this as not planned Won't fix, can't repro, duplicate, stale Sep 21, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
component: generation Generation of robot projects
Projects
None yet
Development

No branches or pull requests

5 participants