UniversalRobots / Universal_Robots_Client_Library

A C++ library for accessing the UR interfaces that facilitate the use of UR robotic manipulators by external applications.
Apache License 2.0
117 stars 88 forks source link

Specifically set RTDE pipeline producer to FIFO scheduling #139

Closed fmauch closed 1 year ago

fmauch commented 1 year ago

Regardless of the actual kernel used, setting the producer thread to FIFO scheduling seems to improve robustness.

codecov[bot] commented 1 year ago

Codecov Report

Patch coverage: 70.45% and project coverage change: +0.01 :tada:

Comparison is base (2e69bc1) 77.48% compared to head (0141342) 77.49%.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #139 +/- ## ========================================== + Coverage 77.48% 77.49% +0.01% ========================================== Files 95 96 +1 Lines 3939 3959 +20 Branches 385 385 ========================================== + Hits 3052 3068 +16 - Misses 662 667 +5 + Partials 225 224 -1 ``` | [Impacted Files](https://app.codecov.io/gh/UniversalRobots/Universal_Robots_Client_Library/pull/139?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=UniversalRobots) | Coverage Δ | | |---|---|---| | [include/ur\_client\_library/queue/atomicops.h](https://app.codecov.io/gh/UniversalRobots/Universal_Robots_Client_Library/pull/139?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=UniversalRobots#diff-aW5jbHVkZS91cl9jbGllbnRfbGlicmFyeS9xdWV1ZS9hdG9taWNvcHMuaA==) | `63.88% <ø> (-1.39%)` | :arrow_down: | | [src/helpers.cpp](https://app.codecov.io/gh/UniversalRobots/Universal_Robots_Client_Library/pull/139?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=UniversalRobots#diff-c3JjL2hlbHBlcnMuY3Bw) | `50.00% <50.00%> (ø)` | | | [include/ur\_client\_library/comm/pipeline.h](https://app.codecov.io/gh/UniversalRobots/Universal_Robots_Client_Library/pull/139?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=UniversalRobots#diff-aW5jbHVkZS91cl9jbGllbnRfbGlicmFyeS9jb21tL3BpcGVsaW5lLmg=) | `71.91% <94.73%> (+6.31%)` | :arrow_up: | | [src/rtde/rtde\_client.cpp](https://app.codecov.io/gh/UniversalRobots/Universal_Robots_Client_Library/pull/139?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=UniversalRobots#diff-c3JjL3J0ZGUvcnRkZV9jbGllbnQuY3Bw) | `55.97% <100.00%> (ø)` | | ... and [2 files with indirect coverage changes](https://app.codecov.io/gh/UniversalRobots/Universal_Robots_Client_Library/pull/139/indirect-changes?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=UniversalRobots)

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.