Thermostat shouldn't expose Humidity cluster if null.

t0bst4r / home-assistant-matter-hub

Publish your Home-Assistant Instance using Matter.

Apache License 2.0

146 stars 8 forks source link

Thermostat shouldn't expose Humidity cluster if null. #168

Closed KipK closed 36 minutes ago

KipK commented 4 days ago

Humidity cluster shouldn't be exposed if climate entity doesn't have it as attribute.

This is not a priority issue as GH app is bugged and won't always displays it, But Google speakers will says there's 0% humidity in the related room.

t0bst4r commented 22 hours ago

Could we use TARGET_HUMIDITY of Supported Features for that? https://developers.home-assistant.io/docs/core/entity/climate/#supported-features

KipK commented 19 hours ago

I think this is for enabling humidity setpoint for climate devices ain't it?

t0bst4r commented 17 hours ago

Yes, seems like that, but having the humidity null could also mean it is currently unavailable but will be available later. Is it reliable to check if its not set at all (undefined in JavaScript) ?