-
Notifications
You must be signed in to change notification settings - Fork 276
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
removed unneeded plugin update #944
Conversation
Signed-off-by: Jenn Nguyen <[email protected]>
Signed-off-by: Jenn Nguyen <[email protected]>
Signed-off-by: Louise Poubel <[email protected]>
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 cleaning this up! Works for me. Let's just wait for CI results.
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.
ran into this issue on ign-gazebo4 so ported the fix over and confirmed that the issue is fixed.
JointPositionController tests are failing on CI though
Signed-off-by: Louise Poubel <[email protected]>
Should be fixed by waiting for updates in 20cdc9f |
Codecov Report
@@ Coverage Diff @@
## ign-gazebo3 #944 +/- ##
===============================================
- Coverage 77.84% 77.80% -0.05%
===============================================
Files 220 220
Lines 12589 12579 -10
===============================================
- Hits 9800 9787 -13
- Misses 2789 2792 +3
Continue to review full report at Codecov.
|
🦟 Bug fix
Summary
When a plugin was added an
Update
was called immediately. With #497 updates are also being called in a loop, which was causing deadlocks since both could be called at the same time. This PR removes the initial update (after the plugin is inserted) since updates are already happening periodically.Checklist
codecheck
passed (See contributing)Note to maintainers: Remember to use Squash-Merge