I would like to suggest to implement an addEventDetector method to the PropagatorBuilder instance.
I’m aware that in cases where we build several propagator, they would use the same event detector. To fix this, we could use the “create” method, defined in AbstractDetector (and move it to the EventDetector interface later on), to create different event detector for each instance.
My specific use case concerns impulsive maneuvers that i would like to configure directly in my NumericalPropagatorBuilder instance.
This subject is a bit tricky but i would like to hear your thoughts about this ?
As we already have propagator builder and ODE integrator builder, it would seem more consistent to also have event detector builder rather than moving the create method up to the interface.
I know that some Orekit estimators, like BatchLSEstimator and KalmanEstimator, use PropagatorBuilder as input. I think it would be useful to be able to add an event detector to such PropagatorBuilder. Or is that already possible?
Unfortunatly that is not currently possible and, as @luc suggested, the most consistent way to add this feature would be to add an equivalent builder class to all EventDetector.