Skip to content

solving_station_info #20

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

Merged
merged 1 commit into from
Jun 6, 2021
Merged

solving_station_info #20

merged 1 commit into from
Jun 6, 2021

Conversation

jposada202020
Copy link
Contributor

This resolves #9 .

According to the data sheet page 28, this has to be setup only at powerup, however for testing purposes I change the station every three seconds, behavior in the video is normal as mentioned in the datasheet, the databuffer has a length of 4 characters, so it will update half buffer at a time

Results

20210604_195139_1

Hardware

Adafruit CircuitPython 6.2.0 on 2021-04-05; Adafruit Feather RP2040 with rp2040

with a Adafruit SI4713

TEST CODE

import board
import digitalio
import adafruit_si4713
import time
from random import choice

si_reset = digitalio.DigitalInOut(board.D5)
si4713 = adafruit_si4713.SI4713(board.I2C(), reset=si_reset, timeout_s=0.5)
si4713.tx_frequency_khz = 87600
si4713.tx_power = 115

stations = [b'-', b'IO', b'Neo', b'Hans', b"Cappy", b'Neokey', b'Minerva', b'Blinka', b'Adafruit']

while True:
    si4713.configure_rds(0xADAF, station=choice(stations), rds_buffer=b'Electronic Hits')
    time.sleep(2)

@jposada202020
Copy link
Contributor Author

@caternuson hacky but works, for a almost two years issue, I say lets not get into the rabbit hole. there are scary things in the library LOL

@caternuson
Copy link
Contributor

What if station name is greater than 8 characters?

@jposada202020
Copy link
Contributor Author

Well at least here, that does not apply.I verify all the radio stations in FM and they do not have a name longer than 8. I do not know where you are. If you use a station longer it will cut to 8 characters. If any, we identify where the problem is :).
I could transfer what I find out to the issue and close this PR, what do you think?

@caternuson
Copy link
Contributor

Let's just merge this. This stuff isn't super well documented in the datasheets unfortunately. This seems to at least make it work better. If it still is some kind of issue, can open a new one later.

@jposada202020
Copy link
Contributor Author

Thanks :)

@jposada202020 jposada202020 merged commit d10230d into adafruit:master Jun 6, 2021
@jposada202020 jposada202020 deleted the solving_station_information branch June 6, 2021 22:52
adafruit-adabot added a commit to adafruit/Adafruit_CircuitPython_Bundle that referenced this pull request Jun 8, 2021
Updating https://github.com/adafruit/Adafruit_CircuitPython_BME280 to 2.6.5 from 2.6.4:
  > Moved default branch to main
  > Merge pull request adafruit/Adafruit_CircuitPython_BME280#52 from jposada202020/memory_otimization
  > Moved CI to Python 3.7

Updating https://github.com/adafruit/Adafruit_CircuitPython_HTU31D to 1.1.1 from 1.1.0:
  > Merge pull request adafruit/Adafruit_CircuitPython_HTU31D#6 from gmparis/main
  > Moved CI to Python 3.7
  > Added help text and problem matcher
  > Added pull request template

Updating https://github.com/adafruit/Adafruit_CircuitPython_ICM20X to 2.0.8 from 2.0.7:
  > Linted
  > Moved default branch to main
  > Moved CI to Python 3.7
  > Added help text and problem matcher
  > Added pull request template

Updating https://github.com/adafruit/Adafruit_CircuitPython_IRRemote to 4.1.0 from 4.0.6:
  > Moved default branch to main
  > Merge pull request adafruit/Adafruit_CircuitPython_IRRemote#42 from danielballan/nonblocking
  > Moved CI to Python 3.7
  > Added help text and problem matcher
  > Added pull request template

Updating https://github.com/adafruit/Adafruit_CircuitPython_L3GD20 to 2.3.6 from 2.3.5:
  > Merge pull request adafruit/Adafruit_CircuitPython_L3GD20#23 from jposada202020/correcting_measuremnt_units
  > Moved default branch to main
  > Moved CI to Python 3.7
  > Added help text and problem matcher
  > Added pull request template

Updating https://github.com/adafruit/Adafruit_CircuitPython_SGP40 to 1.1.0 from 1.0.3:
  > Merge pull request adafruit/Adafruit_CircuitPython_SGP40#4 from CrakeNotSnowman/voc_algorithm_dev
  > Moved CI to Python 3.7
  > Added help text and problem matcher
  > Added pull request template

Updating https://github.com/adafruit/Adafruit_CircuitPython_SHTC3 to 1.1.2 from 1.1.1:
  > Moved default branch to main
  > Merge pull request adafruit/Adafruit_CircuitPython_SHTC3#11 from jposada202020/master
  > Moved CI to Python 3.7
  > Added help text and problem matcher
  > Added pull request template

Updating https://github.com/adafruit/Adafruit_CircuitPython_SI4713 to 1.3.2 from 1.3.1:
  > Moved default branch to main
  > Merge pull request adafruit/Adafruit_CircuitPython_SI4713#20 from jposada202020/solving_station_information

Updating https://github.com/adafruit/Adafruit_CircuitPython_SSD1306 to 2.11.6 from 2.11.5:
  > Corrected url
  > Merge pull request adafruit/Adafruit_CircuitPython_SSD1306#64 from jposada202020/correcting_example_link

Updating https://github.com/adafruit/Adafruit_CircuitPython_Colorsys to 2.0.1 from 2.0.0:
  > Moved default branch to main
  > Merge pull request adafruit/Adafruit_CircuitPython_Colorsys#17 from GomiHgy/patch-1
  > Moved CI to Python 3.7
  > Added help text and problem matcher
  > Added pull request template

Updating https://github.com/adafruit/Adafruit_CircuitPython_FunHouse to 2.1.4 from 2.1.3:
  > Merge pull request adafruit/Adafruit_CircuitPython_FunHouse#17 from jposada202020/adding_new_guides
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.

Short station names not updating
2 participants