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

[timeseries] update sktime dependency range #2877

Merged
merged 1 commit into from
Feb 9, 2023

Conversation

canerturkmen
Copy link
Contributor

Issue #, if available:

#2841

Description of changes:

As of v0.7, we are deprecating sktime in AG-TS, due to instability and efficiency issues. Nevertheless, we update the version ranges for conda compatibility. There are no regressions or additional faults in regression tests with v0.14 or v0.15, we cap at 0.16 as we will deprecate in the next release.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@canerturkmen canerturkmen added the module: timeseries related to the timeseries module label Feb 9, 2023
@github-actions
Copy link

github-actions bot commented Feb 9, 2023

Job PR-2877-63fb1c5 is done.
Docs are uploaded to http://autogluon-staging.s3-website-us-west-2.amazonaws.com/PR-2877/63fb1c5/index.html

Copy link
Contributor

@Innixma Innixma left a comment

Choose a reason for hiding this comment

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

LGTM! I resolved the merge conflict by rebasing with mainline. Thanks for the contribution!

@Innixma Innixma merged commit 825b6fd into autogluon:master Feb 9, 2023
@github-actions
Copy link

github-actions bot commented Feb 9, 2023

Job PR-2877-3dae8d2 is done.
Docs are uploaded to http://autogluon-staging.s3-website-us-west-2.amazonaws.com/PR-2877/3dae8d2/index.html

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
module: timeseries related to the timeseries module
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants