dewenni / ESP_Buderus_KM271

Control your Buderus Logamatic 2107 or HS-2105 with MQTT or Home Assistant
MIT License
54 stars 10 forks source link

plötzlich falsche Aussentemperatur? #52

Closed kalleick closed 5 months ago

kalleick commented 7 months ago

Hallo,

ich habe plötzlich 255 Grad Außentemperatur, was könnte das sein, Sensor defekt?

2023-12-02 10_42_08-Buderus Logamatic

Iam-klm commented 7 months ago

Das Problem ist bekannt und ist nur auf dem Dashboard falsch. Unter dem Reiter allgemeine Anzeige stimmen die Werte. Schönes Wochenende.

Heusel commented 7 months ago

Kann Iam-klm bestätigen unter MQTT/ Grafana / allgemeine Werte stimmen die Temperaturen.

Scheint nur im Dashboard bei negativen Temperaturen falsch darzustellungen.

Ich habe Zeile 1140/1141 aus der Datei WEBUI.cpp in Verdacht. Hier sollte anstelle von uint8ToString ein int8ToString stehen.

bisher:

geändert:

Hab ne Testversion für mich gebaut, allerdings hab ich jetzt keine Minusgrade mehr und kein Kältespray im Haus. Somit noch keinen abschliessenden Test .
Wer also heute eine kalte Nacht erwartet kann gerne mit testen :-)

dewenni commented 7 months ago

Ja da ist der Fehler. Das habe ich auch bereits korrigiert und ist somit im nächsten Release gefixed. Ich wollte jetzt nur alles deswegen ein neues Release machen. Ist ja wirklich nur die Anzeige im Dashboard der WebUI.

MQTT Werte sind davon nicht betroffen.

Grüße Sven

Heusel commented 7 months ago

Super danke für den schnellen fix und das coole Projekt von euch. Dann freuen wir uns auf das nächste Release. Deswegen lohnt es sich nicht wirklich was neues rauszubringen. Ist ja jetzt ausdiskutiert

Franck78 commented 5 months ago

@Heusel , "I built a test version for myself, but now I no longer have temperatures below zero and no cold spray in the house. So no final test yet."

 // Outdoor Temp
  ESPUI.updateLabel(id.dash.tmp_out_act,  int8ToString(kmStatusCpy.OutsideTemp - 20));

ah ah, no need to freeze, just subtract something and your are done. So yes, int8 is good fix. The thing is, are there others to find ;)

dewenni commented 5 months ago

solved with v3.2.5