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

Fix bug in admittance controller access to reference interfaces #1434

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

marip8
Copy link

@marip8 marip8 commented Dec 19, 2024

This PR fixes an issue in the admittance controller where it always tries to access both the position and velocity reference interfaces even if one of them is not provided. Currently, if the chainable_command_interfaces parameter only includes position or velocity, the admittance controller will complain with the following message:

Export of the state or reference interfaces failed with following error: The internal storage for reference values 'reference_interfaces_' variable has size '6', but it is expected to have the size '12' equal to the number of exported reference interfaces. Please correct and recompile the controller with name 'admittance_controller' and try again.

To fix this issue, the controller should only access the reference interface(s) that match its chainable command interface(s)

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.

1 participant