Skip to content
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

Fix radio.de API limitation of 100 rows #27

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

ttobsen
Copy link
Collaborator

@ttobsen ttobsen commented Nov 27, 2018

The radio.de API just returns 100 stations per request. This is not acceptable, especially for the country and language categories where thousands of stations are available.

With this MR, the start parameter is used to loop over all stations of a category. The performance is pretty poor, but at least every station of a category is available.

@jcjdammers
Copy link

@ttobsen , @dersphere ,

When is this fix going to be released?
I am really waiting for it. It is annoying that the menu items only show the first 100 stations and there is no possibility to browse further.

@ttobsen
Copy link
Collaborator Author

ttobsen commented Dec 18, 2019

The problem is that @dersphere does not seem to be active anymore. I also tried to get in contact by email, but no chance. I think the only chance is, to fork the project and release it under another name.

@dersphere
Copy link
Owner

@ttobsen and @jcjdammers: Yes, sorry I don't have time for that anymore.
You have my permission to do whatever you think makes sense with this addon!
You can keep the name or rename it, fork it or can get full ownership of this repository. You can also remove my name from it.
Just tell me how you want to proceed and thank you for the time you put into it!

@ttobsen
Copy link
Collaborator Author

ttobsen commented Dec 19, 2019

Hi @dersphere, don't be worry. I'm very happy that someone spent so much time into this app.

If you don't mind, I would fork it on Gitlab.com (because I'm more into Gitlab than Github) and mirror it back to Github, keeping name and everything else (also you as the original author).

Giving me some permissions on this repo would be great. Than I would at least merge the changes to fix this bug and update the README after forking.

@dersphere
Copy link
Owner

You now should have push permission. Sure, sounds good.

@ttobsen
Copy link
Collaborator Author

ttobsen commented Dec 19, 2019

Thanks a lot. Maybe I find some time over winter holiday to optimize the 100 row limitation, the current implementation is very slow.

@jcjdammers You can download the Zip archive from https://github.com/ttobsen/plugin.audio.radio_de/archive/fix-100-rows-limitation.zip and install it manually on your Kodi device. The current fix isn't very perfomant, but I can at least reach every station.

I will create an update where the stations are prefetched. Glad that are winter holidays soon. 😃

@jcjdammers
Copy link

@ttobsen Many thanks, much appreciated. Will your changes also get pushed to the official Kodi add-on repo?

@ttobsen
Copy link
Collaborator Author

ttobsen commented Dec 19, 2019

Yes, I will push them, but I first want to reimplement the fix. Currently you can reach all stations, but the addon is slowed down and does not make much fun to handle. In winter holidays, I have some time to do improvements and then push the new version as soon as possible.

@jcjdammers
Copy link

@ttobsen Just received the update via Kodi repository. It works great and performance is fast !!!
Thanks.

@ttobsen
Copy link
Collaborator Author

ttobsen commented Jan 1, 2020

@jcjdammers This wasn't me. It seems that the addon is maintained also in the official XBMC addon repository, see https://github.com/XBMC-Addons/plugin.audio.radio_de

I just tried the new version 2.3.2 and it seems that the 100 station limitation is still there. Is this problem fixed in your setup?

Edit: My fault, I forgot to update to latest version :-(

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants