Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

[ForceTorqueSensorBroadcaster] Create ParamListener and get parameters on configure (backport #698) #750

Merged
merged 2 commits into from
Dec 5, 2023

Conversation

mergify[bot]
Copy link
Contributor

@mergify mergify bot commented Aug 18, 2023

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


Mergify commands and options

More conditions and actions can be found in the documentation.

You can also trigger Mergify actions by commenting on this pull request:

  • @Mergifyio refresh will re-evaluate the rules
  • @Mergifyio rebase will rebase this PR on its base branch
  • @Mergifyio update will merge the base branch into this PR
  • @Mergifyio backport <destination> will backport this PR on <destination> branch

Additionally, on Mergify dashboard you can:

  • look at your merge queues
  • generate the Mergify configuration with the config editor.

Finally, you can contact us on https://mergify.com

…s on configure (#698)

* Create ParamListener and get parameters on configure

* Declare parameters for test_force_torque_sensor_broadcaster

Since the parameters are not declared on init anymore, they cannot be
set without declaring them before

---------

Co-authored-by: Bence Magyar <bence.magyar.robotics@gmail.com>
(cherry picked from commit 32aaef7)
Copy link

codecov bot commented Dec 5, 2023

Codecov Report

Merging #750 (ad27118) into humble (13ba381) will increase coverage by 45.26%.
The diff coverage is 66.66%.

Additional details and impacted files
@@             Coverage Diff             @@
##           humble     #750       +/-   ##
===========================================
+ Coverage        0   45.26%   +45.26%     
===========================================
  Files           0       40       +40     
  Lines           0     3667     +3667     
  Branches        0     1732     +1732     
===========================================
+ Hits            0     1660     +1660     
- Misses          0      827      +827     
- Partials        0     1180     +1180     
Flag Coverage Δ
unittests 45.26% <66.66%> (?)

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

Files Coverage Δ
...roadcaster/src/force_torque_sensor_broadcaster.cpp 43.93% <66.66%> (ø)

... and 39 files with indirect coverage changes

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants