Skip to content

Add AsyncFunctionHandlerParams initialization through ROS parameters (backport #402)#413

Merged
saikishor merged 1 commit into
jazzyfrom
mergify/bp/jazzy/pr-402
Sep 25, 2025
Merged

Add AsyncFunctionHandlerParams initialization through ROS parameters (backport #402)#413
saikishor merged 1 commit into
jazzyfrom
mergify/bp/jazzy/pr-402

Conversation

@mergify

@mergify mergify Bot commented Sep 25, 2025

Copy link
Copy Markdown

I've added a way to initialize the current parameter from the ROS nodes directly, much easier when using with nodes and controllers


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

@codecov-commenter

codecov-commenter commented Sep 25, 2025

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 64.86486% with 13 lines in your changes missing coverage. Please review.
✅ Project coverage is 84.65%. Comparing base (471295c) to head (b79332d).
⚠️ Report is 25 commits behind head on jazzy.

Files with missing lines Patch % Lines
.../include/realtime_tools/async_function_handler.hpp 56.66% 5 Missing and 8 partials ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##            jazzy     #413      +/-   ##
==========================================
- Coverage   85.63%   84.65%   -0.98%     
==========================================
  Files          19       19              
  Lines        1448     1499      +51     
  Branches      128      146      +18     
==========================================
+ Hits         1240     1269      +29     
- Misses        124      133       +9     
- Partials       84       97      +13     
Flag Coverage Δ
unittests 84.65% <64.86%> (-0.98%) ⬇️

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

Files with missing lines Coverage Δ
...ealtime_tools/test/test_async_function_handler.cpp 100.00% <100.00%> (+1.31%) ⬆️
.../include/realtime_tools/async_function_handler.hpp 74.87% <56.66%> (-7.40%) ⬇️
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@saikishor saikishor merged commit 6bccbc4 into jazzy Sep 25, 2025
13 checks passed
@saikishor saikishor deleted the mergify/bp/jazzy/pr-402 branch September 25, 2025 09:43
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.

2 participants