radical-squared / aquatemp

Home Assistant AquaTemp Heat Pump Climate Entity
33 stars 9 forks source link

fail to set target temperature after HA upgrade to 2024.8 #94

Open eddysteurs opened 1 month ago

eddysteurs commented 1 month ago

*After upgrading HA to 2024.8 it fails to set the target temperature to different value. On the HA temperature card (and climate entity) it does change the value but it does not flow to the actual device. So on next refresh from all sensors it is back on the original value. When setting it via the temperature card (or an automation) is gives error 'Failed to perform the action climate/set_temperature. '<' not supported between instances of 'float' and 'NoneType''

AfmanS commented 4 weeks ago

Probably related to https://github.com/home-assistant/core/issues/123612

Here's my traceback:

  File "/usr/src/homeassistant/homeassistant/components/automation/__init__.py", line 755, in async_trigger
    return await self.action_script.async_run(
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/src/homeassistant/homeassistant/helpers/script.py", line 1799, in async_run
    return await asyncio.shield(create_eager_task(run.async_run()))
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/src/homeassistant/homeassistant/helpers/script.py", line 463, in async_run
    await self._async_step(log_exceptions=False)
  File "/usr/src/homeassistant/homeassistant/helpers/script.py", line 527, in _async_step
    self._handle_exception(
  File "/usr/src/homeassistant/homeassistant/helpers/script.py", line 557, in _handle_exception
    raise exception
  File "/usr/src/homeassistant/homeassistant/helpers/script.py", line 525, in _async_step
    await getattr(self, handler)()
  File "/usr/src/homeassistant/homeassistant/helpers/script.py", line 764, in _async_call_service_step
    response_data = await self._async_run_long_action(
                    ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/src/homeassistant/homeassistant/helpers/script.py", line 727, in _async_run_long_action
    return await long_task
           ^^^^^^^^^^^^^^^
  File "/usr/src/homeassistant/homeassistant/core.py", line 2763, in async_call
    response_data = await coro
                    ^^^^^^^^^^
  File "/usr/src/homeassistant/homeassistant/core.py", line 2806, in _execute_service
    return await target(service_call)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/src/homeassistant/homeassistant/helpers/service.py", line 999, in entity_service_call
    single_response = await _handle_entity_call(
                      ^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/src/homeassistant/homeassistant/helpers/service.py", line 1071, in _handle_entity_call
    result = await task
             ^^^^^^^^^^
  File "/usr/src/homeassistant/homeassistant/components/climate/__init__.py", line 938, in async_service_temperature_set
    if check_temp < min_temp or check_temp > max_temp:
       ^^^^^^^^^^^^^^^^^^^^^
TypeError: '<' not supported between instances of 'float' and 'NoneType'
eddysteurs commented 2 weeks ago

no I dont think it is the same problem as my temperature I want to set is well within min and max temp range. I guess it might be cause (link to HA update) because I do not get all the correct parameters in the climate entity via de aquatemp integration for my phnix (hitemp wifi) heatpump. Since many month I override them via an automation (min, max temp, state and target temperature) with the appropriate aquatemp sensors (or attribute) and this allowed me to change the target temp via the climate device but since HA 2024.8 it now gives this error mentioned above, is this something you can fix in aquatemp integration?