Skip to content

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

Merged
merged 13 commits into from
Mar 4, 2025
Merged

Fix support for numpy-2 #9109

merged 13 commits into from
Mar 4, 2025

Conversation

jleibs
Copy link
Member

@jleibs jleibs commented Feb 21, 2025

What

In 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

Copy link

github-actions bot commented Feb 21, 2025

Latest documentation preview deployed successfully.

Result Commit Link
070e0f4 https://landing-ok205mpfh-rerun.vercel.app/docs

Note: This comment is updated whenever you push a commit.

Copy link

github-actions bot commented Feb 21, 2025

Web viewer built successfully. If applicable, you should also test it:

  • I have tested the web viewer
Result Commit Link Manifest
070e0f4 https://rerun.io/viewer/pr/9109 +nightly +main

Note: This comment is updated whenever you push a commit.

@jleibs jleibs changed the title Update to numpy-2 and torch 2.3.1 Fix support for numpy-2 Feb 21, 2025
@Wumpf Wumpf linked an issue Feb 22, 2025 that may be closed by this pull request
@emilk
Copy link
Member

emilk commented Feb 23, 2025

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.

@abey79 abey79 self-requested a review March 4, 2025 09:10
Copy link
Member

@abey79 abey79 left a 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
Copy link
Member

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
Comment on lines 600 to 603
platforms = ["linux-64", "linux-aarch64", "osx-arm64", "osx-64", "win-64"]
platforms = ["linux-64", "osx-arm64", "win-64"]
Copy link
Member

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
Copy link
Member

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 :)

@jleibs
Copy link
Member Author

jleibs commented Mar 4, 2025

Still, I feel pretty OK with dropping support for it.

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

@jleibs jleibs merged commit 0e8302b into main Mar 4, 2025
41 checks passed
@jleibs jleibs deleted the jleibs/numpy-2-b branch March 4, 2025 15:31
@emilk emilk mentioned this pull request Apr 11, 2025
16 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants