Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[ENH] Added method plot_components() to the STLForecaster forecaster #6423

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

ericjb
Copy link
Contributor

@ericjb ericjb commented May 15, 2024

STL decomposition is an important and widely used technique in time series analysis. It is supported in sktime via the STLForecaster class. While it can be used as a step in a chain, it is also helpful - and common - to plot the decomposed series - observed, trend, seasonal, residuals. The method plot_components() has been added to the STLforecaster class to support this.

For an example of how to use this see section 2 in the attached pdf.

PullRequestNotes.pdf

Copy link
Collaborator

@fkiraly fkiraly left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice!

At the start, we should add checks and informative messages for the user, if:

  • the estimator is not fitted
  • matplotlib is not present, it is a soft dependency

This can be addressed by:

  • calling check_is_fitted
  • calling _check_soft_dependencies

(you can search for examples in the code base - kindly let me know if you need pointers)

@fkiraly fkiraly added module:forecasting forecasting module: forecasting, incl probabilistic and hierarchical forecasting enhancement Adding new functionality labels May 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Adding new functionality module:forecasting forecasting module: forecasting, incl probabilistic and hierarchical forecasting
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants