nervetattoo / simple-thermostat

A different take on the thermostat card for Home Assistant ♨️
MIT License
743 stars 112 forks source link

Error when adjusting temp and hiding a setpoint #352

Open jheikkila920 opened 2 years ago

jheikkila920 commented 2 years ago

Describe the bug When you hide either the high or low setpoint and try to adjust the set point, an error occurs: Failed to call service climate/set_temreture. some but not all values in the same group of inclusion 'temerature' @ data[]

Expected behavior When hiding one of the set points, you should be able to still adjust the setpoint of unhidden setpoint.

# The least ammount of code possible to reproduce my error
type: custom:simple-thermostat
header: false
entity: climate.ac_zone_thermostat
setpoints:
  target_temp_low: null
  target_temp_high:
    hide: true

Screenshots image

Browser

joe-cole1 commented 2 years ago

Same error here.

Just to pile on with expected behaviour... it would be extremely handy to have an "offset" or "differential" setting when hiding one of the setpoints. Then you could adjust one while keeping the other within a set temp difference of the visible setpoint.

alexlmiller commented 2 years ago

Same error here as well - looks like the high setpoint just still needs to be passed as well when hidden and adjusting the low setpoint

AvenueSupplies commented 1 year ago

Same here, rats nest so no idea where to start

Uncaught (in promise) Object { code: "invalid_format", message: "some but not all values in the same group of inclusion 'temperature' @ data[]" } ​ code: "invalid_format" ​ message: "some but not all values in the same group of inclusion 'temperature' @ data[]" ​

: Object { … } ​​ __defineGetter__: function __defineGetter__() ​​ __defineSetter__: function __defineSetter__() ​​ __lookupGetter__: function __lookupGetter__() ​​ __lookupSetter__: function __lookupSetter__() ​​ __proto__: ​​ constructor: function Object() ​​ hasOwnProperty: function hasOwnProperty() ​​ isPrototypeOf: function isPrototypeOf() ​​ propertyIsEnumerable: function propertyIsEnumerable() ​​ toLocaleString: function toLocaleString() ​​ toString: function toString() ​​ valueOf: function valueOf() ​​ : function __proto__() ​​ : function __proto__()
heffneil commented 1 year ago

Is the original author maintaining this? I can't get it working and its quite frustrating. I know other people are using this without incident so I am really confused what is going on.