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

make it more clear that ltt helps install *all* PyTorch distributions #103

Merged
merged 1 commit into from
Oct 24, 2022

Conversation

pmeier
Copy link
Owner

@pmeier pmeier commented Oct 23, 2022

Closes #95.

#95 (comment) indicated that our documentation is not sufficient. We need to make it more clear that ltt not only helps installing torch, but rather all PyTorch distributions.

To do that I did two things:

  1. Instead of just talking about "PyTorch distributions", the README now uses "PyTorch distributions like torch, torchvision, torchaudio, and so on" in the first two sections.
  2. The example installation commands now also include torchvision and torchaudio instead of just torch.

@arnicas If you re-read the updated README now, is it clear enough that ltt not only helps with torch?

@pmeier
Copy link
Owner Author

pmeier commented Oct 23, 2022

Rendered version

@arnicas
Copy link

arnicas commented Oct 24, 2022 via email

@pmeier pmeier merged commit 8e37c9b into main Oct 24, 2022
@pmeier pmeier deleted the improve-readme branch October 24, 2022 08:36
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.

Thank god for this, you saved me
2 participants