radio.check_rds producing gibberish

[githubguy007]

I'm getting garbage characters randomly displayed on the serial port when radio.check_rds() is called in the main loop when running this example code.

Eventually this results in Feather crashes when running this example on Adafruit Feather ESP32-S2 and Feather ESP32-S2 TFT with "Adafruit CircuitPython 7.3.3 on 2022-08-29."

Traceback (most recent call last):
File "code.py", line 162, in <module>
File "tinkeringtech_rda5807m.py", line 385, in check_rds
File "tinkeringtech_rda5807m.py", line 478, in read16
OSError: [Errno 19] No such device

Code done running.
Auto-reload is on. Simply save files over USB to run them or enter REPL to disable.

Press any key to enter the REPL. Use CTRL-D to reload.

[tinkeringtech]

Hi, do you get audio from the headphones? When the signal strength for the tuned station is weak, you would typically see garbage on the serial monitor. Does RDS work when tuned to other stations. I have not seen that timeout before.

[githubguy007]

Yup, getting audio.

And, I'm seeing semblances of human readable text on some stations. I'll work on my antenna, and perhaps relocate the receiver to see if I can make it cleaner.

Thanks!

[githubguy007]

Still experiencing the timeout issue....

[tinkeringtech]

I tried to reproduce this on a Adafruit Metro ESP32-S2 express, but no luck. Must be something specific to memory allocation on a ESP32-S2 since your error happens on the RDS data. Will see if I can hit the error.

[lambtor]

Still experiencing the timeout issue....

Like the screenshot here, I run into a similar issue. RDS data seems to update text, but longer strings don't get the ends removed when new data is shorter than older data.

[tinkeringtech]

@lambtor apologies, we don't have a fix for this at this time. When the library was initially created, it was for short RDS strings, so you must be hitting the limits of what the RDS buffer can handle.

[HilmarSt]

In tinkeringtech_rda5807m.py , in def replace_element(index, text, newchar) replace `

    # this used to be an AND but that would make no sense. Changed to OR
    if newchar < 127 or newchar > 31:
        newlist[index] = chr(newchar)

´ with `

    if (newchar < 127) and (newchar > 31):
        newlist[index] = chr(newchar)
    else:
        newlist[index] = '_' # or any character you like

´