Skip to content

Add low bandwidth conversions to MeshRadio#10595

Merged
vidplace7 merged 2 commits into
developfrom
low-bandwidth-conversions
Jun 1, 2026
Merged

Add low bandwidth conversions to MeshRadio#10595
vidplace7 merged 2 commits into
developfrom
low-bandwidth-conversions

Conversation

@vidplace7
Copy link
Copy Markdown
Member

@vidplace7 vidplace7 commented May 31, 2026

Adds support for low bandwidth LoRa conversion codes (7.8 kHz, 10.4 kHz, 15.6 kHz, 20.8 kHz, 41.7 kHz) in bwCodeToKHz and bwKHzToCode functions in MeshRadio.h.

Changes Made

  • Added bandwidth code mappings for codes 8, 10, 16, 21, and 42 in bwCodeToKHz()
  • Added corresponding reverse mappings in bwKHzToCode()
  • Added test cases for all new bandwidth codes in test_bwCodeToKHz_specialMappings
  • Added round-trip test test_bwCodeToKHz_roundTrip verifying bwKHzToCode(bwCodeToKHz(code)) == code for all special codes to prevent regressions

🤝 Attestations

  • I have tested that my proposed changes behave as described.
  • I have tested that my proposed changes do not cause any obvious regressions on the following devices:
    • Heltec (Lora32) V3
    • LilyGo T-Deck
    • LilyGo T-Beam
    • RAK WisBlock 4631
    • Seeed Studio T-1000E tracker card
    • Other (please specify below)

@vidplace7 vidplace7 requested review from Copilot and thebentern May 31, 2026 19:36
@vidplace7 vidplace7 added the enhancement New feature or request label May 31, 2026
@github-actions github-actions Bot added the needs-review Needs human review label May 31, 2026
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds bandwidth code/kHz conversions for narrower LoRa bandwidths (7.8, 10.4, 15.6, 20.8, 41.7 kHz) to support Amateur Radio experimentation via manual overrides. Presets are not affected.

Changes:

  • Extend bwCodeToKHz with codes 8, 10, 16, 21, 42.
  • Extend bwKHzToCode with matching reverse ranges.

@vidplace7 vidplace7 force-pushed the low-bandwidth-conversions branch from a3e837b to 76993ce Compare May 31, 2026 21:12
@thebentern thebentern requested a review from Copilot June 1, 2026 10:53
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated 1 comment.

Comment thread src/mesh/MeshRadio.h
Copilot AI changed the title Add low-bandwidth conversions to MeshRadio Add low bandwidth conversions to MeshRadio Jun 1, 2026
@vidplace7 vidplace7 merged commit cbfa8d7 into develop Jun 1, 2026
85 checks passed
@vidplace7 vidplace7 deleted the low-bandwidth-conversions branch June 1, 2026 22:24
@vidplace7 vidplace7 removed the needs-review Needs human review label Jun 1, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants