Native AdaptableInterval

Hello,

I would like to introduce native AdaptableInterval implementations.
At the moment in version 12, there are all the elements for a non trivial one built in NodeDetector, but it’s not directly accessible.
Similarly, in ApsideDetectorTest, there is one implemented. That would actually avoid having to build the detector with an orbit as input.
What do you say? Can you think of any other detector that would benefit from a public, native AdpatableInterval?

Cheers,
Romain.

This could be interesting, but what would be the decision criterion for selecting the max check ? The implementation in node detector monitors the nodes whereas the implementation in apside detector monitors the anomaly. Do you intend to provide a few different implementations according to these use cases?

Hi Luc,

Yes and the idea would to be very clear in the naming and documentation about which event the particular AdaptableInterval are meant for.

Cheers,
Romain

1 Like