micro-ROS / system_modes

System modes for ROS 2 and micro-ROS
Apache License 2.0
43 stars 9 forks source link

Modes observer #68

Closed norro closed 3 years ago

norro commented 3 years ago

Introduces a modes observer class that

The mode observer is supposed to be instantiated by+inside a ROS 2 node.

https://github.com/micro-ROS/system_modes/issues/66

codecov[bot] commented 3 years ago

Codecov Report

Merging #68 (7e94b56) into master (8ef3b27) will decrease coverage by 0.15%. The diff coverage is 18.86%.

:exclamation: Current head 7e94b56 differs from pull request most recent head c298443. Consider uploading reports for the commit c298443 to get more accurate results Impacted file tree graph

@@            Coverage Diff             @@
##           master      #68      +/-   ##
==========================================
- Coverage   27.63%   27.48%   -0.16%     
==========================================
  Files          12       14       +2     
  Lines         995     1048      +53     
  Branches      714      754      +40     
==========================================
+ Hits          275      288      +13     
+ Misses        275      272       -3     
- Partials      445      488      +43     
Impacted Files Coverage Δ
system_modes/src/system_modes/mode_observer.cpp 14.00% <14.00%> (ø)
system_modes/include/system_modes/mode_impl.hpp 81.48% <100.00%> (+1.48%) :arrow_up:
...ystem_modes/include/system_modes/mode_observer.hpp 100.00% <100.00%> (ø)
system_modes/src/system_modes/mode_manager.cpp 16.54% <0.00%> (ø)
system_modes/src/system_modes/mode_monitor.cpp 0.79% <0.00%> (ø)
system_modes/src/system_modes/mode_inference.cpp 35.69% <0.00%> (+0.81%) :arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 8ef3b27...c298443. Read the comment docs.