ekaterinailin / AltaiPony

Find flares in Kepler and TESS light curves. Notebooks for quickstart inside.
https://altaipony.readthedocs.io
MIT License
26 stars 10 forks source link

Add tests that also perform de-trending steps. #26

Closed ekaterinailin closed 4 years ago

ekaterinailin commented 6 years ago

What needs to be created or improved?

Testing the de-trending part is computationally intensive. We still need to test these parts of the code.

Goal:

Write tests that assert that de-trending is triggered when necessary, and leads to similar results each time.

What is the goal / expected behaviour?

I shall return helpful error messages when the attributes of a given FlareLightCurve do not have the de-trended part but need them, or catch these exceptions on the go if this is what we would expect as users.

ekaterinailin commented 6 years ago

You can close #16 when you close this one.

ekaterinailin commented 4 years ago

Now covered by sample_flare_recovery and its tests.