-
Notifications
You must be signed in to change notification settings - Fork 482
Fix support for numpy-2 #9109
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
Fix support for numpy-2 #9109
Conversation
Latest documentation preview deployed successfully.
Note: This comment is updated whenever you push a commit. |
Web viewer built successfully. If applicable, you should also test it:
Note: This comment is updated whenever you push a commit. |
The last Intel MacBook was released 2021, which is not that long ago (feels like it's been a decade). Still, I feel pretty OK with dropping support for it. I really doubt there are a lot of intel Mac developers out there. |
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.
As noted, there are things that I dont like much here, but then again I like the previous situation even less 😅 Regardless, thanks for the grunt work!
PLATFORM: macos-x64 | ||
WHEEL_ARTIFACT_NAME: macos-x64-wheel | ||
secrets: inherit | ||
# TODO(#9108): Test macos wheels |
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.
I'm afraid that will backfire, but I don't see a way around it, and this PR answers an actual, pressing need. So yeah, let's take the plunge.
pixi.toml
Outdated
platforms = ["linux-64", "linux-aarch64", "osx-arm64", "osx-64", "win-64"] | ||
platforms = ["linux-64", "osx-arm64", "win-64"] |
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.
While osx-64
is in definitely in the past, im afraid linux-aarch64
is in the future. We'll have to deal with that sooner than later.
pixi.toml
Outdated
@@ -658,22 +665,6 @@ stdio = { path = "examples/python/stdio", editable = true } | |||
structure_from_motion = { path = "examples/python/structure_from_motion", editable = true } | |||
|
|||
# TODO(jleibs): Do this with less duplication |
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.
looks like you just did :)
To be clear: we are not dropping support for intel macs. We still build and publish wheels for them. We just can't run our unit-tests until we fix: #9108 |
What
test_any_values.py
#9105AnyValue
is having issues with numpy 2 (?) #8486In order to move our pixi project to numpy to, we also had to upgrade pytorch to 2.3+ (we might consider jumping all the way to 2.6, but I was trying to find the minimum bump to risk breaking other things).
Unfortunately torch-2.2 is the last version that supported intel macs.
I disabled the wheel-testing on nightly with an issue to re-enable: #9108