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 bme280 #98

Open
wants to merge 31 commits into
base: release/2.2.0
Choose a base branch
from
Open

Add bme280 #98

wants to merge 31 commits into from

Conversation

jmadden173
Copy link
Contributor

@jmadden173 jmadden173 commented Jun 10, 2024


name: Pull Request
about: Submit a pull request
title: ''
labels: ''
assignees: ''
reviewers: 'jmadden173'

Name/Affiliation/Title
John Madden, Maintainer, UCSC

Purpose of the PR
Add support for the bme280 temperature/pressure/humidity sensor.

Development Environment
Linux spruce 6.8.2-arch2-1 #1 SMP PREEMPT_DYNAMIC Thu, 28 Mar 2024 17:06:35 +0000 x86_64 GNU/Linux
PlatformIO Core, version 6.1.7
Used ST-Link V3 MINIE

Test Procedure
Describe the test procedure. For purely software, specifying the unit test is sufficient. Sometimes external configurations are required to test a hardware component. Create a new main in examples/ and describe the expected output. A photo or diagram is very helpful.

Additional Context
Every once in a while the sensors read extreme measurements for no apparent reason (See #99). Also modified the soil-power-sensor-protobuf library to include a optional parameter for raw measurements.

Closes #87

@jmadden173 jmadden173 linked an issue Jun 10, 2024 that may be closed by this pull request
@jmadden173 jmadden173 self-assigned this Jun 19, 2024
@jmadden173
Copy link
Contributor Author

Need to version bump python package before merge.

@jmadden173 jmadden173 requested a review from stevegtaylor June 26, 2024 17:14
@jmadden173 jmadden173 requested a review from jlin143 September 3, 2024 20:30
@jmadden173 jmadden173 changed the base branch from main to release/2.2.0 October 22, 2024 21:26
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.

Add BME280 sensor
1 participant