From cc6065099b440785d5edfccb120c33969950109a Mon Sep 17 00:00:00 2001 From: Mikolaj Wielgus Date: Sat, 16 Aug 2025 23:30:11 +0200 Subject: [PATCH] fix(router/thetastar): Don't forget to rm. probe when there's shorter path already This fixes a fatal race condition that made tht_de9_to_tht_de9 always fail since 7eff7ad9f0a24a9688ab5235f01115f7c85b1ff4. --- src/router/thetastar.rs | 6 ++++++ 1 file changed, 6 insertions(+) diff --git a/src/router/thetastar.rs b/src/router/thetastar.rs index 895a64b..e58dff5 100644 --- a/src/router/thetastar.rs +++ b/src/router/thetastar.rs @@ -319,6 +319,12 @@ where // No need to add neighbors that we have already reached through a // shorter path than now. if *entry.get() <= next_score { + // We just remove the probe instantly + // here instead of doing it in + // ThetastarState::Probing or a new + // state to avoid complicating. + strategy.remove_probe(&self.graph); + self.state = ThetastarState::VisitingProbeOnNavedge( visited_navnode, curr_navedge,