progrock-stable icon indicating copy to clipboard operation
progrock-stable copied to clipboard

Fix conda environment not setting up correctly in Linux

Open WorldTeacher opened this issue 2 years ago • 4 comments

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 avatar Sep 01 '22 12:09 WorldTeacher

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.

WorldTeacher avatar Sep 01 '22 13:09 WorldTeacher

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.

lowfuel avatar Sep 02 '22 11:09 lowfuel

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.

WorldTeacher avatar Sep 03 '22 21:09 WorldTeacher

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

rockerBOO avatar Sep 21 '22 00:09 rockerBOO