You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Currently the propagation state which incorporated the navigator and stepper state cannot be initialized outside the propagation. This is not optimal for externalSurfaces in the Navigator or the list of surfaces in the DirectNavigator. Currently we hack our way around that by setting the state via code in the actor that only get executed once.
My proposed solution is to decouple the propagation state creation from the propagate call #2552
Apart from that it would be beneficial IMO to move some of the state properties or config properties into an option struct that is part of the PropagationOptions and can be set for each propagation individually.
The text was updated successfully, but these errors were encountered:
This issue/PR has been automatically marked as stale because it has not had recent activity. The stale label will be removed if any interaction occurs.
Currently the propagation state which incorporated the navigator and stepper state cannot be initialized outside the propagation. This is not optimal for
externalSurfaces
in theNavigator
or the list of surfaces in theDirectNavigator
. Currently we hack our way around that by setting the state via code in the actor that only get executed once.Listing some code consequences here:
KalmanFitter
external surface initialization https://github.com/acts-project/acts/blob/main/Core/include/Acts/TrackFitting/KalmanFitter.hpp#L377DirectNavigator
initialization https://github.com/acts-project/acts/blob/main/Core/include/Acts/Propagator/DirectNavigator.hpp#L74My proposed solution is to decouple the propagation state creation from the propagate call #2552
Apart from that it would be beneficial IMO to move some of the state properties or config properties into an option struct that is part of the
PropagationOptions
and can be set for each propagation individually.The text was updated successfully, but these errors were encountered: