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

Control board named 'ankle_setup_control_board', works with linear motion in range 0-100 [mm], fixed indentation, no more skipping park #706

Merged
merged 1 commit into from
Jan 23, 2025

Conversation

AntonioViscomi
Copy link

With this PR the joint interprets setpoints as linear displacements within the range [-100, 0] mm, providing a more intuitive representation of motion.
To improve clarity, the control board has been renamed to ankle_setup_control_board.
Additionally, the joint parks correctly when exiting the yarprobotinterface, as shown in the debug logs:

[DEBUG] ankleSetupCalib : Moving to park position, joint: 0  
[DEBUG] ankle-setup-calibrator finished park device ankle-setup-mc_remapper  

Copy link
Member

@pattacini pattacini left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @AntonioViscomi

Apparently, you touched files belonging to R1SN003 that shouldn't have been modified.

Can you check that?

@pattacini
Copy link
Member

Judging from the commits...

image

...it seems that you have to rebase on top of upstream devel somehow.

…r motion in range 0-100 [mm], fixed indentation
@AntonioViscomi
Copy link
Author

AntonioViscomi commented Jan 23, 2025

...it seems that you have to rebase on top of upstream devel somehow.

Thanks for pointing this out to me. I think I mistakenly rebased on top of master. It should be fine now.

Copy link
Member

@pattacini pattacini left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great!

@pattacini pattacini merged commit acfc916 into robotology:devel Jan 23, 2025
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