home-assistant / core

:house_with_garden: Open source home automation that puts local control and privacy first.
https://www.home-assistant.io
Apache License 2.0
72.8k stars 30.5k forks source link

my.tado.com delivers no sensorDataPoints for type "HOT_WATER" #7059

Closed wmalgadey closed 7 years ago

wmalgadey commented 7 years ago

https://github.com/home-assistant/home-assistant/issues/6943#issuecomment-293270162

{  
   "tadoMode":"AWAY",
   "geolocationOverride":false,
   "geolocationOverrideDisableTime":null,
   "preparation":null,
   "setting":{  
      "type":"HOT_WATER",
      "power":"OFF",
      "temperature":null
   },
   "overlayType":null,
   "overlay":null,
   "openWindow":null,
   "link":{  
      "state":"ONLINE"
   },
   "activityDataPoints":{  

   },
   "sensorDataPoints":{  

   }
}
wmalgadey commented 7 years ago

@CrazyChaos I don't think, your hot water "device" will work with a climate device! It seems that it is not delivering data for the current data. Can you "turn it on" and post the state result again?

CrazyChaos commented 7 years ago

Hey wmalgadey, hereby:

{  
   "tadoMode":"HOME",
   "geolocationOverride":false,
   "geolocationOverrideDisableTime":null,
   "preparation":null,
   "setting":{  
      "type":"HOT_WATER",
      "power":"ON",
      "temperature":{  
         "celsius":55.00,
         "fahrenheit":131.00
      }
   },
   "overlayType":null,
   "overlay":null,
   "openWindow":null,
   "link":{  
      "state":"ONLINE"
   },
   "activityDataPoints":{  

   },
   "sensorDataPoints":{  

   }
}
CrazyChaos commented 7 years ago

I use the Tado thermostat btw. Not a climate device.

wmalgadey commented 7 years ago

Yes, I think that the "HOT_WATER" type is for your heating system (a boiler!?). And this has apparently no sensorDataPoints :(

CrazyChaos commented 7 years ago

Correct, I do have a boiler in the unit... Now what?

wmalgadey commented 7 years ago

Maybe it just gives sensorDataPoints, when it actually does something? Do you see the current temperature for this zone in my.tado.com?

I would skip the creation of climate devices for zones with type == HOT_WATER. In the moment I create additional sensors for type==HEATING, but I could create special sensors for HOT_WATER. So currently I could just present the information for the current temperature setting (55°C in your example).

wmalgadey commented 7 years ago

the commit https://github.com/wmalgadey/home-assistant/commit/a7e88bdadf5a4468f0355190e8ac72acfe6ca57e fixed your problem, but it is no good solution, when there is no actual data to present, isn't it?

CrazyChaos commented 7 years ago

The only thing you see and can do is set the temperature and set is on or off for the same periodes as the wanted temperature at home. That's it. The current temperature isn't displayed.

wmalgadey commented 7 years ago

Ok! Than I will remove the climate device for this type and will just add some sensors. To see the state and the temperature setting

CrazyChaos commented 7 years ago

And not able to control them?

wmalgadey commented 7 years ago

I think a climate device is to much, for just switching it on and of. I will look how it is possible to add this action to a sensor!

does the on/off switch and the temperature setting work in the moment?

CrazyChaos commented 7 years ago

What do you mean with in the moment? Direct? Yes they do. You can only change the temperature when it is ON between 30 and 65 `C. But that maybe has to do with my central heating.

wmalgadey commented 7 years ago

Yes I mean, if you could change the on/off state with the current implementation?

CrazyChaos commented 7 years ago

No I can't. Can't do anything tbh

CrazyChaos commented 7 years ago

Hey @wmalgadey, it is quite quiet here. Just curious, did you make some progress?

wmalgadey commented 7 years ago

I didn't forget about this :)

But I am quite busy at work in the moment. I hope I can start off again end of next week.

CrazyChaos commented 7 years ago

You did fix: 6943, did you make some progress on this one?

wmalgadey commented 7 years ago

I did some progress on this. Maybe you try my custom component? I merged the changes from hass, so it can be used to test the functionality (https://github.com/wmalgadey/tado_component)

balloobbot commented 7 years ago

There hasn't been any activity on this issue recently. Due to the high number of incoming GitHub notifications, we have to clean some of the old issues, as many of them have already been resolved with the latest updates.

Please make sure to update to the latest Home Assistant version and check if that solves the issue. Let us know if that works for you by adding a comment :+1:

balloobbot commented 7 years ago

This issue will be auto-closed because there hasn't been any activity for a few months. Feel free to open a new one if you still experience this problem 👍