-
Notifications
You must be signed in to change notification settings - Fork 368
Documentation for model-based controllers #1650
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
Conversation
(cherry picked from commit 5178503) Co-authored-by: Christoph Fröhlich <[email protected]>
Co-authored-by: Bence Magyar <[email protected]> (cherry picked from commit af54985) Co-authored-by: Christoph Fröhlich <[email protected]>
…eriod_ in diff_drive_controller.cpp (ros-controls#644) (ros-controls#777) Co-authored-by: Jules CARPENTIER <[email protected]> (cherry picked from commit 77420cb) Co-authored-by: julescarpentier <[email protected]>
…r.yaml. (ros-controls#822) (ros-controls#823) (cherry picked from commit 0196eeb) Co-authored-by: Tony Baltovski <[email protected]>
…er works. (ros-controls#756) (ros-controls#868) (cherry picked from commit 6352a70) Co-authored-by: chama1176 <[email protected]>
…s on configure (backport ros-controls#698) (ros-controls#750) * [ForceTorqueSensorBroadcaster] Create ParamListener and get parameters on configure (ros-controls#698) * Create ParamListener and get parameters on configure * Declare parameters for test_force_torque_sensor_broadcaster Since the parameters are not declared on init anymore, they cannot be set without declaring them before --------- Co-authored-by: Bence Magyar <[email protected]> (cherry picked from commit 32aaef7) * Fix "parameter is already declared" error --------- Co-authored-by: Noel Jiménez García <[email protected]> Co-authored-by: Christoph Froehlich <[email protected]>
…ros-controls#1596) (ros-controls#1597) Co-authored-by: Felix Exner (fexner) <[email protected]>
…1454) (ros-controls#1460) Co-authored-by: Christoph Fröhlich <[email protected]>
Update documentation to include dynamics-based controllers
@v8p1197, all pull requests must be targeted towards the |
This pull request is in conflict. Could you fix it @v8p1197? |
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.
@v8p1197 you need to open the PRs targeting the master branch. I recommend creating a new PR using the new API of the master branch. We can later backport the changes to older distros
Thank you @saikishor. I have created a new PR. |
This PR updates
ros2_controller
's documentation to open the possibility of creating model-based controllers, e.g. gravity compensantion PD controllers or inverse dynamics controllers.To access the robot dynamics model, the inverse dynamics solver is proposed (here the release repository).
The solver will be soon released via
rosdep
forhumble
(here the updatedrosdistro
index).