search

freitdav / BDRthermostatHA

BDR (Baxi, De Dietrich, Remeha) thermostat custom component integration for Home Assistant
GNU General Public License v3.0
48 stars 21 forks source link

Allow BDRThermostat to work with multiple zones #19

Closed 3klips closed 1 year ago

3klips commented 1 year ago

This modification allow the component to work with mutiple zones. capabilities["centralHeatingZones"] is now a dictionary where the key is the zone name. Class BdrThermostat take the zone name as parameter.

pawelhulek commented 1 year ago

@3klips This is actually fixing my issue with climate control! Thank you for this contribution! 🎉 https://github.com/freitdav/BDRthermostatHA/issues/14 @freitdav I can confirm this is working quite nicely on DeDeitrich!

Videx1 commented 1 year ago

As i understand correctly with this you will be able to control more zones? Mine addon doesnt show any zone beacuse its 3 zone. I tried adding the code but the integration breaks, please add it to main. And yes a great integration i have been searching for the past year.

3klips commented 1 year ago

Hey @Videx1 , I think you have another issue. I have two zones and without the modification, the integration was working but was displaying only one zone.

But as @pawelhulek mentioned, I've tested only on a DeDietrich boiler.

jrathert commented 1 year ago

Thanks @3klips - I was facing the same issue (I have two zones in my Remeha Calenta Ace 28c), and to make it even more complicated, my first zone is actually not used at all, so showing really weird values. I was just about to think how to integrate more zones into the API code, when I saw your changes. Works like a charm! Had to find out, though, that when adding the integration to HA, at first all zones get added to the dashboard and then you have to hide/deactive those not needed - but I am really a HA noob ;-). Thanks again!

freitdav commented 1 year ago

Thank you, great contribution.