-
Notifications
You must be signed in to change notification settings - Fork 14
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
Build torch wheels on CI #352
Conversation
cb5e1fd
to
882bb87
Compare
We do not link to libpython
This allows to build a single wheel able to support multiple torch versions, picking the correct shared library to load at runtime
Instead of pulling a separate metatensor version from github
882bb87
to
fbfd35b
Compare
path: wheels | ||
name: sdist | ||
|
||
- name: Re-upload a single wheels artifact |
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.
Why do we need this? Aren't the wheels already stored?
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.
They are stored, but since actions/upload-artifact@v4
, each upload is a separate file. This enables us to give a single link to download in the PR
Another part of #281
📚 Documentation preview 📚: https://featomic--352.org.readthedocs.build/en/352/
⚙️ Download Python wheels for this pull-request (you can install these with pip)