Skip to content

[JTC] Disable use of PID adapter if in open-loop mode (backport #551)#740

Merged
bmagyar merged 2 commits intoros-controls:humblefrom
christophfroehlich:bp/pr-551
Aug 7, 2023
Merged

[JTC] Disable use of PID adapter if in open-loop mode (backport #551)#740
bmagyar merged 2 commits intoros-controls:humblefrom
christophfroehlich:bp/pr-551

Conversation

@christophfroehlich
Copy link
Copy Markdown
Member

A manual backport of #551 without merge conflicts.

@christophfroehlich
Copy link
Copy Markdown
Member Author

Failing CI is due to flaky tests, but imho not directly related to this change because open_loop is never set

Copy link
Copy Markdown
Member

@bmagyar bmagyar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thanks!

@bmagyar
Copy link
Copy Markdown
Member

bmagyar commented Aug 7, 2023

Tests were at first failing on JTC but now they are all cleared up, the flaky admittance controller test won't block merging

@bmagyar bmagyar merged commit 2889038 into ros-controls:humble Aug 7, 2023
@christophfroehlich christophfroehlich deleted the bp/pr-551 branch August 8, 2023 05:44
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.

3 participants