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 pyproject.toml file for packaging #33

Merged
merged 3 commits into from
Sep 5, 2024
Merged

Update pyproject.toml file for packaging #33

merged 3 commits into from
Sep 5, 2024

Conversation

asim29
Copy link
Collaborator

@asim29 asim29 commented Aug 28, 2024

Uploading the package to TestPyPI for now to test if it works. However, I'm getting the following error which I'm not sure how to deal with:

$ python3 -m twine upload --repository testpypi dist/*

Uploading distributions to https://test.pypi.org/legacy/
Enter your API token: 
Uploading amulet-0.1.0-py3-none-any.whl
100% ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 75.2/75.2 kB • 00:00 • 44.9 MB/s
WARNING  Error during upload. Retry with the --verbose option for more details.                                       
ERROR    HTTPError: 400 Bad Request from https://test.pypi.org/legacy/                                                
         'torch (>=2.2.0+cu118,<3.0.0)' is invalid for 'requires-dist'. See                                           
         https://packaging.python.org/specifications/core-metadata for more information. 

Seems like it's an issue with how we've defined the torch versions in pyproject.toml

@asim29 asim29 requested a review from sebszyller August 28, 2024 22:23
@sebszyller
Copy link
Contributor

No clue, you'll have to debug

@asim29
Copy link
Collaborator Author

asim29 commented Sep 4, 2024

Things work now, I did have to make a change in the pytorch version. Based on the comment on this issue it should work

Copy link
Contributor

@sebszyller sebszyller left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@asim29 asim29 merged commit ac4cc7d into main Sep 5, 2024
1 check passed
@asim29 asim29 deleted the package-project branch September 5, 2024 17:39
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.

2 participants