We have the need at Airbus to perform propagation through maneuvers with variable thrust. We think this could be a contribution to Orekit. The idea would be to reuse most of the code that have been developed in ConstantThrustManeuver (for instance in term of event detectors, acceleration computation, and initialization). Therefore, it would avoid duplicating most of the code in ConstantThrustManeuver.
Instead of the thrust, the trust direction, and the isp, the constructor would take a generic interface as an input. This interface would demand to implements 4 methods in order to retrieve the thrust and the flow rate from the state, and their equivalent for fields parameters
In that case, the ConstantThrustManeuver would become a specific case of this new class, with an implementation of the interface that provides constant thrust and constant flow rate during the maneuver.
Do you think it could be a good contribution?