search

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

Make URSim log files available as artifacts #149

Closed fmauch closed 1 year ago

fmauch commented 1 year ago

This is an attempt to make debugging failing tests easier.

codecov[bot] commented 1 year ago

Codecov Report

Patch coverage: 69.38% and project coverage change: +0.03 :tada:

Comparison is base (8dbe2a1) 78.36% compared to head (a0c6487) 78.39%.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #149 +/- ## ========================================== + Coverage 78.36% 78.39% +0.03% ========================================== Files 100 101 +1 Lines 4409 4434 +25 Branches 449 450 +1 ========================================== + Hits 3455 3476 +21 - Misses 690 694 +4 Partials 264 264 ``` | [Impacted Files](https://app.codecov.io/gh/UniversalRobots/Universal_Robots_Client_Library/pull/149?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=UniversalRobots) | Coverage Δ | | |---|---|---| | [examples/dashboard\_example.cpp](https://app.codecov.io/gh/UniversalRobots/Universal_Robots_Client_Library/pull/149?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=UniversalRobots#diff-ZXhhbXBsZXMvZGFzaGJvYXJkX2V4YW1wbGUuY3Bw) | `25.80% <0.00%> (-1.78%)` | :arrow_down: | | [include/ur\_client\_library/ur/dashboard\_client.h](https://app.codecov.io/gh/UniversalRobots/Universal_Robots_Client_Library/pull/149?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=UniversalRobots#diff-aW5jbHVkZS91cl9jbGllbnRfbGlicmFyeS91ci9kYXNoYm9hcmRfY2xpZW50Lmg=) | `100.00% <ø> (ø)` | | | [src/helpers.cpp](https://app.codecov.io/gh/UniversalRobots/Universal_Robots_Client_Library/pull/149?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/149?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/149?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%> (ø)` | | | [src/ur/dashboard\_client.cpp](https://app.codecov.io/gh/UniversalRobots/Universal_Robots_Client_Library/pull/149?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=UniversalRobots#diff-c3JjL3VyL2Rhc2hib2FyZF9jbGllbnQuY3Bw) | `61.69% <100.00%> (+0.46%)` | :arrow_up: | ... and [3 files with indirect coverage changes](https://app.codecov.io/gh/UniversalRobots/Universal_Robots_Client_Library/pull/149/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.

fmauch commented 1 year ago

@urrsk I didn't use the dashboard_client functionality as it was easier to do it using a direct netcat call from the command line, but I think it is still fine to keep this inside this PR. We'll simply merge this using both commits.

urrsk commented 1 year ago

Maybe the script should be placed in the script folder..