-
Notifications
You must be signed in to change notification settings - Fork 37
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 use a string pattern on a bytes-like object #97
Comments
The Verbose Log doesn't show the error you got, for any of us to troubleshoot the code, we would need potentially your Console Output that hopefully would show which line of the python code it's failing on. Otherwise, @Nicnl latest build of the offline EXE can be found here with most of the fixes thus far (I think this was the latest one he posted): #86 (comment) |
The console output is just these four lines,
Thanks for the link to the EXE. The binary build does work on my end. |
Hmm, odd, sorry I can't help more, but at least the updated EXE works for you. |
Hey The packaged exe works because it's not running Python 3 but Python 2 When I started fixing the Crunchy issues, I myself tried to run it under Python 3, but I encountered the same problems I'm not sure why the project was written in Python 2 in the first place, but well |
looks like the different between those two version is big. |
There is an issue with an anime called saki the first season is 1920x1072 not 1920x1080 elif str(resolution).lower() in ['1072p', '1072', 'fhd']:
info_url = "http://www.crunchyroll.com/xml/?req=RpcApiVideoPlayer_GetStandardConfig&media_id=%s&video_format=106&video_quality=62¤t_page=%s" % (
video_id, url)
resolution_to_find = "1920x1072" but I saw your message:
I downloaded @Nicnl fork of this project. anyone have any idea about this issue? |
I tried to fix this by converting htmlsource from byte to string, but the broke the download process, so I will leave it for people who understand python. |
Check the latest binary now. |
So which version of Python should we be using, the ReadMe just says Python 3 |
It should work on both. When I had started this project, Python 2 was the choice. But, then I had made changes that would make this work on both, Python 3 and Python 2. I'll try to re-write things by next week and see if it's any better. |
Thanks @Xonshiz for the work on this |
Still getting error running on Windows PowerShell with Python 3.7.4 with latest code
|
Command Line Arguments You Gave: main.py -i "https://www.crunchyroll.com/isekai-quartet/episode-1-come-together-quartet-783515" -u "xxx" -p "xxx"
Verbose Log Link: https://pastebin.com/YeR1evk2
Long Explanation: I first encountered this issue using Python 3.7.3. Thinking it might be related to #27 so I installed Python 3.5.4 but it still shows the same error. The verbose error log is from 3.5.4. Windows binary hasn't been updated to address the "cannot extract language error" that was fixed in master.
The text was updated successfully, but these errors were encountered: