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

Fix conda environment not setting up correctly in Linux #28

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

WorldTeacher
Copy link

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.

remove -e git+https, as they fail on Arch Linux
update to make install on Arch Linux possible
@WorldTeacher
Copy link
Author

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
Copy link
Owner

lowfuel commented Sep 2, 2022

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
Copy link
Author

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
Copy link

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

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.

3 participants