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

Adding UV installation instructions to README #6349

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

dsantiago
Copy link

As UV gains traction as a package management tool, I'm adding instructions to READ.me for those who have the interest to install and better control the package usage.

@robinjhuang
Copy link
Collaborator

uv is great.

uv init might error out after this PR goes in since pyproject.toml already exists: #6386

Also, I believe simply adding the requirements is not enough if you want CUDA or RoCM torch? Have you tried setting any of this stuff up in pyproject.toml?

https://docs.astral.sh/uv/guides/integration/pytorch/

Would be great to have something like:

  • cuda index for windows or linux by default
  • cpu nightly index for macos

@dsantiago
Copy link
Author

Hi there, thanks to asking... I did not stress the command because I don't have an easy way to do it (multiple SOs).

About pyproject.toml file, I checked the current options but I don't think both can co-exist. There's also a config file called uv.toml but I understood it works on the system level... I need to think of an option, but of couse you guys can propose if it's of interest to have an easy fresh install option with UV....

@dsantiago
Copy link
Author

@robinjhuang I updated the READ.me as I tested with the other PR #6386 pyproject.toml file. It was just needed to don't call uv init as uv reutilizes the pyproject.toml file without conflicts. You can use uv add .... normaly as it will generate the dependency entry in the file so it is good to go.

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