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

[examples] Solving the diffusion issue of incompatibility issue#3169 #3170

Merged
merged 5 commits into from
Mar 20, 2023
Merged

[examples] Solving the diffusion issue of incompatibility issue#3169 #3170

merged 5 commits into from
Mar 20, 2023

Conversation

NatalieC323
Copy link
Contributor

Updated GitHub documentation:

  1. requirements.txt content is incompatible with the version and cannot be downloaded. It is recommended to change the content opencv-python==4.6.0 to opencv-python==4.6.0.66 (otherwise an error will be reported saying that the corresponding version cannot be found). Similarly, torchmetrics==0.6 should be changed into torchmetrics==0.7 (otherwise it will raise an error of incompatibility)

  2. The instructions 'pip install colossalai' and 'pip install xformer' given in Readme will forward to the latest version. To download the suitable version for this training, pip install colossalai==0.2.5, pip install xformer==0.12.0 is recommended instead

  3. Cuda version installation guide is added to Readme. The recommended version is cuda11.6 to 11.8.

@NatalieC323 NatalieC323 changed the title [ColossalAI/examples/images/diffusion]: Solving the issue of incompatibility [ColossalAI/examples/images/diffusion]: Solving the issue of incompatibility issue#3169 Mar 17, 2023
@binmakeswell binmakeswell changed the title [ColossalAI/examples/images/diffusion]: Solving the issue of incompatibility issue#3169 [examples] Solving the diffusion issue of incompatibility issue#3169 Mar 20, 2023
@binmakeswell
Copy link
Member

Please resolve the conflicting files, thanks.
examples/images/diffusion/README.md

@Fazziekey
Copy link
Contributor

the PR looks good, but it still have some conflict, please run git pull origin main --rebase

Copy link
Contributor Author

@NatalieC323 NatalieC323 left a comment

Choose a reason for hiding this comment

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

I have deleted all ":"s in the Readme file and the python version has been changed back to 3.9.12

@Fazziekey Fazziekey enabled auto-merge (squash) March 20, 2023 06:03
@Fazziekey Fazziekey merged commit 4e921cf into hpcaitech:main Mar 20, 2023
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.

None yet

3 participants