Thanks to visit codestin.com
Credit goes to github.com

Skip to content

MAINT postpone erroring in 1.9 when dealing with integer in PDP #30432

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

Merged
merged 1 commit into from
Dec 11, 2024

Conversation

glemaitre
Copy link
Member

@glemaitre glemaitre commented Dec 8, 2024

Follow-up to #30409 to delay the moment we error with numerical integer dtype.
We postpone to 1.9 because the fix will not be included in the 1.6 release.

ping @jeremiedbb

Copy link

github-actions bot commented Dec 8, 2024

✔️ Linting Passed

All linting checks passed. Your pull request is in excellent shape! ☀️

Generated for commit: 75af6c9. Link to the linter CI: here

@lesteve
Copy link
Member

lesteve commented Dec 9, 2024

This will be an error with pandas 3 (it is already an error with pandas-dev). I did not find any info about the pandas 3.0 release timeline but maybe it would make sense to include it in 1.6.1?

Copy link
Member

@ogrisel ogrisel left a comment

Choose a reason for hiding this comment

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

I think it's fine to keep to 1.9, irrespective of the fact that we plan to release in 1.6.1 or 1.7.0. This warning does not cause much code complexity in our code base, so it's not costly to keep it for a long time.

@lesteve lesteve merged commit 3b75eb9 into scikit-learn:main Dec 11, 2024
34 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants