set radio idle before reading temperature#57
Conversation
|
This PR will add a few bytes to the adafruit_rfm69 library (I have not checked the actual size) and may impact the CP builds for the feather M0 RFM69. I suppose this could also be "fixed" with a documentation note that instructs the user to always call .idle() before reading the .temperature property. This will require the user code to have the extra call. Adding it as in the PR is "cleaner". |
tannewt
left a comment
tannewt
left a comment
There was a problem hiding this comment.
Thanks! We can tweak the CP builds if needed when we update the frozen version.
Updating https://github.com/adafruit/Adafruit_CircuitPython_EPD to 2.17.0 from 2.16.0: > Merge pull request adafruit/Adafruit_CircuitPython_EPD#100 from adafruit/jd79667 > Merge pull request adafruit/Adafruit_CircuitPython_EPD#99 from adafruit/uc8253 Updating https://github.com/adafruit/Adafruit_CircuitPython_RFM69 to 2.1.26 from 2.1.25: > Merge pull request adafruit/Adafruit_CircuitPython_RFM69#57 from jerryneedell/jerryn_temperature Updating https://github.com/adafruit/Adafruit_CircuitPython_SPA06_003 to 1.0.1 from 1.0.0: > remove debugging print. move float() cast to coeff reading instead of equations. Updating https://github.com/adafruit/Adafruit_CircuitPython_Qualia to 2.1.2 from 2.1.1: > Merge pull request adafruit/Adafruit_CircuitPython_Qualia#11 from makermelissa/main
2 participants
fixes #56
tested on feather rp2040 rfm69 (CP 10.0.0-rc.0) and in Raspberry Pi 5 with RFM69 bonnet.