Skip to content

Commit

Permalink
fix: Reset navigation state correctly in KF reverse pass (#2368)
Browse files Browse the repository at this point in the history
We use a different abort mechanism in the KF and do not need to tell the navigator about the target surface

Pulled this out of #2336 where it fixed a bug
  • Loading branch information
andiwand committed Aug 17, 2023
1 parent 0eb5954 commit a22a8be
Showing 1 changed file with 3 additions and 1 deletion.
4 changes: 3 additions & 1 deletion Core/include/Acts/TrackFitting/KalmanFitter.hpp
Original file line number Diff line number Diff line change
Expand Up @@ -535,10 +535,12 @@ class KalmanFitter {
result.passedAgainSurfaces.push_back(&st.referenceSurface());

// Reset navigation state
// We do not need to specify a target here since this will be handled
// separately in the KF actor
navigator.resetState(
state.navigation, state.geoContext, stepper.position(state.stepping),
state.options.direction * stepper.direction(state.stepping),
&st.referenceSurface(), targetSurface);
&st.referenceSurface(), nullptr);

// Update material effects for last measurement state in reversed
// direction
Expand Down

0 comments on commit a22a8be

Please sign in to comment.