-
-
Notifications
You must be signed in to change notification settings - Fork 15
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
ImageFilter imports numpy, causes issues on Arch Linux #59
Comments
I realize now that I'm using Leia, which is a version back, and thus not using the latest of your helper. Tell me to pound sand in that scenario. For what it's worth, I'm holding back for other compatibility reasons. |
This bug is reproducible in Ubuntu 21.04, Kodi 19.3 Crash log (snipped as nothing useful in the log, and cause of error is identified):
Output from debug logging when forcing an error in numpy's
As things stand, it looks like that any libs that pull in numpy basically break Kodi :/ I am happy to investigate workarounds (I do a fair amount in CV) but don't want to invest time if this would be of no use to you. |
I've just seen your wiki note re: this being a known issue. I have solved my issue (for now) by upgrading my system pillow to v8.4.0, but I'm fairly sure this kind of thing is likely to be a recurring theme with any plugins that depend on something that imports numpy, even if that issue does turn out to be upstream from the plugin. |
Hello, see the following log:
This reply from a numpy maintainer could very well be important: numpy/numpy#18848
"I'm sorry you are getting bounced around from repo to repo, but there's no bug in numpy that we re-introduced recently. numpy has never supported being run in Python sub-interpreters. Probably what changed is something in one of the Kodi plugins, like TMDbHelper, causing an import of numpy (probably through Pillow). If Kodi has moved to using sub-interpreters, all of the plugins will have to studiously avoid importing numpy entirely."
For my own purposes, I worked around the issue by simply removing the refrerence to ImageFilter, lines 12 and 76. I do not use this enough to know whether or not that breaks anything else, so I don't want to submit a PR, but I will if you think it appropriate. Essentially, l've lost the gaussian blur here.
Thanks for the awesome addon(s) by the way, much appreciated.
Kindly,
Daniel
The text was updated successfully, but these errors were encountered: