Trunks1982 / Daikin-Rotex-HPSU-CAN

MIT License
15 stars 6 forks source link

Raumsoll as number #14

Closed TNTLarsn closed 6 months ago

TNTLarsn commented 6 months ago

Die Einstellung von Numerischen Werten sollte mit Hilfe einer "number" Entität erfolgen, nicht mit einem Auswahlfeld. Über den Anzeigemodus box oder slider kann individuell entschieden werden.

Trunks1982 commented 6 months ago

Was meinste dann auch WW Verstellung auch umändern?

TNTLarsn commented 6 months ago

Was meinste dann auch WW Verstellung auch umändern?

Ich denke schon. Hast du ja bereits gemacht! Ich würde noch einen PR für die Heizkurve erstellen.

PS: Anschließend sollten auch noch diese ganzen if conditions entfallen und die Berechnung der CAN-Befehle dynamisch erfolgen. https://github.com/munzili/ESPAltherma hast du ja bereits geforked. Ich denke "richtiger" wäre es dieses Repository komplett als Library einzubinden und dann zu verwenden. Hierzu reichen allerdings meine Skills noch nicht und es fehlt mir die Zeit hierfür.

Trunks1982 commented 6 months ago

Die dynamische Berechnung hätte ich gerne am Anfang schon gehabt aber programmieren kann ich nicht wirklich.Hab mir das alles mühselig zusammen gebaut🙈.Aber cool das du einwenig helfen kannst.

Glaube die Heizkurve würde ich als box wählen und nicht als slider.

Trunks1982 commented 6 months ago

Sehe gerade Heizkurve wird nicht verstellt.

TNTLarsn commented 6 months ago

Sehe gerade Heizkurve wird nicht verstellt.

Heizkurve habe ich selbst nicht implementiert. In welcher Richtung wird "nicht verstellt"? Vielleicht hier https://github.com/Trunks1982/Daikin-Rotex-HPSU-CAN/blob/e0b6e3b0f39672591a90cbe79b1802ae54e59c7a/Rotex-Daikin-CAN.yaml#L1287 noch folgendes hinzfügen?

auto call = id(set_heizkurve).make_call();
call.set_value(temperature);
call.perform();
Trunks1982 commented 6 months ago

Hatte ich eingebaut.Mhm funktionierte vorher.Sehe nur keinen Fehler.

TNTLarsn commented 6 months ago

id = set_heizkurve

Trunks1982 commented 6 months ago

?sehe den Fehler nicht

TNTLarsn commented 6 months ago

?sehe den Fehler nicht

https://github.com/Trunks1982/Daikin-Rotex-HPSU-CAN/pull/16