Skip to content

Add thread renaming in async function handler (backport #550) - #558

Merged
christophfroehlich merged 1 commit into
jazzyfrom
mergify/bp/jazzy/pr-550
Jul 30, 2026
Merged

Add thread renaming in async function handler (backport #550)#558
christophfroehlich merged 1 commit into
jazzyfrom
mergify/bp/jazzy/pr-550

Conversation

@mergify

@mergify mergify Bot commented Jul 30, 2026

Copy link
Copy Markdown

Description

Add thread renaming capabilities in async_function_handler

  • by default, the component name truncated to 15 chars
  • otherwise, configurable from yaml, alongside other async params

Tied to ros-controls/ros2_control#3482 , look there for explanation and demo

Did you use Generative AI?

Yeah, for the windows and macOS versions of thread rename. Used Gemini Pro. Did not test those versions, though.


This is an automatic backport of pull request #550 done by Mergify.

@mergify

mergify Bot commented Jul 30, 2026

Copy link
Copy Markdown
Author

Tick the box to add this pull request to the merge queue (same as @mergifyio queue).

  • Queue this pull request

@codecov-commenter

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 0% with 16 lines in your changes missing coverage. Please review.
✅ Project coverage is 83.97%. Comparing base (0a48300) to head (5806f96).

Files with missing lines Patch % Lines
realtime_tools/src/realtime_helpers.cpp 0.00% 9 Missing ⚠️
.../include/realtime_tools/async_function_handler.hpp 0.00% 5 Missing and 2 partials ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##            jazzy     #558      +/-   ##
==========================================
- Coverage   84.99%   83.97%   -1.02%     
==========================================
  Files          19       19              
  Lines        1519     1535      +16     
  Branches      142      147       +5     
==========================================
- Hits         1291     1289       -2     
- Misses        135      150      +15     
- Partials       93       96       +3     
Flag Coverage Δ
unittests 83.97% <0.00%> (-1.02%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines Coverage Δ
.../include/realtime_tools/async_function_handler.hpp 72.09% <0.00%> (-2.43%) ⬇️
realtime_tools/src/realtime_helpers.cpp 34.21% <0.00%> (-2.94%) ⬇️

... and 1 file with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@christophfroehlich
christophfroehlich merged commit cc3521e into jazzy Jul 30, 2026
14 checks passed
@christophfroehlich
christophfroehlich deleted the mergify/bp/jazzy/pr-550 branch July 30, 2026 11:27
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