Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

Add method to get the current callback time and period #192

Merged
merged 2 commits into from
Nov 14, 2024

Conversation

saikishor
Copy link
Member

@saikishor saikishor commented Nov 8, 2024

This would be needed for the statistics of the controllers and the hardware components

@codecov-commenter
Copy link

codecov-commenter commented Nov 8, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 73.11%. Comparing base (c0d6b2b) to head (322b5d2).

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #192      +/-   ##
==========================================
+ Coverage   72.97%   73.11%   +0.13%     
==========================================
  Files           8        8              
  Lines         396      398       +2     
  Branches       65       65              
==========================================
+ Hits          289      291       +2     
  Misses         68       68              
  Partials       39       39              
Flag Coverage Δ
unittests 73.11% <100.00%> (+0.13%) ⬆️

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 91.46% <100.00%> (+0.21%) ⬆️

@saikishor saikishor force-pushed the retrieve/more/async/data branch from 33b955e to 2427a3e Compare November 8, 2024 15:24
@saikishor saikishor marked this pull request as ready for review November 10, 2024 16:39
@bmagyar bmagyar merged commit b3a342a into ros-controls:master Nov 14, 2024
26 checks passed
@saikishor saikishor deleted the retrieve/more/async/data branch November 14, 2024 16:10
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.

4 participants