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

Aufschalttemperatur wird falsch gelesen #61

Closed HMMike closed 5 months ago

HMMike commented 5 months ago

Hi,

Bei der IBN ist mir noch etwas aufgefallen. Zu dem Wunsch, die Aufschalttemperatur ändern zu können (siehe vorheriges Issue) fiel mir auf, das die Aufschalttemperatur schon falsch gelesen wird.

Im MQTT steht der Wert "AUS", definitiv eingestellt ist bei mir 2 Grad (Kontroliert an der Logamatik selber und nochmal mit der testweise rückgebauten KM271 und meinem alten Pi unter python

die Aufschalttemperatur findet sich in dem Telegramm: (vllt auch direkt im HK2 gucken)

HK1: 00 15 Aufschalttemp xx xx xx xx xx xx xx

HK2 00 4D AUfschalttemp2 xx xx xx xx xx xx xx

Wenn ich C nicht ganz verlernt habe, hast du da im KM271 ccp an der Stelle einen TIppfehler

case 0x0015: if (config.km271.use_hc1) { kmConfigNum.hc1_switch_on_temperature = data[2+4]; snprintf(kmConfigStr.hc1_switch_on_temperature, sizeof(kmConfigStr.hc1_switch_on_temperature), "%s", cfgArray.SWITCH_ON_TEMP[config.lang][limit(0, kmConfigNum.hc1_switch_on_temperature, 10)]); mqttPublish(addCfgTopic(cfgTopic.HC1_SWITCH_ON_TEMP[config.lang]), kmConfigStr.hc1_switch_on_temperature, config.mqtt.config_retain); // "CFG_HK1_Aufschalttemperatur" => "0015:0,a"

Der Offset ist da 2+0 und nicht 2+4, wie es im Quelltext steht. der HK2 müsste übrigens richtig sein, da steht nur data [2]

Gruss Michael

dewenni commented 5 months ago

Hallo Michael,

danke für den Hinweis. Ich habe es mir auch gerade angeschaut und du hast völlig recht, auch in deiner Analyse von meinem Code.

data[2+4] muss geändert werden in data[2+0] - ist wohl ein Tippfehler gewesen oder copy/paste

Das mit dem 2+x habe ich übrigens nur wegen der Lesbarkeit gemacht. In den ersten zwei Bytes steht die "Adresse" drin und danach kommen die eigentlichen Nutzdaten.

Kannst du es für dich selbst jetzt erstmal direkt selbst anpassen im Code und die geänderte SW auf das Board spielen, oder bist du auf ein Release angewiesen was du flashen kannst?

Aktuell fehlt mir noch etwas die Zeit an dem Projekt weiter zu machen. Die Zeit kommt sicher wieder, aber im Moment habe ich noch ein paar andere Dinge fertig zu machen.

HMMike commented 5 months ago

Ich hab leider nicht die Erfahrung mit den ESPs, um mir da selber das Programm neu zu compilieren. Hab ich für die ESPs noch nie gemacht

Da die Karte verbaut ist, müsste ich dann das OTA update machen können

Franck78 commented 5 months ago

@HMMike If you are able to build something for Arduino, this is exactly the same process for ESP32. More, it is easy to use this code with Arduino IDE, that's what I did.

Tip: when viewing source code in github, click at the line number, click the three dot, use it to link the line.

dewenni commented 5 months ago

solved with v3.2.5