-
Notifications
You must be signed in to change notification settings - Fork 580
Add remote API for RDS STATION and RADIOTEXT #1373
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
Conversation
|
Thanks. This looks like it could be useful. Out of curiosity, what have you used it for? An RDS scanner? One issue I would anticipate: Gqrx's RDS decoder (a copy of gr-rds) reports partially-received program service name & radiotext. Thus there's a good chance that querying these values through the API will return incomplete data. Here in North America many broadcasters dynamically change the program service name to display scrolling text, and Gqrx will happily return a mixture of old and new data during changes. Do you think it would be useful to modify gr-rds so that it only reports complete program service name & radiotext? |
In the standard, this field is called "Program Service name" or "PS name". Maybe something like |
|
I've tested this PR and it works very well combined with my improved RDS decoder. |
We used to have a discord bot that can stream FM stations using GQRX, and it was used to report played song.
I found that also, some stations do it as well. I'm not sure how could it be fixed tough. This was the reason I wrote "usually name of the played song".
Again, I'm not sure how could it be done to work well, but worth a try, bud possibly in another PR. |
Renamed to that. |
|
I'm working on some improvements to gr-rds that should make this work better: bastibl/gr-rds#86 |
|
I've pulled the gr-rds improvements into Gqrx here: #1394 |
|
I did some cleanup (d9727bd) and manually merged this. As vladisslav2011 had done, I changed the default value for PS name and RadioText to the empty string. I also removed the string trimming, since I expect the improvements in #1394 make it unnecessary. Downstream applications can apply their own string transformations if they so desire. Please let me know the names of the authors of this code so I can add them to the readme. |
This PR adds 2 new queries to the "get parameter" command
p:RDS_STATIONRDS_PS_NAMEreports program service name (usually station text)RDS_RADIOTEXTreports the radio text (usually name of the played song)Example: