search

micro-ROS / system_modes

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

Separate interface package system_modes_msgs #74

Closed norro closed 3 years ago

norro commented 3 years ago

Separated msg and srv types into seperate package system_modes_msgs. This seems to be the ROS 2 way anyway, was requiring a hack beforehand, and seems necessary to get the windows build working. (I have a working windows build based on this change now)

codecov-commenter commented 3 years ago

Codecov Report

Merging #74 (beda889) into master (2917384) will decrease coverage by 0.76%. The diff coverage is 0.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #74      +/-   ##
==========================================
- Coverage   32.63%   31.87%   -0.77%     
==========================================
  Files          14       14              
  Lines        1048     1048              
  Branches      754      754              
==========================================
- Hits          342      334       -8     
- Misses        132      136       +4     
- Partials      574      578       +4
Impacted Files Coverage Δ
system_modes/include/system_modes/mode_manager.hpp 100.00% <ø> (ø)
system_modes/include/system_modes/mode_monitor.hpp 100.00% <ø> (ø)
...ystem_modes/include/system_modes/mode_observer.hpp 100.00% <ø> (ø)
system_modes/src/system_modes/mode_manager.cpp 17.29% <0.00%> (-1.13%) :arrow_down:
system_modes/src/system_modes/mode_monitor.cpp 26.98% <ø> (ø)
system_modes/src/system_modes/mode_observer.cpp 20.00% <ø> (ø)
system_modes/include/system_modes/mode_impl.hpp 81.48% <0.00%> (-3.71%) :arrow_down:
system_modes/src/system_modes/mode_inference.cpp 35.96% <0.00%> (-1.09%) :arrow_down:
... and 1 more

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 2917384...beda889. Read the comment docs.