-
Notifications
You must be signed in to change notification settings - Fork 205
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
PID controller and diff drive controller chaining #710
base: master
Are you sure you want to change the base?
Conversation
I think you found a bug, I proposed a fix in ros-controls/ros2_controllers#1513 |
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.
Thanks for your submission! Some thoughts:
- Please cleanup the docs to focus on the controller chain (e.g., remove the MockHardware part)
- Update the test_diffbot_launch.py to check for all controllers
- I suggest that you add some dynamics to the hardware interface, a first-order or second-order low pass, so that the PID controller actually makes sense.
When the following stuff is fixed/merged we should include to the docs (maybe in follow-up PRs)
- view_controller_chains output Fix view_controller_chains ros2_control#2026
- add plots of pal_statistics to show the PID controller action Integrate pal_statistics for introspection of controllers, hardware components and more ros2_control#1918
Thank you so much for responding so quickly. The configuration changes are really nice, love them. I tested the fix you made, the activation is working now. I'll continue the rest of work and include the suggestions that you made. Left some comments on your PR ros-controls/ros2_controllers#1513. |
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.
see #1528 for a proper fix of the interfaces. could you have a look there and review please?
], | ||
) | ||
|
||
# start the base controller in inactive mode |
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.
# start the base controller in inactive mode |
# "--inactive", | ||
"--ros-args", | ||
"--log-level", | ||
"debug", |
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.
# "--inactive", | |
"--ros-args", | |
"--log-level", | |
"debug", |
Address #568. This is to add a new example_16 based on the use case outlined in the controller manager doc section Motivation, Purpose and Use. It uses example_2 as a base and adds a chained 'diff_drive_controller' and two PID controllers with following flow.
This requires following changes from ros2_controllers:
New Examples
If you propose adding a new example, make sure that your new example has: