-
Notifications
You must be signed in to change notification settings - Fork 120
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
Parsing of favorite_counts ends up with NoneType is not callable exception. #365
Comments
Never mind... Seems like the issue was caused by fiddling around and overwriting the git verrsion with pip update. Git version has all the necessary fixes. |
Hello, I also encountered the issue "Tried to get image id failed." Sorry, I really don't understand these programming languages. |
The fix for me was to update the nhentai tool. Preferably you should uninstall any and all pip versions (if you have any) and install the one from git since pip installation of versions other than 0.4.18 doesn't create the executable that GUI needs (as reported in #364). As for v0.4.18 itself, it seems like structure of website slightly changed and that version wasn't robust enough to handle it, but I'd have to take a closer look. Though it would explain why it was working fine up to certain point. |
I think this should be fixed already? |
I'm abroad so that I cannot push a fix commit, if the problem is figured out could you submit a pull request for this bug? |
I did notice that it has been fixed in git already, but for some reason installation with pip grabs version that still has the issue. I've made sure just now that it still happens. |
Seem need a new release on pypi?
Best Regards,
Ricter Zheng
DennouNeko ***@***.***>于2025年1月6日 周一19:40写道:
… I did notice that it has been fixed in git already, but for some reason
installation with pip grabs version that still has the issue. I've made
sure just now that it still happens.
It seems that pip installs this commit
<bb41e50>
because it has the updated version string, but not the next one with fix
for #361 <#361> that was
committed the same day.
So there's nothing to fix on git side.
—
Reply to this email directly, view it on GitHub
<#365 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ABIJXR7QPDNN4WCKTYVSLG32JJTUTAVCNFSM6AAAAABUNN75JKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKNZSHEZTOMBXGY>
.
You are receiving this because you commented.Message ID:
***@***.***>
|
Pretty much. That should fix the issue with pip install. |
Publish a new version |
pip downloaded v0.5.17.2, I guess there was one more version published in the mean time :) |
It looks like version 0.4.18 of nhentai tool is stopping to work. At first it was just from time to time, but now I just kept getting "Tried yo get image id failed" every time.
I've tried updating to most recent 0.5.17. Pip installation didn't generate the executable in scripts, but at least manual instalation worked.
The tool was now giving me exception while parsing favorite_counts, regardless if I was downloading favorites or doujin directly from ID.
After a bit of investigation, it seems like there's just a small mistake in parser.py.
Line 152 should be changed from
to
And just in case, using the nHentai GUI v1.06 on Windows 10 with Python 3.12.
The text was updated successfully, but these errors were encountered: