Core: OZW climate: mode 'off' invalid

Created on 16 Jul 2020  路  6Comments  路  Source: home-assistant/core

The problem


Toggling HVAC mode to off results in error.

Environment

  • Home Assistant Core release with the issue: 0.113b0
  • Last working Home Assistant Core release (if known): 0.112.3
  • Operating environment (OS/Container/Supervised/Core): container
  • Integration causing this issue: ozw
  • Link to integration documentation on our website: n/a

Problem-relevant configuration.yaml

n/a

Traceback/Error logs

2020-07-16 13:27:41 WARNING (MainThread) [homeassistant.components.ozw.climate] Received an invalid hvac mode: off

Additional information

When attempting to toggle the HVAC unit to off via the UI card this error was populated into the log.

ozw

Most helpful comment

Thanks, I actually fixed this (typo) in the PR that's about to be merged.
https://github.com/home-assistant/core/pull/37713/commits/c27e398e5c72bfdfa4378068f1b979a6e7879e8b

All 6 comments

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

Hmmm, this may happen if the device doesn't have an idle/off mode.
Can you provide the mqtt dump please so I can have a look ?

I can confirm the same. The rest of the modes are working except for Off. If I set the mode to off from the thermostat itself it's reported correctly in HA.

mqtt_dump.txt

OpenZWave/2/node/5/instance/1/commandclass/64/value/89128980/

{
    "Label": "Mode",
    "Value": {
        "List": [
            {
                "Value": 0,
                "Label": "Off"
            },
            {
                "Value": 1,
                "Label": "Heat"
            },
            {
                "Value": 2,
                "Label": "Cool"
            },
            {
                "Value": 3,
                "Label": "Auto"
            },
            {
                "Value": 11,
                "Label": "Heat Econ"
            },
            {
                "Value": 12,
                "Label": "Cool Econ"
            }
        ],
        "Selected": "Heat",
        "Selected_id": 1
    },
    "Units": "",
    "ValueSet": true,
    "ValuePolled": false,
    "ChangeVerified": false,
    "Min": 0,
    "Max": 0,
    "Type": "List",
    "Instance": 1,
    "CommandClass": "COMMAND_CLASS_THERMOSTAT_MODE",
    "Index": 0,
    "Node": 5,
    "Genre": "User",
    "Help": "Set the Thermostat Mode",
    "ValueIDKey": 89128980,
    "ReadOnly": false,
    "WriteOnly": false,
    "Event": "valueChanged",
    "TimeStamp": 1594827082
}

Thanks, I actually fixed this (typo) in the PR that's about to be merged.
https://github.com/home-assistant/core/pull/37713/commits/c27e398e5c72bfdfa4378068f1b979a6e7879e8b

@marcelveldt Could you point me to a related PR nr?

Was this page helpful?
0 / 5 - 0 ratings