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

Cannot set PID gains fast #82

Open
azompas opened this issue Jun 18, 2019 · 3 comments
Open

Cannot set PID gains fast #82

azompas opened this issue Jun 18, 2019 · 3 comments

Comments

@azompas
Copy link

azompas commented Jun 18, 2019

Hello,

I am working on a some kind of adaptive PID control strategy where I need to set the PID gains fast. However the update of the dynamic reconfigure takes too much time, making my controller to not behave as it should.

https://github.com/ros-controls/control_toolbox/blob/kinetic-devel/src/pid.cpp#L241

Would it be possible to provide a function without the update dynamic reconfigure functionality?

Thank you.

@bmagyar
Copy link
Member

bmagyar commented Jun 19, 2019

Are you sure this is happening because of the reconfigure step and not the mutex-protected realtime publisher?

@azompas
Copy link
Author

azompas commented Jun 20, 2019

Yes, I commented out the dynamic reconfigure and my controller works as expected.
Would it be possible to have an option to disable the reconfigure part?
Or, if you are willing to merge, we could write a simpler class without the dynamic reconfigure part and make a pull request.
I believe that this functionality will be useful to anyone who wants to create an adaptive PID controller.

@bmagyar
Copy link
Member

bmagyar commented Jul 2, 2019

I would prefer if you could implement this by introducing a new flag to the class named disable_dynamic_reconfigure and have it set to true by default (so that we don't change the existing default behaviour).

Happy to review and host it with the official packages :)

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

No branches or pull requests

2 participants