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

Beacon settings are not provided in frequency plans #64

Closed
reissjason opened this issue Jun 24, 2023 · 3 comments
Closed

Beacon settings are not provided in frequency plans #64

reissjason opened this issue Jun 24, 2023 · 3 comments
Assignees
Labels
needs/triage We still need to triage this

Comments

@reissjason
Copy link

Summary

Basic station expects the LNS to provide beacon parameters in radio_conf. It makes sense that the LNS would control the beacon parameters as it sets all the radio channels.

Support for beacons settings and tests are available in the current release of the TTN stack.
https://github.com/TheThingsNetwork/lorawan-stack/blob/v3.26/pkg/pfconfig/lbslns/lbslbs_test.go#L151
...

Environment

  • Frequency plan: all
  • Region: all
  • End device brand, model and firmware: all
  • Gateway brand, model and firmware: all
  • Network Server version: all

Steps to Reproduce

  1. Connect basic station
  2. bcning field is not provided in the radio_conf message as shown in station debug.

What is already there? What do you see now?

The bcning field can be sent to basic station since v3.21.2
https://www.thethingsnetwork.org/forum/t/upgrade-to-the-things-network-v3-21-2-completed/58810

https://github.com/TheThingsNetwork/lorawan-stack/blob/98e65a7069fd55593fa20fa5a7f44b15bfeba02d/pkg/pfconfig/lbslns/lbslns.go#L251
...

What is missing? What do you want to see?

Beaconing settings in frequency plans or additional plans for optional beaconing.
...

How do you propose to implement this?

...

Can you do this yourself and submit a Pull Request?

...

@github-actions github-actions bot added the needs/triage We still need to triage this label Jun 24, 2023
@adriansmares
Copy link
Contributor

adriansmares commented Jun 27, 2023

The Things Stack provides the bcning since version 3.21.2 (TheThingsNetwork/lorawan-stack#5745), and our automatic testing also checks for this internally.

Could you please post some debug Basics Station logs with a version which does not support the bcning ? What frequency plans did you test this with ?

@reissjason
Copy link
Author

Perhaps I was wrong. I am seeing the "bcning" field. We had been testing basic station 2.0.5 which I see has a known issue for transmitted beacon settings. We have updated to 2.0.6 and now see beacons at our end-device and receive downlink in ping slots.

@reissjason
Copy link
Author

Working after upgrade to Basic Station 2.0.6.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs/triage We still need to triage this
Projects
None yet
Development

No branches or pull requests

2 participants