-
Notifications
You must be signed in to change notification settings - Fork 366
Add detach async policy for rate critical frameworks (backport #2477) #2600
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
Conversation
(cherry picked from commit aebc2b8) # Conflicts: # doc/migration.rst # doc/release_notes.rst
Cherry-pick of aebc2b8 has failed:
To fix up this pull request, you can check it out locally. See documentation: https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/reviewing-changes-in-pull-requests/checking-out-pull-requests-locally |
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## jazzy #2600 +/- ##
=======================================
Coverage 89.04% 89.04%
=======================================
Files 147 147
Lines 16594 16638 +44
Branches 1400 1405 +5
=======================================
+ Hits 14776 14816 +40
Misses 1288 1288
- Partials 530 534 +4
Flags with carried forward coverage won't be shown. Click here to find out more.
🚀 New features to boost your workflow:
|
For frameworks such as EtherCAT that run at high frequency. if the deadline is not met, then the master will start showing errors and eventually it might lead to non-functional system. For this reason, instead of the previous approach where the CM acts as a scheduler for the async activity, I've added a possibility for the component to manage it's own cycle
Needs: ros-controls/realtime_tools#383
Needs: ros-controls/realtime_tools#402
This is an automatic backport of pull request #2477 done by Mergify.