nachonavarro/seasonal-esd-anomaly-detection

Inaccurate description of decomposition

Closed this issue · 3 comments

zdgriffith commented

In the README:

To circumvent this, STL decomposition is used.

However, the seasonal_decompose method from statsmodels is used currently which doesn't include LOESS. There is an STL implementation that was merged a few weeks ago that could be added as an option: statsmodels/statsmodels#5926

nachonavarro commented

Hey @zdgriffith nice finding, would you mind fixing it and submitting a pull request? I can't work on it at the moment!

SwathyKrishnan29 commented

In the README:

To circumvent this, STL decomposition is used.

However, the seasonal_decompose method from statsmodels is used currently which doesn't include LOESS. There is an STL implementation that was merged a few weeks ago that could be added as an option: statsmodels/statsmodels#5926

Hi I am unable to use the STL function of statsmodels even after updating the package what to do?

nachonavarro commented

I've changed dependencies and now I'm using this which does include LOESS.