speleolontra / daikin_residential_altherma

Home Assistant Integration specific for BRP069A78 Heat Pump Model, via Daikin App Onecta. Users have good experience also for BRP069A62 and BRP069A61 (but not tested directly).
GNU General Public License v3.0
98 stars 14 forks source link

Read madoka wired thermostat values #2

Closed zauan closed 2 years ago

zauan commented 2 years ago

Hi there,

Great work on the integration. would it be possible to also pull data from the madoka thermostat ( for example current temperature )?

The temperature from madoka is seen in the native android app

zauan commented 2 years ago

I've made some modification to make the climate control to work properly.

You can find the changes at https://github.com/zauan/daikin_residential_altherma/commit/d63bd33c1a0105e56f5a7fbc7689dfc5309e5b0e

just let me know if you want me to create a pull request. From what I saw in your code, you've disabled this features for some reason :)

arn1s-m commented 2 years ago

I guess because if you don't have the remote it returns no values and no use. Is it actually worth it (madoka)? Somehow the hard-wiring requirement is keeping me away from getting it.

zauan commented 2 years ago

For me it works. I disliked the idea of a hardwire, however, I installed the heat pump in a new building and it was easy for me to wire the madoka thermostat.

In the end, with the modifications i've made to this integration I can fully use all the features from Hassio climate control:

image

If this will get merged in master I will probably work on the hot water control as I see that this is not included in the climate control

speleolontra commented 2 years ago

Hi Zauan, i see your edits... When i started this adventure (modify custom integration for Daikin air conditioning), I had no experience. But one of first thingh that I undertood is that I dont have a target temperature in my JSON. In fact my plant has 2 thermostat wifi (one per floor) that commands heat pump with a relè..

Without a target temp, I used Leaving water temp to display in climate control. immagine

Because another user has modified my code, my idea was to try to transfer values from const.py file to configuration.yaml, and manage null values to prevent errors... It's possible in your opinion? Any user could configure their own integration for their system by reading their own response json. Thanks Lontra Lontra

PS:

The temperature from madoka is seen in the native android app

Could you post an image of your app with temp of madoka? is Daikin Residential Controller App?

MicheleMercuri commented 2 years ago

PS:

The temperature from madoka is seen in the native android app

Could you post an image of your app with temp of madoka? is Daikin Residential Controller App?

Hi, here there is what you asked. Underlined in yellow is madoka temperature. Screenshot_20220109_003251

speleolontra commented 2 years ago

OK, it seems that is optional value...

In fact is not present in my interface... immagine

I can only set the offset from climatic curve. At the moment I didn't find a solution for transfer constants to configuration file, that is a possible solution to mantain only one integration. If you have an idea...

Lontra

jwillemsen commented 2 years ago

I have also a modoka thermostat, set on 22, as far as I can see it is set under temperatureControl, roomTemperature

daikin_data.json.txt

jwillemsen commented 2 years ago

Looks not all users do have a roomTemperature, maybe the controlMode can be useful and check if there is a roomTemperature below operationModes. Will try to implement such a thing later this week

MicheleMercuri commented 2 years ago

Hi many thanks 4 you excellent work, Can you implement water temperature that heat pump send to radiators/fancoils? I will see that value on heat pump display. regards.

jwillemsen commented 2 years ago

Please attach the daikin_data.json file that this integration downloads into your HA config directory? I will compare it to my Altherma data and see what I can easily expose.

MicheleMercuri commented 2 years ago

ok attached to this post. regards. daikin_data.json.txt

maybe that value is "leavingWaterTemperature".

speleolontra commented 2 years ago

@MicheleMercuri I think yes, is "leavingWaterTemperature". is not visible yet in your sensors? immagine

jwillemsen commented 2 years ago

Took me longer as I though, forget a @property@ that caused some strange errors. See https://github.com/speleolontra/daikin_residential_altherma/pull/13/files for my PR, when there is a roomTemperature in the returned JSON that is now shown and the target temperature can be set

MicheleMercuri commented 2 years ago

@MicheleMercuri I think yes, is "leavingWaterTemperature". is not visible yet in your sensors? immagine

No, isn't visible. I think due to heat pump settings, when I set Altherma to use madoka sensor that value is disappeared from mobile app and it is visible only on heat pump display.

jwillemsen commented 2 years ago

@MicheleMercuri could you attach the daikin_data.json when you use the madoka sensor and the value is disappeared from the mobile app, let us see if they provide it from their cloud, when the data is not there also home assistant can't show it.

MicheleMercuri commented 2 years ago

@MicheleMercuri could you attach the daikin_data.json when you use the madoka sensor and the value is disappeared from the mobile app, let us see if they provide it from their cloud, when the data is not there also home assistant can't show it.

It's here https://github.com/speleolontra/daikin_residential_altherma/files/7936906/daikin_data.json.txt

jwillemsen commented 2 years ago

leavingWaterTemperature does seem to be part of sensoryData, maybe wait until my changes are merged @MicheleMercuri and update to the latest master version and try again?

MicheleMercuri commented 2 years ago

leavingWaterTemperature does seem to be part of sensoryData, maybe wait until my changes are merged @MicheleMercuri and update to the latest master version and try again?

Ok I will try as soon as new update is published

jwillemsen commented 2 years ago

Can you retest, this should work now

MicheleMercuri commented 2 years ago

Can you retest, this should work now

Works perfectly excellent works 👍 many thanks. Regards. Screenshot_20220129_215124

jwillemsen commented 2 years ago

@zauan, can you retest with master and close this issue when it works?

zauan commented 2 years ago

Hi guys! Sorry for disappearing :) ... fully moved to the new house. I've just tested and it works perfectly.

I will close this ticket and remove my repository so we can have this in one place