search

lowfuel / progrock-stable

Stable Diffusion with some Proggy Enhancements
Other
167 stars 19 forks source link

Fix conda environment not setting up correctly in Linux #28

Open WorldTeacher opened 2 years ago

WorldTeacher commented 2 years ago

I tried installing this fork on Linux (Arch if that matters), but encountered this error while installing:

Collecting package metadata (repodata.json): done
Solving environment: done
Preparing transaction: done
Verifying transaction: done
Executing transaction: / By downloading and using the CUDA Toolkit conda packages, you accept the terms and conditions of the CUDA End User License Agreement (EULA): https://docs.nvidia.com/cuda/eula/index.html

done
Installing pip dependencies: - Ran pip subprocess with arguments:
['/home/alexander/anaconda3/envs/prs/bin/python', '-m', 'pip', 'install', '-U', '-r', '/mnt/database/progrock-stable/condaenv.pfqp9jje.requirements.txt']
Pip subprocess output:
Obtaining taming-transformers from git+https://github.com/CompVis/taming-transformers.git@master#egg=taming-transformers (from -r /mnt/database/progrock-stable/condaenv.pfqp9jje.requirements.txt (line 20))
  Cloning https://github.com/CompVis/taming-transformers.git (to revision master) to ./src/taming-transformers

Pip subprocess error:
  WARNING: Did not find branch or tag 'master', assuming revision or ref.
ERROR: Command errored out with exit status 128: git rev-parse HEAD Check the logs for full command output.

failed

CondaEnvException: Pip failed

To fix this, I removed the -e git... part in the .yaml file and added it to the readme.md, where I added the instructions you provide in progrockdiffusion. This can probably be done in the .yaml file, but I have no knowledge in that area, so I opted for this way.

WorldTeacher commented 2 years ago

It'd probably be better to rename the "for Linux" part I added to the readme to "Windows/Linux" since it also affects the windows installation if the yaml gets changed.

lowfuel commented 2 years ago

Hi - going to see if I can replicate this, because a number of people have installed this on linux and macOS without issue, and I would prefer to keep the install handled by the yaml file if possible. So, I'm holding off on merging this until I can verify it as well as explore a solution that doesn't require additional install commands.

WorldTeacher commented 2 years ago

Thanks for letting me know ^^ Finding a way that doesn't require more install commands would be better. Let me know if I can do anything to help.

rockerBOO commented 2 years ago

I had installed this on Linux (Arch) without any addition steps or problems. Maybe there was some intermittent problems with GitHub.