Fix comparator for direct assignment ordering (#288)

Signed-off-by: Yadunund <yadunund@openrobotics.org>
open-rmf · Jul 27, 2023 · 4f92a92 · 4f92a92
1 parent 6caa604
commit 4f92a92
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/rmf_fleet_adapter/src/rmf_fleet_adapter/TaskManager.hpp b/rmf_fleet_adapter/src/rmf_fleet_adapter/TaskManager.hpp
@@ -83,18 +83,18 @@ class TaskManager : public std::enable_shared_from_this<TaskManager>
   {
     bool operator()(const DirectAssignment& a, const DirectAssignment& b) const
     {
-      // Sort by start time and then sequence_number if tie
+      // Sort by ascending order of start time and then sequence_number if tie.
       const auto start_time_b =
         b.assignment.request()->booking()->earliest_start_time();
       const auto start_time_a =
         a.assignment.request()->booking()->earliest_start_time();
 
       if (start_time_b == start_time_a)
       {
-        return b.sequence_number < a.sequence_number;
+        return a.sequence_number < b.sequence_number;
       }
 
-      return start_time_b < start_time_a;
+      return start_time_a < start_time_b;
     }
   };