janw-cz / JWA_BME280

Arduino library for BME280 sensor based on reference driver
15 stars 9 forks source link

[Question regarding your Wrapper]: Proposal for Improvement #17

Open hasenradball opened 2 years ago

hasenradball commented 2 years ago

I hope you can understand my gernan? If not I can explain in english.

Hallo,

ich finde Deinen Wrapper Ansatz sehr gut, weil ich auf den Arduino Nano eine Anwendung schreiben möchte und da brauch eine sehr kleine Library. Mir sind zwei Dinge aufgefallen, die ich gerne als PR einbringen würde.

  1. Die Bosch Library kann sowohl als Float als auch Integer Werte zurückliefern, ich würde die library so anpassen dass man beides nutzen kann. Das kann ma mit einer minimalen Änderung in Deinem Wrapper ermöglichen.
  2. Würde ich gerne eine weitere Methode einführen, die mir quasi den Luftdruck bei NN zurückgibt. Der Wert wird auch für Wetter Vorhersagen verwendet. (siehe Adafruit BME280 Lib) float seaLevelForAltitude(float altitude, float pressure);

Ich würde gerne hier contributen weil ich den Ansatz toll finde. Die Adafruit Lib, die ich bisher verwende ist mir viel zu ausgeblasen.

Gib mir bescheid, ob es für Dich passt :-)

VG Frank