open-rmf / rmf_traffic

Traffic management libraries for RMF
Apache License 2.0
28 stars 26 forks source link

missing updates for jammy #73

Closed marcoag closed 2 years ago

marcoag commented 2 years ago

Bug fix

Fixed bug

Compilation on jammy.

Fix applied

Adding some missing includes to optional and side improvements.

codecov[bot] commented 2 years ago

Codecov Report

Merging #73 (84ddec5) into main (2fe3162) will not change coverage. The diff coverage is 50.00%.

@@           Coverage Diff           @@
##             main      #73   +/-   ##
=======================================
  Coverage   22.62%   22.62%           
=======================================
  Files         364      364           
  Lines       30020    30020           
  Branches    13837    13836    -1     
=======================================
  Hits         6792     6792           
- Misses      13921    13940   +19     
+ Partials     9307     9288   -19     
Flag Coverage Δ
tests 22.62% <50.00%> (ø)

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

Impacted Files Coverage Δ
rmf_traffic/include/rmf_traffic/Route.hpp 0.00% <ø> (ø)
rmf_traffic/include/rmf_traffic/agv/Graph.hpp 18.18% <ø> (ø)
rmf_traffic/src/rmf_traffic/Trajectory.cpp 74.89% <ø> (ø)
...affic/agv/planning/impl_MinimalTravelHeuristic.hpp 0.00% <0.00%> (ø)
...thirdparty/fcl/include/fcl/geometry/shape/sphere.h 0.00% <ø> (ø)
rmf_traffic/src/rmf_traffic/DetectConflict.cpp 49.65% <100.00%> (+0.16%) :arrow_up:
rmf_traffic/test/unit/utils_Conflict.hpp 26.66% <0.00%> (ø)
rmf_traffic/test/unit/agv/test_Planner.cpp 13.02% <0.00%> (ø)
...traffic/test/unit/agv/test_compute_plan_starts.cpp 5.86% <0.00%> (ø)
... and 1 more