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.26k stars 30.24k forks source link

Netatmo climate rooms have a battery_level attribute #46151

Closed jansauer closed 3 years ago

jansauer commented 3 years ago

The problem

The Netatmo climate API has the concept of rooms. Thermostats and Valves are associated to Rooms to allow room-by-room heating control. As part of this multiple devices can be set in a same room. Currently home-assistant creates one climate domain entity per room, including a battery_level attribute. It is unclear which device this attribute refers to.

What i would expect is a climate entity per room without a battery_level and additionally a (sensor) entity per Netatmo device containing battery_level and maybe other device specific informations.

What is version of Home Assistant Core has the issue?

2021.2.1

What was the last working version of Home Assistant Core?

No response

What type of installation are you running?

Home Assistant OS

Integration causing the issue

netatmo

Link to integration documentation on our website

https://www.home-assistant.io/integrations/netatmo/#climate

Example YAML snippet

# Put your YAML below this line

Anything in the logs that might be useful for us?

# Put your logs below this line

My personal setup consists of a Netatmo Smart Thermostat and Smart valves in my living room. This results in the following entity in home assistant:

Screen Shot 2021-02-07 at 12 48 02

Via the API i get the following home status informations for these modules:

{
  "anticipating": false,
  "battery_level": 4061,
  "boiler_status": true,
  "boiler_valve_comfort_boost": false,
  "firmware_revision": 75,
  "id": "xxxxxx",
  "reachable": true,
  "rf_strength": 60,
  "type": "NATherm1",
  "bridge": "xxxxxx",
  "battery_state": "high"
},
{
  "battery_level": 2760,
  "firmware_revision": 85,
  "id": "xxxxxx",
  "reachable": true,
  "rf_strength": 76,
  "type": "NRV",
  "bridge": "xxxxxx",
  "battery_state": "low"
}

The netatmo app and webapp are warning me that the valves battery is low while home assistant only lists the living rooms battery level with 98% 😕

probot-home-assistant[bot] commented 3 years ago

netatmo documentation netatmo source (message by IssueLinks)

cgtobi commented 3 years ago

Thanks for letting us know. HA does not actually care about the individual valves but the rooms they are associated with since that is the level on what HA can control Netatmo valves/rooms. I understand that it is inconvenient with HA only reporting one of many valves per room and I'll take this into consideration about a future refactoring but this is nothing that can or will be fixed quickly.

github-actions[bot] commented 3 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 👍 This issue has now been marked as stale and will be closed if no further activity occurs. Thank you for your contributions.