Skip to content
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

Release request: Could we get an updated release that builds on MacOS? #3846

Closed
sarahec opened this issue Sep 9, 2024 · 12 comments
Closed
Labels

Comments

@sarahec
Copy link

sarahec commented Sep 9, 2024

Summary

I'm a maintainer over at NixOS and we have many packages that depend on faiss (directly or indirectly) , but are all broken on Darwin. I've tried to cherry-pick patches to no avail. Would you be able to release a point release that works on Darwin? That would enable support for hundreds of packages.

@mengdilin
Copy link
Contributor

@sarahec which version of faiss are you on and is this for OSX x86_64 for OSX arm64? We release to conda forge every night for OSX arm64 (e.g. https://anaconda.org/pytorch/faiss-cpu)

@sarahec
Copy link
Author

sarahec commented Sep 11, 2024

We are on release 1.8.0 (see nixpkgs search) and build for linux-{x86,arm64} and OSX (Darwin)-{x86,aarch64} whenever possible.

We're currently building from source and that's our preferred way. Also, we prefer published source releases, which is why we're back on 1.8.0.

I understand the testing and other work that goes into making a release, so my request may be a long shot. I had to try.

P.S. The main blocker seems to be the issue compiling swig-generated code due to the lack of SWIGTYPE_p_long and SWIGTYPE_p_unsigned_long while building the Python wrapper on MacOS. If there's a way to patch the code so it works again, we could patch out the bug while waiting for next versions of swig and/or faiss. (This also may be related to using swig 4.2.1)

@sarahec sarahec changed the title Release request: Could we get an updated release that works on MacOS? Release request: Could we get an updated release that builds on MacOS? Sep 11, 2024
@mengdilin
Copy link
Contributor

mengdilin commented Sep 12, 2024

Is the x86 version failing or faiss compilation is failing for both?

We have working nightly releases for OSX arm64 that runs on macOS 14 off of the main branch and part of that release compiles faiss from source and runs all of the relevent python tests along with it. Can you try compiling with the latest code on master and see if it works? If so, I can talk to the team about cutting a release.

@sarahec
Copy link
Author

sarahec commented Sep 12, 2024

It's fine on x86, and breaking on macos. I'll try with the latest code on master and report back (thank you!).

@sarahec
Copy link
Author

sarahec commented Sep 12, 2024

Yes, building from master works on both OSX and Linux.

@mengdilin
Copy link
Contributor

We should be doing a release next week :) Will keep you updated

@sarahec
Copy link
Author

sarahec commented Sep 17, 2024

You are amazing. Thank you.

Copy link

This issue is stale because it has been open for 7 days with no activity.

@github-actions github-actions bot added the stale label Sep 25, 2024
@sarahec
Copy link
Author

sarahec commented Sep 25, 2024

The work is still in progress, my silicon friend. Awaiting the next release.

@sarahec
Copy link
Author

sarahec commented Oct 3, 2024

How is the release coming along, if I might ask? (We're starting the release cycle for 24.11 and it would be lovely if we could include this.)

@asadoughi
Copy link
Contributor

I am working on the release this week in #3887

@sarahec
Copy link
Author

sarahec commented Oct 5, 2024

Thank you!!

@sarahec sarahec closed this as completed Oct 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants