Skip to content

Conversation

bmagyar
Copy link
Member

@bmagyar bmagyar commented Sep 13, 2025

Example to add limiting & pal_statistics

@bmagyar bmagyar marked this pull request as draft September 13, 2025 23:08
Update joint_state_topic_hardware_interface.cpp

also export the limited commands

remove print

add limited commands array too

Update joint_state_topic_hardware_interface.hpp

Apply suggestions from code review
@bmagyar bmagyar force-pushed the add-limiter-example branch from 5a8df93 to 691dfd8 Compare September 13, 2025 23:16
@codecov-commenter
Copy link

codecov-commenter commented Sep 13, 2025

Codecov Report

❌ Patch coverage is 0% with 30 lines in your changes missing coverage. Please review.
✅ Project coverage is 0.00%. Comparing base (1f825e9) to head (e4d3b96).

Files with missing lines Patch % Lines
...rface/src/joint_state_topic_hardware_interface.cpp 0.00% 22 Missing ⚠️
...interface/joint_state_topic_hardware_interface.hpp 0.00% 8 Missing ⚠️
Additional details and impacted files
@@          Coverage Diff          @@
##            main     #29   +/-   ##
=====================================
  Coverage   0.00%   0.00%           
=====================================
  Files          1       2    +1     
  Lines        135     161   +26     
  Branches      35      38    +3     
=====================================
- Misses       135     161   +26     
Flag Coverage Δ
unittests 0.00% <0.00%> (ø)

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

Files with missing lines Coverage Δ
...interface/joint_state_topic_hardware_interface.hpp 0.00% <0.00%> (ø)
...rface/src/joint_state_topic_hardware_interface.cpp 0.00% <0.00%> (ø)
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

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