ceholden / yatsm

Yet Another Time Series Model
https://yatsm.readthedocs.org/en/latest/
MIT License
63 stars 30 forks source link

YATSM timeseries estimator API generalization #67

Closed ceholden closed 8 years ago

ceholden commented 8 years ago

Rationale

Generalize YATSM timeseries estimator API to fit more workflows or approaches. Currently there is quite a lot of code in CLI scripts that are specific to models like CCDC or BFAST that wouldn't work for other algorithms. There's also some terminology and methods within the YATSM baseclass that aren't quite right for something like a VCT.

Basically, we need to make timeseries estimators more responsible for things like the setup, preprocessing, and postprocessing. The methods used also need to be more generic.

API

First stab at what YATSM baseclass should define: