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

bme68x pressure units #665

Merged
merged 1 commit into from
Dec 6, 2024
Merged

Conversation

mikeysklar
Copy link
Contributor

@mikeysklar mikeysklar commented Dec 6, 2024

bme68x is showing units as pascals (SI units) which the sensor puts out.

bme28x shows units as hector pascals ( hPa or millibars ) which is the more common way to discuss pressure.

This is a suggestion to stay with the BME28x hPA format which is nothing more than a divide by 100 for the BME68x to maintain consistency.

thread and images courtesy of wsquare58

forum thread on the issue.

IMG_2045

bme68x is show units as pascals.
@tyeth
Copy link
Contributor

tyeth commented Dec 6, 2024

Looks good, thanks Mikey! I've got checking old hPa/millibars in each pressure sensor to go through on a very long todo list, I'll bump it up 😁

@mikeysklar
Copy link
Contributor Author

Thank @tyeth ...

btw, I should mention this is untested. I don't have the BME68x on hand only a BME28x. I can order the BME68x if you want me to test through.

@tyeth
Copy link
Contributor

tyeth commented Dec 6, 2024

I've got some kicking about and will give a quick run through

@tyeth
Copy link
Contributor

tyeth commented Dec 6, 2024

Looks good @mikeysklar
Screenshot 2024-12-06 at 18 33 47

Merged. Will be released next week (or possibly tonight but unlikely). Feel free to share the build assets with the user

@tyeth tyeth merged commit b5dd8ed into adafruit:main Dec 6, 2024
39 checks passed
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.

2 participants