Skip to content

Implement predict method in SARIMAXModel, AutoARIMAModel, SeasonalMovingAverageModel, DeadlineMovingAverageModel #948

Merged
merged 11 commits into from
Sep 21, 2022

Conversation

Mr-Geekman
Copy link
Contributor

@Mr-Geekman Mr-Geekman commented Sep 19, 2022

Before submitting (must do checklist)

  • Did you read the contribution guide?
  • Did you update the docs? We use Numpy format for all the methods and classes.
  • Did you write any new necessary tests?
  • Did you update the CHANGELOG?

Proposed Changes

Look #928.

Closing issues

Closes #928.

@Mr-Geekman Mr-Geekman self-assigned this Sep 19, 2022
@github-actions
Copy link

github-actions bot commented Sep 19, 2022

🚀 Deployed on https://deploy-preview-948--etna-docs.netlify.app

@github-actions github-actions bot temporarily deployed to pull request September 19, 2022 13:15 Inactive
@github-actions github-actions bot temporarily deployed to pull request September 19, 2022 14:13 Inactive
@github-actions github-actions bot temporarily deployed to pull request September 19, 2022 14:19 Inactive
@codecov-commenter
Copy link

codecov-commenter commented Sep 19, 2022

Codecov Report

❗ No coverage uploaded for pull request base (inference@583c0ca). Click here to learn what that means.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             inference     #948   +/-   ##
============================================
  Coverage             ?   49.70%           
============================================
  Files                ?      132           
  Lines                ?     7586           
  Branches             ?        0           
============================================
  Hits                 ?     3771           
  Misses               ?     3815           
  Partials             ?        0           

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

Copy link
Collaborator

@alex-hse-repository alex-hse-repository left a comment

Choose a reason for hiding this comment

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

We definitely need to come up with the standard templates for tests, now it is really hard to review the tests

etna/models/seasonal_ma.py Outdated Show resolved Hide resolved
etna/models/seasonal_ma.py Show resolved Hide resolved
etna/models/seasonal_ma.py Show resolved Hide resolved
etna/models/sarimax.py Show resolved Hide resolved
etna/models/deadline_ma.py Outdated Show resolved Hide resolved
etna/models/deadline_ma.py Show resolved Hide resolved
tests/test_models/test_autoarima_model.py Show resolved Hide resolved
tests/test_models/test_autoarima_model.py Outdated Show resolved Hide resolved
tests/test_models/test_simple_models.py Show resolved Hide resolved
tests/test_models/test_sarimax_model.py Outdated Show resolved Hide resolved
@github-actions github-actions bot temporarily deployed to pull request September 20, 2022 13:51 Inactive
def _make_prediction(
self, df: pd.DataFrame, prediction_interval: bool, quantiles: Sequence[float], dynamic: bool
) -> pd.DataFrame:
"""Make predictions depending on parameters."""
Copy link
Collaborator

Choose a reason for hiding this comment

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

Emphasize that the main parameter which change the behaviour is dynamic

sma_model = model()
def test_sma_model_forecast(simple_df, model):
_check_forecast(ts=simple_df, model=model(), horizon=7)
# sma_model = model()
Copy link
Collaborator

Choose a reason for hiding this comment

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

)))))

@github-actions github-actions bot temporarily deployed to pull request September 21, 2022 14:06 Inactive
@alex-hse-repository alex-hse-repository merged commit d5eb926 into inference Sep 21, 2022
@Mr-Geekman Mr-Geekman mentioned this pull request Oct 26, 2022
4 tasks
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants