-
Notifications
You must be signed in to change notification settings - Fork 80
Conversation
🚀 Deployed on https://deploy-preview-1133--etna-docs.netlify.app |
Codecov Report
📣 This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more @@ Coverage Diff @@
## master #1133 +/- ##
==========================================
+ Coverage 86.90% 86.99% +0.08%
==========================================
Files 177 177
Lines 9746 9818 +72
==========================================
+ Hits 8470 8541 +71
- Misses 1276 1277 +1
📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
Also update the changelog |
TBATSModel, | ||
), | ||
) | ||
def test_seasonal_components_not_fitted(small_periodic_ts, estimator): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
May you also test the combination of components arma + seasonal if it is a possible scenario
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Added
tests/test_models/test_tbats.py
Outdated
assert len(components) == horizon | ||
|
||
|
||
@pytest.mark.long_2 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
May you also mark them as long_1
as it takes 10 min for long_2
and 4 min for long_1
Before submitting (must do checklist)
Proposed Changes
Closing issues
closes #1125