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
71.11k stars 29.79k forks source link

Hue integration: BOSCH_SMART_HOME_INTRUSION #42975

Closed pgrafe closed 3 years ago

pgrafe commented 3 years ago

The problem

I have a Hue controller with some lights and I have a Bosch Smart Home Controller. The Bosch Smart Home Controller is linked with the Hue system, so I can controll Philips lights with the Bosch App. Now I did my first steps into Home Assistant. The Hue integration worked very well, besides one strange issue.

Under lights there are to light groups BOSCH_SMART_HOME_INTRUSION and BOSCH_SMART_HOME_WATER_ALARM which I think are created by the Bosch Smart Home System in the Hue System. I can turn on / off all lights in my house. I think they are for alerts in the Bosch Smart Home System when a intrusion oder water alarm is rised.

The problem now is, that I can not remove or edit these entries because they have no unique ID. I think they should not be listed in Home Assistant at all, because they seem to be hidden groups for the connection of Hue and Bosch only. They are not visible in my Hue App or in the Bosch App.

Environment

I am Running with Home Assistant 0.117.5. The problem was seen in 0.117.2.

Problem-relevant configuration.yaml

Traceback/Error logs

Additional information

grafik

pgrafe commented 3 years ago

Entity page: grafik

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

hue documentation hue source (message by IssueLinks)

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

Hey there @balloob, @frenck, mind taking a look at this issue as its been labeled with an integration (hue) you are listed as a codeowner for? Thanks! (message by CodeOwnersMention)

balloob commented 3 years ago

Would you be able to query your Hue hub and see the API response that contains those groups? That way we would be able to see if there is some property that we can use to filter them out.

pgrafe commented 3 years ago

Would you be able to query your Hue hub and see the API response that contains those groups? That way we would be able to see if there is some property that we can use to filter them out.

Yes. Thanks for looking into it. For everyone who is interrested in querying your Hue Controller. Here a guide: https://developers.meethue.com/develop/get-started-2

Fetching all groups I get this:

{
    "1": {
        "name": "UG Spielkeller",
        "lights": [
            "14",
            "15",
            "12",
            "13",
            "3"
        ],
        "sensors": [],
        "type": "Room",
        "state": {
            "all_on": false,
            "any_on": false
        },
        "recycle": false,
        "class": "Nursery",
        "action": {
            "on": false,
            "bri": 254,
            "hue": 14988,
            "sat": 141,
            "effect": "none",
            "xy": [
                0.4575,
                0.4101
            ],
            "ct": 366,
            "alert": "none",
            "colormode": "ct"
        }
    },
    "2": {
        "name": "Group for wakeup",
        "lights": [
            "16",
            "17",
            "18",
            "9",
            "10",
            "11",
            "19",
            "6"
        ],
        "sensors": [],
        "type": "LightGroup",
        "state": {
            "all_on": false,
            "any_on": true
        },
        "recycle": true,
        "action": {
            "on": true,
            "bri": 254,
            "ct": 156,
            "alert": "none",
            "colormode": "ct"
        }
    },
    "3": {
        "name": "DG Anna",
        "lights": [
            "18",
            "16",
            "17"
        ],
        "sensors": [],
        "type": "Room",
        "state": {
            "all_on": false,
            "any_on": false
        },
        "recycle": false,
        "class": "Kids bedroom",
        "action": {
            "on": false,
            "bri": 254,
            "ct": 443,
            "alert": "none",
            "colormode": "hs"
        }
    },
    "4": {
        "name": "OG Eltern",
        "lights": [
            "19"
        ],
        "sensors": [],
        "type": "Room",
        "state": {
            "all_on": false,
            "any_on": false
        },
        "recycle": false,
        "class": "Bedroom",
        "action": {
            "on": false,
            "bri": 254,
            "ct": 443,
            "alert": "none",
            "colormode": "ct"
        }
    },
    "5": {
        "name": "EG Küche",
        "lights": [
            "7",
            "8"
        ],
        "sensors": [],
        "type": "Room",
        "state": {
            "all_on": true,
            "any_on": true
        },
        "recycle": false,
        "class": "Kitchen",
        "action": {
            "on": true,
            "bri": 254,
            "ct": 443,
            "alert": "none",
            "colormode": "ct"
        }
    },
    "6": {
        "name": "OG Matthea",
        "lights": [
            "6"
        ],
        "sensors": [],
        "type": "Room",
        "state": {
            "all_on": true,
            "any_on": true
        },
        "recycle": false,
        "class": "Kids bedroom",
        "action": {
            "on": true,
            "bri": 254,
            "ct": 156,
            "alert": "none",
            "colormode": "ct"
        }
    },
    "7": {
        "name": "DG Noah",
        "lights": [
            "11",
            "9",
            "10"
        ],
        "sensors": [],
        "type": "Room",
        "state": {
            "all_on": false,
            "any_on": false
        },
        "recycle": false,
        "class": "Kids bedroom",
        "action": {
            "on": false,
            "bri": 254,
            "ct": 443,
            "alert": "none",
            "colormode": "ct"
        }
    },
    "8": {
        "name": "DG",
        "lights": [
            "18",
            "16",
            "17",
            "11",
            "9",
            "10",
            "1",
            "5"
        ],
        "sensors": [],
        "type": "Zone",
        "state": {
            "all_on": false,
            "any_on": true
        },
        "recycle": false,
        "class": "Top floor",
        "action": {
            "on": true,
            "bri": 1,
            "ct": 443,
            "alert": "none",
            "colormode": "hs"
        }
    },
    "9": {
        "name": "DG Treppe",
        "lights": [
            "1",
            "5"
        ],
        "sensors": [],
        "type": "Room",
        "state": {
            "all_on": true,
            "any_on": true
        },
        "recycle": false,
        "class": "Staircase",
        "action": {
            "on": true,
            "bri": 1,
            "alert": "none"
        }
    },
    "10": {
        "name": "BOSCH_SMART_HOME_INTRUSION",
        "lights": [
            "7",
            "19",
            "5",
            "6",
            "11",
            "12",
            "18",
            "13",
            "3",
            "14",
            "8",
            "1",
            "16",
            "17",
            "9",
            "15",
            "10"
        ],
        "sensors": [],
        "type": "LightGroup",
        "state": {
            "all_on": false,
            "any_on": true
        },
        "recycle": false,
        "action": {
            "on": false,
            "bri": 254,
            "hue": 8418,
            "sat": 140,
            "effect": "none",
            "xy": [
                0.4573,
                0.41
            ],
            "ct": 443,
            "alert": "none",
            "colormode": "ct"
        }
    },
    "11": {
        "name": "BOSCH_SMART_HOME_WATER_ALARM",
        "lights": [
            "7",
            "19",
            "5",
            "6",
            "11",
            "12",
            "18",
            "13",
            "3",
            "14",
            "8",
            "16",
            "1",
            "17",
            "9",
            "15",
            "10"
        ],
        "sensors": [],
        "type": "LightGroup",
        "state": {
            "all_on": false,
            "any_on": true
        },
        "recycle": false,
        "action": {
            "on": false,
            "bri": 254,
            "hue": 8418,
            "sat": 140,
            "effect": "none",
            "xy": [
                0.4573,
                0.41
            ],
            "ct": 443,
            "alert": "none",
            "colormode": "ct"
        }
    },
    "12": {
        "name": "OG",
        "lights": [
            "19",
            "6"
        ],
        "sensors": [],
        "type": "Zone",
        "state": {
            "all_on": false,
            "any_on": true
        },
        "recycle": false,
        "class": "Upstairs",
        "action": {
            "on": true,
            "bri": 254,
            "ct": 156,
            "alert": "none",
            "colormode": "ct"
        }
    },
    "13": {
        "name": "EG",
        "lights": [
            "7",
            "8"
        ],
        "sensors": [],
        "type": "Zone",
        "state": {
            "all_on": true,
            "any_on": true
        },
        "recycle": false,
        "class": "Downstairs",
        "action": {
            "on": true,
            "bri": 254,
            "ct": 443,
            "alert": "none",
            "colormode": "ct"
        }
    },
    "14": {
        "name": "UG",
        "lights": [
            "14",
            "15",
            "12",
            "13",
            "3"
        ],
        "sensors": [],
        "type": "Zone",
        "state": {
            "all_on": false,
            "any_on": false
        },
        "recycle": false,
        "class": "Staircase",
        "action": {
            "on": false,
            "bri": 254,
            "hue": 14988,
            "sat": 141,
            "effect": "none",
            "xy": [
                0.4575,
                0.4101
            ],
            "ct": 366,
            "alert": "none",
            "colormode": "ct"
        }
    }
}

