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.

Sign up for GitHub

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

Jump to bottom

always use SARIMAX instead of ARIMA, it is deprecated #211

Closed
ihadanny opened this issue Nov 7, 2019 · 2 comments · Fixed by #219
Closed

always use SARIMAX instead of ARIMA, it is deprecated #211

ihadanny opened this issue Nov 7, 2019 · 2 comments · Fixed by #219
Labels
feature request A tag for feature requests

Comments

@ihadanny
Copy link

ihadanny commented Nov 7, 2019

see: statsmodels/statsmodels#6225 (comment)
@ihadanny ihadanny added the feature request A tag for feature requests label Nov 7, 2019
@tgsmith61591
Copy link
Member

Cool! This would actually make our lives significantly easier than having to straddle three different classes under the hood. Thanks for pointing this out to us!

tgsmith61591 added a commit that referenced this issue Nov 14, 2019
@tgsmith61591
This should wrap up #211
7e539b5
@tgsmith61591 tgsmith61591 mentioned this issue Nov 14, 2019
Merged
10 tasks
tgsmith61591 added a commit that referenced this issue Nov 15, 2019
@tgsmith61591
This should wrap up #211
f79b719
tgsmith61591 added a commit that referenced this issue Nov 18, 2019
@tgsmith61591
This should wrap up #211
011fe47
@tgsmith61591
Copy link
Member

Merged in #219

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
feature request A tag for feature requests
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[MRG+1] SARIMAX only alkaline-ml/pmdarima
2 participants
@ihadanny @tgsmith61591