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
Cat5 Plugin Issue with Kodi Kryton #14
Comments
@solbu have you encountered this? |
@Cat5TV: No. @GitMarshallBill: Which version of the plugin do you run? |
Thanks @solbu ... I wonder then if this is a possible compatibility issue with the newer Kodi 17.6. Will need to recreate the error to see what's going on. |
I am using 1.1.3
|
Okay everything was great and working on my Kodi installed from Ubuntu repos... so I removed that and installed 17.6 and sure enough I see the same issue. Debugger log doesn't give any helpful details (ie. an error). |
|
@nateuk any chance you're interested in working with us on this? |
I've been testing a few things on our end, like outputting just one show instead of all in case it has something to do with something silly like a French character, but no luck. Still not able to get the new Kodi to load the channel. The error log/debug log is of absolutely no help. Anyone have experience with this stuff and can review the code to see if perhaps something in it is deprecated in 17.6? |
I have opened a forum thread here: https://forum.kodi.tv/showthread.php?tid=326782 |
My log file from a Debian Stretch VM is attached. |
Talked to one of our python devs. Your add-on uses the content type |
wow - @da-anda seriously? I will try this right away - thank you! |
thanks @solbu - I will wait rather than committing my own. Cheers! |
@solbu - let me know when updates have happened and I can test on my devices |
@GitMarshallBill I'll commit @solbu 's PR as soon as it comes in, so you can re-download at that point. I imagine he'll want to increment the version number as well. :D |
Merged. @GitMarshallBill please test and confirm. |
btw, if you would like to slightly improve the speed of the add-on on low power devices like the PI, then move as much logic as you can into a separate file and keep the main file as small as possible. Reason is, Kodi will cache the parsed version of imported files (pyo) but not the main file. So every time the add-on is triggered, the entire file needs to be parsed/compiled by the Python interpreter again and again and again, while imported files are only processed once and can be called instantly. |
Oh that is fantastic news. Thank you @solbu @da-anda and @GitMarshallBill ... time to redo our on-air demo of the Kodi channel :) |
@da-anda - just caught your final comment as I closed this. I'm entirely green at this point regarding Kodi add-on development. If you are able & willing, I'd greatly love more information about how to achieve this. Even a quick demo using our channel, or ultimately a PR would be something else. As it is, I wouldn't know where to look or how to implement what you're recommending :) I'm a PHP/bash/javascript dev so it doesn't take me long to understand the code... just not at the point where I can "create" add-on code yet... have to work from examples. Thanks! |
well, I have the same background as you. I only know about python what I read on the forums or catch up on our Slack channel, and that's something I catched up. But this change likely is bringing almost 0 improvement. However you might be able to really improve the speed by adding some caching and maybe improve the XML parsing of the RSS feed (or maybe even switch over to JSON). Again, I'm not a Python dev and can only give some suggestions. Our forum might be the best way to get qualified help. On a sidenote: It would be nice if you would submit your add-on to our official repository, so that everybody can enjoy your shows and not just people already knowing you. Might get you some more patreons ;) |
@da-anda: How do we do that? btw, I already run a public cat5 repo for my own use, and also to learn how it's done. |
Ha - thanks @da-anda - as you were typing your message I was typing the Issue to add Cat5 to the add-on repository :D Great minds, right? Thank you. I didn't want to submit it until I had the confirmation that this issue was fixed. Yeah, I think caching and/or doing as you suggested earlier may have more of an impact than you think. Look at shows.php and look at the size of the RSS files ... especially Category5 Technology TV. It's loading SD, HD, LD, MP3, etc.... for each show... and we've been on the air for 11 years... so yes, I think you are onto something here that could greatly increase the performance - thank you. Now, to see if @solbu can make sense of your comments and I'll just push "merge" LOL! Thanks again! |
Woh - @solbu what does the repository do? Keep it up to date?! Maybe THAT should be submitted rather than our channel itself? Should I fork it? :) |
@Cat5TV: The idea was that you could use it to run an official cat5 repo. I mailed you info on it before christmas, but perhaps it drowned in other emails. :-) |
Oh yes - ha! And yes, it worked its way down and I totally forgot (SORRY!). Yeah, I'd love to see it in the official Repo... feels to me like that's the best way to give our free content to the Kodi community. But I'm still really interested in your repo idea. But as noted above, I have yet to get my head around the Kodi add-on development and how it all works. Thanks for doing what you do... I couldn't do this without you. |
@Cat5TV: But yes, the repo update the video addon when a new release is made. That way users who install the repo addon will automatically get new releases, when the repo owner run two make commands. |
check out http://kodi.wiki/view/HOW-TO:Create_add-on_PRs_using_Git_Subtree_Merging on how to PR a add-on (if I'm not mistaken). edit: the correct branch would be the min Kodi version this add-on compatible with. So if it's compatible with Helix+, PR to the Helix branch and it will become available for all Kodi versions since Helix. (just as an example) |
Well, it should work with ... everything since at least 2015 when it was first developed, and probably before that. Not sure how to tell the actual ver. |
Our python devs just confirmed that JSON parsing should be quite fast in Python, so you might want to switch to JSON if that's easily possible and the service is not used for anything else. Actually, a real webservice that's serving the data in chunks might even be better, but that's up to you and your amount of sparetime ;) |
@Cat5TV: According to http://kodi.wiki/view/Addon.xml#Dependency_versions |
just tested it on Jarvis and get some urllib errors. So probably better test it upfront |
Bummer. I had it working fine on Kodi 15, so figured it'd work fine with 16 too. Should I delete my Issue and wait till we've tested more? Or what should I do? What are the urllib errors you're getting? Is it possible one of our site assets is down and that's all you're seeing? |
I still have kodi 14.0 installed in a system, and it works on that. |
seems to be an issue with the SSL handshake. When I switch the URL to http it's loading the categories. |
That looks like an SSL/TLS problem. it fails due to a TLS/SSL problem. |
Try changing the shows.php file to http instead of https and tell me if that fixes it. (line 22 of default.py) |
yes, that fixes it, but since the URLs returned by shows.php (which has some odd format, being HTML mixed with XML in the body) are also https, it stops working after the categories. It's likely a bug in urllib on my system (i.e. the version shipped with the Windows version) |
okay point line 22 to http and shows2.php - I will make that http only and you can let me know. |
it might be fixable by providing some SSL context to urllib2, but I'm also a python noob |
There's no confidential info exchanged. Removing ssl is a fine solution too (there's no reason for it other than just 'best practice') so really, if removing https fixes it, I'm fine with that. |
No need to change to shows2.php now. I made shows.php auto-detect SSL and act accordingly. Does it work now when you change https:// to http:// on line 22 of default.py? |
yes, it's working. Talked to some of our devs and there are known issues with SSL on Jarvis, so best really might be to switch over to http |
Perfect. I'll update the add-on to use http. Thanks! |
I'm calling it a night now, I'm a couple hours ahead of Canada ;) Keep up the good work on your channel, checked out a couple episodes. Nice stuff 👍 |
Nice to know we can still support old kodi versions. :-) |
Thank you @da-anda - your help today has been invaluable. Same goes to @solbu for the fix, and of course @GitMarshallBill for letting us know of the issue - thanks team! |
just noticed that I've been watching you @Cat5TV all the time in your tech talk episodes ;) |
@da-anda been watching Category5 Technology TV you mean? That's our flagship, and then we've got the smaller shows as well. Hope you've enjoyed it! Glad to finally be seeing it make its way to Kodi (hopefully through the add-on repo). :) |
Using Kodi Krypton v17.6 on Windows 10, Debian 9 or Ubuntu 16.04 plugin will load and show in MyAdd-ons but when you select it display remains empty and will not load videos.
The text was updated successfully, but these errors were encountered: