Skip to content

Bump requirements#613

Merged
GaelVaroquaux merged 4 commits intoskrub-data:mainfrom
LilianBoulard:bump_versions
Jun 15, 2023
Merged

Bump requirements#613
GaelVaroquaux merged 4 commits intoskrub-data:mainfrom
LilianBoulard:bump_versions

Conversation

@LilianBoulard
Copy link
Member

Bump the minimal versions of all dependencies and removes the requirement on requests (we don't use directly).

For numpy, scipy and pandas, versions date back to mid/late 2022, and scikit-learn 1.2.1 was released in January.

@LilianBoulard LilianBoulard added the dependencies Pull requests that update a dependency file label Jun 15, 2023
@LilianBoulard LilianBoulard self-assigned this Jun 15, 2023
Comment on lines -125 to -129
# (we need to require the experimental feature for scikit-learn versions
# earlier than 1.0):
from sklearn.experimental import enable_hist_gradient_boosting

# We can now import the |HGBR| from ensemble
Copy link
Member

Choose a reason for hiding this comment

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

Finally 👍

@LilianBoulard
Copy link
Member Author

Should be good to review, thanks!

Copy link
Member

@GaelVaroquaux GaelVaroquaux left a comment

Choose a reason for hiding this comment

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

Nice!!!

It's so good to have all those cleanups. Thanks for going through the code and finding them.

LGTM

@GaelVaroquaux
Copy link
Member

LGTM. Merging

@GaelVaroquaux GaelVaroquaux merged commit 444f689 into skrub-data:main Jun 15, 2023
@LilianBoulard LilianBoulard deleted the bump_versions branch June 16, 2023 08:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants