-
Notifications
You must be signed in to change notification settings - Fork 87
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] Update VARMAX
pandas 2.0
#1416
Conversation
updated pandas version to 2.0
Thank you for contributing to
|
@MatthewMiddlehurst, is this change a fix?, or should we deprecate certain functions to ensure compatibility? local Tests are currently being skipped due to the absence of soft dependencies. |
No, this is not the fix required. You will see issues emerge if you remove the dependency line completely. |
@MatthewMiddlehurst , Thanks! I will work accordingly. |
-modified the tests to check for pandas>2.0.0. -removed unused import
[Draft] to fix `KeyError` from `test_all_forecasters`
This reverts commit 28e5f32.
I think it's best to leave this for now, IMO. We have some significant changes to the module planned which may make this irrelevant either way. |
I see, I ll Close this for now! |
updated pandas version to 2.0
Reference Issues/PRs
#291