search

mariolukas / Watterott-CO2-Ampel-Plus-Firmware

Firmware for Watterott CO2 Ampel (Plus Version)
GNU General Public License v3.0
15 stars 13 forks source link

Temperaturoffset #44

Open awatterott opened 2 years ago

awatterott commented 2 years ago

Der Temperaturoffset wird im SCD30 fest abgespeichert und daher braucht man ihn nur einmal setzen. Ich würde daher vorschlagen den Offset nur zu setzen, falls er auf 0 eingestellt ist. Dadurch würde die Einstellung auch nicht bei einem Update verloren gehen und bei der Pro Version ist der Wert auch niedriger als wie bei der Plus Version, wo das WLAN-Modul unter dem Sensor sitzt.

https://github.com/mariolukas/Watterott-CO2-Ampel-Plus-Firmware/blob/main/CO2-Ampel_Plus/Sensor.cpp#L172

  if(co2_sensor.getTemperatureOffset() == 0)
  {
    co2_sensor.setTemperatureOffset(TEMPERATURE_OFFSET);
  }
mariolukas commented 2 years ago

Das klingt logisch und nachvollziehbar. Ich denke das werde ich anpassen.

btittelbach commented 1 year ago

Should be fixed now with marioluks last merge of my fork. Additional merge request also adds/fixes the ability to calibrate the temperature offset.