-
-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
[BUG] remove alpha
arg from _boxcox
, remove private method dependencies, ensure scipy 1.8.0 compatibility
#2468
Conversation
alpha
arg from _boxcox
, remove _boxcox_conf_interval dependency
, scipy 1.8.0 compatibilityalpha
arg from _boxcox
, remove private method dependencies, ensure scipy 1.8.0 compatibility
This PR fixes #1994, what I think is the last issue that prevents sktime from being compatible with The fix consists of the following:
|
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.
LGTM
tests don't run yet - why is everything stuck, @lmmentel? |
There was an incident with GHA recently which might've created a queue? 🤷 |
* upstream/main: [ENH] more forecaster scenarios for testing: using `X` (sktime#2462) less uncertainty samples (sktime#2497) [ENH] remove error message on exogeneous X from DirRec reducer (sktime#2463) [BUG] fix accidental overwrite of default method/arg sequences in test scenarios (sktime#2457) changed references to fit-in-transform to fit_is_empty (sktime#2494) [MNT] loosen strict upper bound on `scipy` to 1.9.0 (sktime#2474) [DOC] Added clustering module to API docs (sktime#2429) [ENH] Add prediction intervals for `UnobservedComponets` forecaster (sktime#2454) [BUG] remove `alpha` arg from `_boxcox`, remove private method dependencies, ensure scipy 1.8.0 compatibility (sktime#2468) [BUG] removed metric integration notebook (sktime#2476) [ENH] refactor `_predict_moving_cutoff` and bugfix, outer `update_predict_single` should be called (sktime#2466) [ENH, BUG] Distance refactor and bug fix (sktime#2268) [ENH] add new argument return_tags to all_estimators (sktime#2410)
This PR fixes #1994, what I think is the last issue that prevents sktime from being compatible with
scipy
1.8.0.The fix is to remove the otherwise unused
alpha
arg from_boxcox
, which allows to remove the_boxcox_conf_interval dependency
entirely