So it seems to be the type "LightGroup". Now I also recognized, that the Group "Group for wakeup" has the same type. Its a group build by the Hue app for the Routine "Wakeup".

So it may be that these could be usefull, but I think they should be hidden. Because in die Hue App they are generated automatically and are also hidden and can not be edited. Instead of the type "Zone". If I change these in the App they also change there name in Home Assistant.

balloob commented 3 years ago

Here are the available group types: https://developers.meethue.com/develop/hue-api/groupds-api/#214_notes

LightGroup is the default for user generated groups. I don't know if that note is outdated and that nowadays it's all Room that matters.

Mariusthvdb commented 3 years ago

Hi Paulus, please allow me to ask this, since you mention checking for filtering on properties: would it be possible to add a core HA distinction for Zones? I created a dedicated Fr for that when zones became available but that might have slipped attention. would be great if we could filter on Zone, if eg is_hue_zone could be set for these light entities in HA, like we now haveis_hue_group

pgrafe commented 3 years ago

Here are the available group types: https://developers.meethue.com/develop/hue-api/groupds-api/#214_notes

LightGroup is the default for user generated groups. I don't know if that note is outdated and that nowadays it's all Room that matters.

Could be Room or Zone nowadays.

Mariusthvdb commented 3 years ago

yes, where Zones are/can be member of a Room. Very useful in larger setups, hence my request to have a HA indication for is_hue_zone:

still, dont want to diverge this issue, so if needed and Paulus would be willing to implement, I will again repost the FR.

balloob commented 3 years ago

Shouldn't we just deprecate is_hue_group and instead include group_type: <string>?

Mariusthvdb commented 3 years ago

Shouldn't we just deprecate is_hue_group and instead include group_type: <string>?

which then could be 'Room', 'Zone', 'Entertainment' and 'LightGroup'?

yes please! that would be awesome progress. If adding that, please also add the class:

coming to think of this some more, would it in anyway be possible to have these entities use another domain than light? thing is, in all templates, and all card configs we need to exclude these rooms and zones, because they are not individual lights (entitities).

using a new attribute group_type would already make that filtering easier. not using light.xxx would so even more.

balloob commented 3 years ago

We can't. They control lights. I rather not have them at all, hence why they are disabled by default.

Mariusthvdb commented 3 years ago

ok, I see. Its not that bad, and indeed they control lights ;-)

adding group_type: then would be great. and class: ...

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.

pgrafe commented 3 years ago

The BOSCH_SMART_HOME - Entries are gone.