-
Notifications
You must be signed in to change notification settings - Fork 914
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
Support for the ClimateGuard RadSens Geiger-Muller tube #5425
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for your contribution, for some minor changes see inline comments.
Since this is a rather specific sensor and we're very close to the flash limit on nRF52 devices, I think it would be good to introduce a macro like |
Do you care to have it inside of this, or can i merge it and create another PR? i wanted to introduce exclude_ethernet too... |
I'm fine with merging this as is. PR looks good otherwise. |
This PR is to support the ClimateGuard RadSense dosimeter.
A fun and educational sensor for Meshtastic; not for safety critical applications.
This is an I2C device which integrates nicely into the firmware along side other sensors.
This pairs with a pull [request to add a field and enumeration to the protobufs](Support for the ClimateGuard RadSens Geiger-Muller tube #626)
Changes:
modules/Telemetry/Sensor/CGRadSensSensor.*
- adds support for reading the sensormodules/Telemetry/EnvironmentTelemetry.cpp
- updates to handle the new sensor, also updates the display code to not show temperature/humidity unless it is present in the measurement.Thanks for your consideration of this, my first contribution to Meshtastic