-
-
Notifications
You must be signed in to change notification settings - Fork 25.8k
FIX add long long for int32/int64 windows compat in NumPy 2.0 #29029
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
Conversation
@jeremiedbb since you have a windows machine where you can build scikit-learn, could you give a try and run the test |
So from the discussion on the feedstock, it seems that this is a rather urgent fix to unblock the migration to NumPy 2.0 on the conda-forge side. I think that it makes sense to make a proper bug fix release (1.4.3) and I'll update the changelog accordingly. |
I find that it's a huge waste of time to make a 1.4.3 today or even maybe tomorrow while we'll release 1.5.0 on monday |
I tested locally on my windows machine and confirm that it fails without the fix and works with this fix |
Please add |
I did not exactly the timeline for the major. But if we release on Monday, I agree. |
We can also argue that we should avoid releasing on Friday. |
That's why I said monday 😄. I aimed to create the release PR tomorrow and try to release on monday |
@jeremiedbb I think this is fine to skip adding an entry in the changelog? |
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
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
closes #29028
Since NumPy 2.0 will switch from
long
tolong long
for Windows machine, then it seems that we have trigger a non-matching type as shown here: conda-forge/scikit-learn-feedstock#259 (comment)This should fix it.