Remove dependency on rmf_dispenser_msgs

[Yadunund]

Fix #35

[codecov]

Codecov Report

Merging #36 (d7dceb1) into main (87a7cad) will increase coverage by 0.10%.
The diff coverage is 56.25%.

@@            Coverage Diff             @@
##             main      #36      +/-   ##
==========================================
+ Coverage   38.80%   38.90%   +0.10%     
==========================================
  Files          27       27              
  Lines        1500     1501       +1     
  Branches      904      905       +1     
==========================================
+ Hits          582      584       +2     
+ Misses        222      221       -1     
  Partials      696      696              

Flag	Coverage Δ
tests	38.90% <56.25%> (+0.10%)	⬆️

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

Impacted Files	Coverage Δ
rmf_task/test/unit/agv/test_TaskPlanner.cpp	24.25% <0.00%> (-0.06%)	⬇️
rmf_task/src/rmf_task/requests/Delivery.cpp	40.16% <60.00%> (+1.63%)	⬆️

[youliangtan]

LGTM! This looks cleaner after removing the dispenser_msgs, no more ros dep. Soon we can also let the task planner take account of the workcell execution wait time, cant wait for that!

[Yadunund]

The changes in this PR make the task planner account for the waiting period at the pickup and dropoff locations 😄

[youliangtan]

Ah I should hav said it clearer. The "workcell" execution time indirectly means pickup/dropoff duration. 😅