-
Notifications
You must be signed in to change notification settings - Fork 43
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
cannot import name 'main' from 'fvid' #27
Comments
I think it's related to #21 (comment) comment and bellow. @Theelgirl halp |
@minhngoc25a @dobrosketchkun It's definitely windows, however I think they need to update fvid. They're using the PyPi version of fvid (0.0.2) where |
@minhngoc25a Try running |
I think it'll be better if we first address #26 and only then update the PyPi version |
Merged. Is that a feature addition or a patch? 1.0.1 or 1.1.0? Edit: Looks like a patch, no new user-facing features, updating version to 1.0.1 |
I think we can classify it as a patch since YouTube is part of the initial idea and functionality. I don't know what to say about the version number; it seems quite arbitrary to me; if there is any convention about it, please use it. |
The version number is described by SemVer 2. |
Then 1.0.1 it is. |
I uninstalled fvid and then used
|
@minhngoc25a |
Yeah try now. This will fix your original error, not the c++ required one. Edit: Just committed again to remove cython from requirements so you should be able to build without cython. |
I still use the same command.
I think I'll install Build Tools for now... |
Ok, can you try once more without Build Tools? I added an except to the part that builds that in order to try and prevent it. If you get an error on name=, try again. |
Looks like we're getting somewhere. Less error is shown. Collecting fvid
Cloning https://github.com/AlfredoSequeida/fvid.git to c:\users\tran bich dung\appdata\local\temp\pip-install-ov408u2i\fvid
ERROR: Command errored out with exit status 1:
command: 'C:\Users\Tran Bich Dung\AppData\Local\Programs\Python\Python38\python.exe' -c 'import sys, setuptools, tokenize; sys.argv[0] = '"'"'C:\\Users\\Tran Bich Dung\\AppData\\Local\\Temp\\pip-install-ov408u2i\\fvid\\setup.py'"'"'; __file__='"'"'C:\\Users\\Tran Bich Dung\\AppData\\Local\\Temp\\pip-install-ov408u2i\\fvid\\setup.py'"'"';f=getattr(tokenize, '"'"'open'"'"', open)(__file__);code=f.read().replace('"'"'\r\n'"'"', '"'"'\n'"'"');f.close();exec(compile(code, __file__, '"'"'exec'"'"'))' egg_info --egg-base 'C:\Users\Tran Bich Dung\AppData\Local\Temp\pip-pip-egg-info-42ki7n0s'
cwd: C:\Users\Tran Bich Dung\AppData\Local\Temp\pip-install-ov408u2i\fvid\
Complete output (6 lines):
Traceback (most recent call last):
File "<string>", line 1, in <module>
File "C:\Users\Tran Bich Dung\AppData\Local\Temp\pip-install-ov408u2i\fvid\setup.py", line 53
name="fvid",
^
SyntaxError: invalid syntax
----------------------------------------
ERROR: Command errored out with exit status 1: python setup.py egg_info Check the logs for full command output.``` |
Yeah, I typoed, try it again. The reason less error is shown is because it errors out earlier. Also, sorry for making you test this, my Windows laptop already has build tools installed from my previous python projects so I don't get the error. |
Errors never ends, I guess...
|
Ok, I guess you'll need to download it for now, I'll start looking for ways to fix that when I have more time later. |
I can't believe that C++ build tools are nearly 5GB... |
Maybe I'll switch to WSL |
Maybe. I just did a last-ditch blanket exception on the setup script, so if this doesn't work, it most likely won't work until we can get someone who's good and experienced with Cython and PyPi. |
And right after I install WSL, tried the command in Powershell, it worked. Thanks for your enthusiasm anyway. |
Ah. Sorry that I couldn't fix it in time :( |
I had ffmpeg in PATH and fvid still doesn't work somehow...
The text was updated successfully, but these errors were encountered: