Comments (5)
The function doesn't retrieve the temperature offset from the sensor, therefore it is set to none ( see function description of the get_sensor_settings e.g. here )
So it might be a bit confusing but it is the currently expected result. 🤔
from bme680.
@nickbroon good point, I see why this is confusing. As @caemor stated this is kind of expected since this temperature offset configuration is not stored in the sensor.
Here are some options I see:
- return the value from `self.tph_sett.temperature_offset, which hides that it does not come from the sensor
- not store it in the
TphSett
struct but separately to distinguish it - add better comments to
TphSett.temperature_offset
I favor 1. or 2. but can not decide if it's really relevant to distinguish if this value comes from the sensor or not. What do you both think ? More ideas are welcome =).
from bme680.
I think 1 would be preferable, as I don't think there is need to distinguish what is stored in the hardware and what is stored in the driver. I tend to think of the driver and hardware as one unit from an api perspective.
from bme680.
I agree with nick that 1 would be the best from a user view
from bme680.
Fixed by 0c90c96 and released on 0.4.2
from bme680.
Related Issues (14)
- Error when trying to run the example HOT 3
- Feature Request: Add setting to offset calibrate the temperature HOT 7
- no_std compatible HOT 1
- Feature Request: SPI Support HOT 1
- Multiple calls to set forced power mode necessary to get a valid read HOT 2
- The newest version always returns the same value for the gasresistance HOT 5
- Owning the delayer appears problematic HOT 7
- Avoid floating point precision when reading values from the sensor
- This crate is awesome but example needs to be improved HOT 1
- BME688 gives constant humidity and gas readings HOT 2
- Why does SettingsBuilder::with_humidity_control set a heater flag?
- Multiple gas heater profile support
- Add optional serde support for `FieldData`
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from bme680.