the78mole / esphome_components

ESPhome Components from the little digger
Other
24 stars 11 forks source link

select.sommer_winter_umschalttemperatur only changing heat circuit 1 #64

Closed AlexKusnezov closed 1 year ago

AlexKusnezov commented 1 year ago

Selector select.sommer_winter_umschalttemperatur is only chnaging heating circuit 1, HK2 stays unchanged, as it was set by 2107. Also sensor binary_sensor.hk2_sommer_modus stays in on state. I can verify that on the 2107 when I switch between circuits. (summer-icon reappears on the display and hk2 won't jump on, stays in summer mode)

Can you take a look please?

jensgraef commented 1 year ago

Hi Alex and welcome to our github page! I had a short look at the code and looks like the 2107 has two select fields for that and that select.sommer_winter_umschalttemperatur affects only one of it. https://github.com/the78mole/esphome_components/blob/main/components/km271_wifi/km271_communication_components.cpp#L175

So I'd suggest changing the code to rename select.sommer_winter_umschalttemperatur to reflect that it only chances heating circuit 1. Then we can add code for another select to change hc2, too. Unfortunately I'm unable to test that as I only have a heater with one circuit available. @qschneider: Would you mind adding and testing the select for hc2?

https://github.com/the78mole/esphome_components/blob/main/components/km271_wifi/km271_communication_components.cpp#L175

dewenni commented 1 year ago

I can confirm that "Sommer ab" and also "Frost ab" are separate values for HC1 and HC2. The setvalue commands are different and also the status values are different.

I have also separated this some time ago in my project. Maybe it is usefull to see the differences and the protocol informations here the related commit: dewenni/ESP_Buderus_KM271

AlexKusnezov commented 1 year ago

@jensgraef thanks guys, I really appreciate the work you have done here. It's truly amazing to able to interface with 2107 in such a granular way.

qschneider commented 1 year ago

@jensgraef i checked telegrams 08 00 65 09 65 65 65 65 (summer hc2 Temp = 9) and 08 31 65 65 65 65 65 04 (frost hc2 Temp=4). I can confirm that the summer mode needs to be setup separately if hc2 is present.

For the frost one it seems to be fine to just use 07 31 65 65 65 65 65 04 which will set the value while sending the hc2 telegramm result in no change or response at all. Can someone with hc2 setup confirm this as well?

qschneider commented 1 year ago

i made the changes to #65, pls review and merge, thx

jensgraef commented 1 year ago

@AlexKusnezov: The change is now on master and you should find a new binary version here: https://github.com/the78mole/esphome_components/blob/gh-pages/README.md

Please close this issue if it's working as expected.

Cheers Jens

AlexKusnezov commented 1 year ago

@jensgraef thanks, what I did before was just pasted this yaml in my esphome: https://github.com/the78mole/esphome_components/blob/main/components/km271_wifi/km271-for-friends.yaml then compiled and installed it via ota. You think I can do this again? I'm not sure if the bin file has everything enabled. When I recieved the unit a lot of this, especially the HC2 stuff was read-only. Is it the same with this bin file?

jensgraef commented 1 year ago

@AlexKusnezov yes, you can also do it that way. It how I do it myself ;-) Here's what need to be changed if you have made any customization in your yaml file and do not want to start from the one in the repository: https://github.com/the78mole/esphome_components/commit/b7919eacc8d0ed6ebf944338ec7b0ba03c741531

AlexKusnezov commented 1 year ago

@jensgraef thanks for the clarification. I kept everything pretty much vanilla besides the wifi stuff. I'll try that tomorrow and report back.

AlexKusnezov commented 1 year ago

@jensgraef everything went evry well. and it works really good. I can now set summer temp for both hc's indivudually. Thank you so much!

I'm just wondering why you can't do this on the Logamatic unit it self, because setting the temp there will change tamp for both hc's

jensgraef commented 1 year ago

That's good to hear! As for why you can't do this via the the menu one the Logamatic: I don't have a clue. I'm sure somebody thought it a good idea ;-)