Skip to content

Update installation.md #6342

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.

Sign up for GitHub

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

Closed
wants to merge 2 commits into from
Closed

Update installation.md #6342

Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
7754528
Update installation.md
Bhavay-2001 Dec 26, 2023
5bee7a0
Merge branch 'main' into patch-1
Bhavay-2001 Jan 3, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 1 addition & 0 deletions docs/source/en/installation.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -48,6 +48,7 @@ pip install diffusers["torch"] transformers
pip install diffusers["flax"] transformers
```
</jax>
Note - Pytorch doesn't support Python 3.12 on Windows. Please use Python 3.11 instead.
</frameworkcontent>
Comment on lines 50 to 52
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
</jax>
Note - Pytorch doesn't support Python 3.12 on Windows. Please use Python 3.11 instead.
</frameworkcontent>
</jax>
Note - Pytorch doesn't support Python 3.12 on Windows. Please use Python 3.11 instead.
</frameworkcontent>

Copy link
Contributor

Choose a reason for hiding this comment

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

Also can we move this statement into the PyTorch "frameworkcontent" section?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Hi @patrickvonplaten, I have made another PR #6438. Please refer to that. I will close this one. Thanks


## Install with conda
Expand Down