The radio playlist are limited to 200 tracks
Maintainers usually reply within 5 days
Nobody has claimed this yet.
Assessment
- Difficulty
- 2/5
- Estimated time
- 1-3 hours
- Newbie friendliness
- 62/100
Research direction
The cap is the countryTrackLimit constant of 200 in external/radios/countries.go, around line 22, which the reporter identifies as the cause. Read that file and find where the constant is used to see how the station list is fetched and truncated. Before changing it, check the code comment explaining the limit, since the maintainer may want a higher cap or pagination rather than removal. Done means the country's full station list is shown, or the limit is raised with a clear reason, and the radio panel still navigates smoothly.
Written by the indexing model from the issue text.
Description
What happened
Using cliamp to browse radio streaming station per countries allow to browse only 200 items althought there are much more
Steps to reproduce
- ./cliamp
- Ctrl + R to refresh the radio stations
- Shift + R to go on the radio panel
- Browse all countries
- Select United States of America
- Select By Name
Expected behavior
All the existing radio for this country should be displayed, but we've got only 200
Screenshots / video
Environment
- cliamp version (
cliamp --version): v2.3.0-713-g02309c90-dirty - OS: Mac Os
- Audio backend (PipeWire / PulseAudio / ALSA):
- Provider(s) involved: Rqdio
In fact I've found the cause in the code. On external/radios/countries.go L22 there is a countryTrackLimit of 200 (or to the the cap much higher). Could we get rid of it in order to display all of the available radios for a country? In contrary of the comments in the code some people would like to browse the complete playlist of a country to see all of the existing web radios easyly. I will do a pull request for that or perhaps you've got another idea about to make my PR less dirty or you don t want to go over 200 displayed radios at the same time :)
- Dominant language
- Go
- Stars
- 4.4k
- Forks
- 307
- Avg merge
- 4d 18h
- Merged PRs (30d)
- 13
Getting set up
- No Dockerfile or Docker Compose file
- Has a pull request template
- No contributing guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
More from bjarneo/cliamp
-
bug
Difficulty 1/5 Under an hour Newbie friendliness 92/100
Maintainers usually reply within 5 days
-
Show the Podcast episodes in newest first orderPossibly taken @lchojnack claimed this 16 days ago. Openenhancement
Difficulty 2/5 1-3 hours Newbie friendliness 69/100
Maintainers usually reply within 5 days
-
bug
Difficulty 2/5 1-3 hours Newbie friendliness 65/100
Maintainers usually reply within 5 days
-
bug
Difficulty 2/5 1-3 hours Newbie friendliness 76/100
bjarneo/cliamp#520 · 2 reactions ·
Maintainers usually reply within 5 days
-
enhancement
Difficulty 2/5 1-3 hours Newbie friendliness 65/100
Maintainers usually reply within 5 days
Similar issues
-
Difficulty 2/5 1-3 hours Newbie friendliness 72/100
Azure/aztfexport#690 · 1 reaction ·
-
Difficulty 2/5 1-3 hours Newbie friendliness 78/100
kubecost/ibm-finops-agent#289 ·
Maintainers usually reply within 2 days
-
ai-driven-fuzzing bug Minor
Difficulty 2/5 1-3 hours Newbie friendliness 78/100
linagora/cozy-stack#5162 ·
Maintainers usually reply within 1 day
-
Difficulty 2/5 1-3 hours Newbie friendliness 76/100
prometheus/prometheus-mcp#213 ·
Maintainers usually reply within 3 days
-
Deleting a client-role keycloak_generic_role_mapper removes all roles of that client from the scope mappingPossibly taken @albarkauss claimed this today. Open
Difficulty 2/5 1-3 hours Newbie friendliness 78/100
keycloak/terraform-provider-keycloak#1786 ·
Maintainers usually reply within 5 days