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

[WIP] added gearboxes inertia #50

Closed
wants to merge 1 commit into from
Closed

Conversation

gabrielenava
Copy link
Collaborator

@gabrielenava gabrielenava commented Oct 10, 2018

added gearboxes inertia to the motors reflected inertia as suggested in this issue. Waiting for testing it on the robot before merging.

@FabioBergonti would you like to review this PR?

@FabioBergonti
Copy link
Member

Are you sure that all the motors have the same harmonic drive?
From the wiki seems that are two type of gears.
I don't know if that page is updated

@gabrielenava
Copy link
Collaborator Author

gabrielenava commented Oct 10, 2018

Mmmh, 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
Copy link
Collaborator Author

gabrielenava commented Oct 10, 2018

I will close this PR because I merged this modification in this other PR. I will copy-paste the comment about which harmonic drives are mounted on the robot.

@gabrielenava gabrielenava deleted the addGearboxInertia branch October 10, 2018 14:32
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