Skip to content
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

Merged
merged 34 commits into from
Jan 10, 2019
Merged

Conversation

gabrielenava
Copy link
Collaborator

@gabrielenava gabrielenava commented Sep 20, 2018

Following the issues:

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:

  • add info to understand if the simulink model is still compiling;
  • close the model when the user closes the GUI;
  • add a warning message if the required integration step is not respected.

@gabrielenava gabrielenava self-assigned this Sep 20, 2018
@gabrielenava
Copy link
Collaborator Author

gabrielenava commented Oct 10, 2018

The two types of harmonic drives listed in the wiki page are the CSD-14 and CSD-17. Their inertia is indeed different:

screenshot from 2018-10-10 13-26-06

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?

@gabrielenava gabrielenava changed the title [WIP] Static GUI for Simulink [WIP] Static GUI for Simulink and add gearboxes inertia Oct 10, 2018
@gabrielenava
Copy link
Collaborator Author

gabrielenava commented Oct 11, 2018

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:

  • there is a third type of harmonic drive that is not included in the wiki. It is used in the forearm/neck joints, so it is not of our interest at the moment.

  • there is a 1:1 correspondence between motor and harmonic drive, i.e. the CSD-14 type is used only with one type of motor and the CSD-17 is used only with another type of motor.

  • there are only 2 type of motors and 2 types of harmonic drives used for the robot shoulder, elbow, legs and torso joints.

Based on these information, I concluded that it was sufficient to check the elbow and hip joints in the CAD, that have two different type of motors (the MOOG equivalent of the KOLLMORGEN RBE-01210 and the MOOG equivalent of the KOLLMORGEN RBE-01211, respectively), and check which harmonic drive is associated with which motor. Note that differently from what is written in the wiki we currently mount MOOG motors on the iCub, not KOLLMORGEN.

In particular, according to my reasoning it seems that the map is the following:

  • torso yaw, shoulder and elbow ---> CSD-14

  • legs, torso pitch and roll ---> CSD-17

I will apply this logic for the moment. @fiorisi can you later confirm that this mapping is correct?

cc @FabioBergonti

@gabrielenava
Copy link
Collaborator Author

Both the GUI and the gearbox inertia have been tested on iCubGenova04 for performing the Yoga ++ demo. The demo was performed as usual, and there were no qualitative differences. Before merging, I would like to fix a couple of issue concerning the GUI.

@gabrielenava
Copy link
Collaborator Author

All possible tests in simulation have been performed. We need to test this PR on the real iCub before merging.

@gabrielenava
Copy link
Collaborator Author

The PR passed the tests on iCubGenova04. I will anyways wait for a testing period before merging.

@gabrielenava gabrielenava changed the title [WIP] Static GUI for Simulink and add gearboxes inertia [UNDER TESTING] Static GUI for Simulink and add gearboxes inertia Dec 19, 2018
@gabrielenava
Copy link
Collaborator Author

I created a release of the current master branch. The controllers have been successfully tested more than one time on iCubGenova04, so I think it is time to merge this PR.

@gabrielenava gabrielenava changed the title [UNDER TESTING] Static GUI for Simulink and add gearboxes inertia Static GUI for Simulink and add gearboxes inertia Jan 10, 2019
@gabrielenava gabrielenava merged commit 88dfec5 into master Jan 10, 2019
@gabrielenava gabrielenava deleted the noSimulinkGui branch January 10, 2019 18:13
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