You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
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.
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.
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 :)
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.
The text was updated successfully, but these errors were encountered: