adafruit / adafruit/Adafruit_CircuitPython_RFM9x
Should we add fix for SF=6?
- Dominant language
- Python
- Stars
- 76
- Forks
- 53
- PR merge metrics
- No merged PRs in 30d
Description
There is a PR in Adadruit_CircuitPython_RFM that fixes a known issue with Spreading Factor = 6
https://github.com/adafruit/Adafruit_CircuitPython_RFM/pull/10
Should this also be implemented here?
It will increase the size of the library and further impact the smaller M0 build (feather M0 rfm9x)?
Users who need SF=6 can use the Adafruit_CircuitPython_RFM library but that will not run on the small M0 boards.
SF=6 is not supported by the RadioHead Arduino library so without it, the adafruit_rfm9x library is still "compatible" with Radiohead ;-)
I don't know how much it will increase the library - likely a few 10s of bytes.
How you would like to proceed?
Contributor guide
No contributing guide indexed for this repository
Research direction
Start by reviewing PR #10 in Adafruit_CircuitPython_RFM and comparing its SF=6 fix with the Adafruit_CircuitPython_RFM9x implementation. Assess the effect on the smaller Feather M0 RFM9x build and RadioHead compatibility; done means a documented decision and, if accepted, a tested implementation.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- python
- Domain
- embedded-iot, networking
- Issue type
- Feature
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Needs clarification
- Newbie friendliness
- 35/100