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

Update build workflow to Python 3.12 #248

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

Update build workflow to Python 3.12 #248

wants to merge 3 commits into from

Conversation

dbarbuzzi
Copy link
Contributor

@dbarbuzzi dbarbuzzi commented Jan 30, 2025

This PR updates the build workflow to use Python 3.12 (and a matching updated nm-action) so that the tarball that gets produced will have a filename compliant with PEP-625.

TODO

@dbarbuzzi dbarbuzzi self-assigned this Jan 30, 2025
@dbarbuzzi
Copy link
Contributor Author

Note re: testing — A build run using the updated workflow was executed here:

https://github.com/neuralmagic/compressed-tensors/actions/runs/13062714332/job/36449151958

As can be seen in the output toward the end of the build step, both the wheel and tarball filenames are using underscores for the package name (compressed_tensors_nightly):

...
compressed_tensors_nightly-0.9.1.20250130-py3-none-any.whl
compressed_tensors_nightly-0.9.1.20250130.tar.gz
...

dhuangnm
dhuangnm previously approved these changes Jan 31, 2025
which installs setuptools
@dsikka dsikka self-requested a review January 31, 2025 15:24
@derekk-nm derekk-nm marked this pull request as ready for review January 31, 2025 15:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants