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

Add custom modem settings #16

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

sharph
Copy link
Collaborator

@sharph sharph commented Feb 20, 2017

  • Adds a new configuration for the modem: SF=12, Cr=4/7, Bw=125kHz, CRC on, Low data rate optimization on
    • Spreading factor 12, 4096 chips/symbol, is the slowest but gets us more link budget: 12.5dB more than what we were using previously according to the RF96 datasheet, p24
    • Coding rate 4/7 was identified as being the best value by @matt-knight in LoRa Modulation Settings #14
    • Bandwidth left as is. Could use more discussion
    • Low data rate optimization is enabled, as it's needed at larger time scales. This somehow makes the LoRa packets resistant to receiver and transmitter becoming unsynchronized
  • Changes default beacon period to 20 minutes, to prepare the network for slower data rates
  • Unrelated: adds BATTERY_SAVING constant so we can turn off battery saving modes completely on radios connected to computer or PocketCHIP, for example

This post was helpful in finding the barely documented bit for low data rate optimization mode.

This is just a first pass, which will hopefully generate more discussion in issue #14 about the best configuration to use. Let's hold off on merging pending discussion about bandwidth and if there are differing opinions about the other settings.

Add battery saving #define constant to turn battery saving modes on and
off
@travisgoodspeed
Copy link
Owner

If we need to do range tests, I could wire up a reflashable node to run nightly builds of this branch in Philly.

@sharph
Copy link
Collaborator Author

sharph commented Mar 4, 2017

@travisgoodspeed That's not a bad idea. If you set it up I'd use it for some comparative testing.

(Sorry for the long delay; work has been busy the past couple weeks.)

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.

2 participants