Skip to content

Conversation

@mergify
Copy link
Contributor

@mergify mergify bot commented Nov 28, 2025

Address #1842

This PR adds a new utility node wrench_transformer_node to force_torque_sensor_broadcaster to transform input Wrench messages to output Wrench messages for a given list of frames.

Feedback on naming is welcome and appreciated.

Testing


This is an automatic backport of pull request #2021 done by [Mergify](https://mergify.com).

…2021)

(cherry picked from commit 3d03572)

# Conflicts:
#	doc/release_notes.rst
@mergify mergify bot added the conflicts label Nov 28, 2025
@mergify
Copy link
Contributor Author

mergify bot commented Nov 28, 2025

Cherry-pick of 3d03572 has failed:

On branch mergify/bp/kilted/pr-2021
Your branch is up to date with 'origin/kilted'.

You are currently cherry-picking commit 3d03572.
  (fix conflicts and run "git cherry-pick --continue")
  (use "git cherry-pick --skip" to skip this patch)
  (use "git cherry-pick --abort" to cancel the cherry-pick operation)

Changes to be committed:
	modified:   force_torque_sensor_broadcaster/CMakeLists.txt
	modified:   force_torque_sensor_broadcaster/README.md
	modified:   force_torque_sensor_broadcaster/doc/userdoc.rst
	new file:   force_torque_sensor_broadcaster/include/force_torque_sensor_broadcaster/wrench_transformer.hpp
	modified:   force_torque_sensor_broadcaster/package.xml
	new file:   force_torque_sensor_broadcaster/src/wrench_transformer.cpp
	new file:   force_torque_sensor_broadcaster/src/wrench_transformer_main.cpp
	new file:   force_torque_sensor_broadcaster/src/wrench_transformer_parameters.yaml
	new file:   force_torque_sensor_broadcaster/test/test_wrench_transformer.cpp

Unmerged paths:
  (use "git add <file>..." to mark resolution)
	both modified:   doc/release_notes.rst

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
Copy link

codecov bot commented Nov 30, 2025

Codecov Report

❌ Patch coverage is 88.10198% with 42 lines in your changes missing coverage. Please review.
✅ Project coverage is 85.20%. Comparing base (32a862f) to head (61ceae7).
⚠️ Report is 1 commits behind head on kilted.

Files with missing lines Patch % Lines
...ensor_broadcaster/test/test_wrench_transformer.cpp 91.27% 14 Missing and 10 partials ⚠️
...rque_sensor_broadcaster/src/wrench_transformer.cpp 78.66% 8 Missing and 8 partials ⚠️
...sensor_broadcaster/src/wrench_transformer_main.cpp 0.00% 2 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           kilted    #2032      +/-   ##
==========================================
+ Coverage   85.13%   85.20%   +0.07%     
==========================================
  Files         144      148       +4     
  Lines       13968    14321     +353     
  Branches     1201     1225      +24     
==========================================
+ Hits        11891    12202     +311     
- Misses       1670     1694      +24     
- Partials      407      425      +18     
Flag Coverage Δ
unittests 85.20% <88.10%> (+0.07%) ⬆️

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

Files with missing lines Coverage Δ
...e_torque_sensor_broadcaster/wrench_transformer.hpp 100.00% <100.00%> (ø)
...sensor_broadcaster/src/wrench_transformer_main.cpp 0.00% <0.00%> (ø)
...rque_sensor_broadcaster/src/wrench_transformer.cpp 78.66% <78.66%> (ø)
...ensor_broadcaster/test/test_wrench_transformer.cpp 91.27% <91.27%> (ø)
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@christophfroehlich christophfroehlich merged commit 380d29a into kilted Nov 30, 2025
14 checks passed
@christophfroehlich christophfroehlich deleted the mergify/bp/kilted/pr-2021 branch November 30, 2025 19:13
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