fix tie and exatrakx?

AJPfleger · Sep 8, 2024 · 8b937bd · 8b937bd
1 parent f768342
commit 8b937bd
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 6 deletions.
diff --git a/Core/include/Acts/Seeding/HoughTransformUtils.ipp b/Core/include/Acts/Seeding/HoughTransformUtils.ipp
@@ -264,10 +264,9 @@ Acts::HoughTransformUtils::PeakFinders::IslandsAroundMax<
     }
   }
   // sort the candidate cells descending in content
-  std::ranges::sort(
-      candidates, [&yieldMap](std::size_t bin1, std::size_t bin2) {
-        return std::tie(yieldMap[bin1], bin1) > std::tie(yieldMap[bin2], bin2);
-      });
+  std::ranges::sort(candidates, {}, [&yieldMap](std::size_t c) {
+    return std::make_tuple(yieldMap[c], c);
+  });
 
   // now we build islands from the candidate cells, starting with the most
   // populated one

diff --git a/Examples/Algorithms/TrackFindingExaTrkX/src/PrototracksToParameters.cpp b/Examples/Algorithms/TrackFindingExaTrkX/src/PrototracksToParameters.cpp
@@ -98,7 +98,7 @@ ProcessCode PrototracksToParameters::execute(
     // case, we want to keep the whole prototrack. Therefore, we operate on a
     // tmpTrack.
     std::ranges::sort(track, {}, [&](const auto &t) {
-      return std::tie(indexToGeoId[t].volume(), indexToGeoId[t].layer());
+      return std::make_tuple(indexToGeoId[t].volume(), indexToGeoId[t].layer());
     });
 
     tmpTrack.clear();

diff --git a/Examples/Algorithms/TrackFindingExaTrkX/src/TruthGraphBuilder.cpp b/Examples/Algorithms/TrackFindingExaTrkX/src/TruthGraphBuilder.cpp
@@ -85,7 +85,7 @@ std::vector<std::int64_t> TruthGraphBuilder::buildFromMeasurements(
 
     // Sort by radius (this breaks down if the particle has to low momentum)
     std::ranges::sort(track, {},
-                      [](const auto& t) { return radiusForOrdering(t); });
+                      [&](const auto& t) { return radiusForOrdering(t); });
 
     if (m_cfg.uniqueModules) {
       auto newEnd = std::unique(