-
Notifications
You must be signed in to change notification settings - Fork 69
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
ergoCub1.0 S/N:000 – Update and improve configuration files #512
Conversation
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It looks good to me, but I would also wait for the other reviewers as they use the robot more often
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
In light of @AntonioConsilvio / @sgiraz clarifications lgtm now!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ok for me!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let me merge this PR straight away. |
What's new:
head-calib.xml
for this issueright_arm-eb22-j7_10-mec.xml
and "right_arm-eb22-j7_10-mc_service.xml" for this issueleft_arm-eb25-j11_12-mc_service.xml
for this issue2FOC
firmware version is now aligned in the configuration files with the one actually installed on the robot.In addition, I added the original calibration order in a commented line for each calibration file; to avoid the risk of entering an incorrect calibration sequence in the event of temporary changes to the calibration order.
Notes:
This was done with the aim of reducing the number of files in the
ergoCubSN000
configuration folder to make navigation easier and smoother. Please check among the deleted files whether they are still in use, as in that case they can be restored.cc @sgiraz @S-Dafarra @lrapetti @GiulioRomualdi @xEnVrE @mfussi66