-
Notifications
You must be signed in to change notification settings - Fork 42
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
Static GUI for Simulink and add gearboxes inertia #42
Conversation
The two types of harmonic drives listed in the wiki page are the I think it is better to ask if there is a table mapping the type of harmonic drive with the associated joint on the robot. @fiorisi do you know where we can retrieve this information? |
I asked the mechanics about a way of knowing which gearbox is associated with which joint. They told me one way is to check the CAD model (long) or to look is someone already documented this info somewhere. However, they also told me what follows:
Based on these information, I concluded that it was sufficient to check the In particular, according to my reasoning it seems that the map is the following:
I will apply this logic for the moment. @fiorisi can you later confirm that this mapping is correct? |
Both the GUI and the gearbox inertia have been tested on |
All possible tests in simulation have been performed. We need to test this PR on the real iCub before merging. |
The PR passed the tests on |
I created a release of the current |
Following the issues:
Simulink takes a lot of time in some iterations wb-toolbox#160
Check if the Simulink required integration step is respected #38
Update all models to launch Simulink from command line #39
I created a static gui that allows to handle a Simulink model (run and stop) without the need of opening the Simulink model interface. This is sufficient to avoid the occasional blocking of Simulink while running, that seems to be related to the real-time update of the Simulink GUI. This new GUI is instead static, i.e. it is never updated while the Simulink model is running.
Also, following #49 I added the gearboxes inertia to the motors reflected inertia calculation.
TO DO: