-
-
Notifications
You must be signed in to change notification settings - Fork 13.3k
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
Error during Installation at windows by building project for dev #1279
Comments
If it helps, you can just run the installer exe and then navigate to /Users/your user/faceswap and the code is all there after install. |
Short answer to your question is. no, I wouldn't except this as a PR I'm afraid. The reason being that a try/except just ignores the error, it does not fix the underlying issue. Which leads me on to, what is the underlying issue? I have never seen this before, so worry that it is an edge case local to your machine. The first thing I would be looking at is where does |
@torzdf I found that |
Note: For general usage questions and help, please use either our FaceSwap Forum
or FaceSwap Discord server. General usage questions are liable to be closed without
response.
Crash reports MUST be included when reporting bugs.
Describe the bug
Hi, I want to contribute to this repo as an assignment of my college class.
So I folked this repo and clone it on my local and i ran setup.py with python 3.9 interpreter installed at anaconda env folder.
and this error is occured...
So I go down to the error occuring code and see what's the problem by debugging. And I found that
the pkg variable should have package name and version but some crazy key(\x1b[0m) was input without any value. That's why the IndexError was occured so I covered that code with try-except that do nothing when except occured. Then, It' works perfectly without any error. So the question I want you to ask is "Is is okay to make an pr about the issues similar to this one(related to setup)? and will you merge it?"
To Reproduce
Steps to reproduce the behavior:
Expected behavior
Merge the pr
Screenshots
Desktop (please complete the following information):
Additional context
Add any other context about the problem here.
Crash Report
There is no crash report about this issue
The text was updated successfully, but these errors were encountered: