Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Regression in Newer Versions With Minimum Set Temperature #149

Open
4 tasks done
BenJewell opened this issue Feb 8, 2025 · 2 comments
Open
4 tasks done

Regression in Newer Versions With Minimum Set Temperature #149

BenJewell opened this issue Feb 8, 2025 · 2 comments

Comments

@BenJewell
Copy link

System Health details

System Information

version core-2025.2.1
installation_type Home Assistant OS
dev false
hassio true
docker true
user root
virtualenv false
python_version 3.13.1
os_name Linux
os_version 6.6.73-haos
arch x86_64
timezone America/New_York
config_dir /config
Home Assistant Community Store
GitHub API ok
GitHub Content ok
GitHub Web ok
HACS Data ok
GitHub API Calls Remaining 4993
Installed Version 2.0.5
Stage running
Available Repositories 1519
Downloaded Repositories 22
Home Assistant Supervisor
host_os Home Assistant OS 14.2
update_channel stable
supervisor_version supervisor-2025.02.0
agent_version 1.6.0
docker_version 27.2.0
disk_total 30.8 GB
disk_used 11.0 GB
healthy true
supported true
host_connectivity true
supervisor_connectivity true
ntp_synchronized true
virtualization vmware
board ova
supervisor_api ok
version_api ok
installed_addons Terminal & SSH (9.16.0), Let's Encrypt (5.2.10), NGINX Home Assistant SSL proxy (3.11.1), File editor (5.8.0), Samba share (12.4.0), Advanced SSH & Web Terminal (20.0.0), Node-RED (19.0.0), OneDrive Backup (2.3.8), Mosquitto broker (6.5.0), Speedtest (1.6.2), Matter Server (7.0.0)
Dashboards
dashboards 5
resources 10
views 31
mode storage
Network Configuration
adapters lo (disabled), enp2s1 (enabled, default, auto), docker0 (disabled), hassio (disabled), veth2c9f57c (disabled), veth8622990 (disabled), veth28fbae9 (disabled), vetha4ad5df (disabled), veth81da43d (disabled), veth99ba75f (disabled), veth227920f (disabled), veth66850a5 (disabled), veth1028c80 (disabled)
ipv4_addresses lo (127.0.0.1/8), enp2s1 (10.0.1.23/8), docker0 (172.30.232.1/23), hassio (172.30.32.1/23), veth2c9f57c (), veth8622990 (), veth28fbae9 (), vetha4ad5df (), veth81da43d (), veth99ba75f (), veth227920f (), veth66850a5 (), veth1028c80 ()
ipv6_addresses lo (::1/128), enp2s1 (fe80::f7be:af2:47d7:a0e9/64), docker0 (fe80::42:f6ff:fe44:1dd2/64), hassio (fe80::42:2fff:fe67:e321/64), veth2c9f57c (fe80::546e:a9ff:fe11:38c1/64), veth8622990 (fe80::b47f:5cff:fe36:7351/64), veth28fbae9 (fe80::c0dc:c9ff:fe54:b32f/64), vetha4ad5df (fe80::1c5a:4dff:fe0b:729d/64), veth81da43d (fe80::40b2:fff:fe7d:32b4/64), veth99ba75f (fe80::3446:2eff:feda:771/64), veth227920f (fe80::509f:cff:fe7a:3f61/64), veth66850a5 (fe80::e031:d4ff:fee2:3fd6/64), veth1028c80 (fe80::40ae:b1ff:fe55:f9ee/64)
announce_addresses 10.0.1.23, fe80::f7be:af2:47d7:a0e9
Recorder
oldest_recorder_run January 26, 2025 at 9:53 PM
current_recorder_run February 8, 2025 at 6:14 PM
estimated_db_size 263.04 MiB
database_engine sqlite
database_version 3.47.1

Checklist

  • I have enabled debug logging for my installation.
  • I have filled out the issue template to the best of my ability.
  • This issue only contains 1 issue (if you have multiple issues, open one issue for each issue).
  • This issue is not a duplicate issue of currently previous issues..

Describe the issue

I can't find the commit at the moment, but there was a change in a recent update that reduced the minimum temperature for my spa from 59 F (15 C) to I think something like 46 F (8 C). Many spas including mine don't support these low temperatures, so they should no be selectable when dragging the set point in Home Assistant. Selecting one of these invalid temperatures does not do anything and eventually the selector reverts back. As a temporary workaround, I fixed it like this:

@property
def min_temp(self) -> float:
    """Get the minimum temperature."""
    # return self._automation_entity.min_temp
    return 59

We should give the user an option to manually set this value if there is no way to get the correct value from the spa automatically, which would be the ideal solution.

Reproduction steps

  1. Create a dashboard card with the climate entity provided by the integration, such as: climate.hot_tub_heater
  2. It SHOULD look like this, with 59 being the minimum setting. This is consistent with the minimum temperature setting selectable from the top side control on the spa:
Image
  1. Before I made the workaround, the range was larger.

Debug logs

Not relevant to this issue.

Diagnostics dump

The option for this is missing, but here is a snapshot from the spa:

{'Library Version': '1.0.1', 'SpaPackStruct.xml revision': '39.0', 'Spa pack': 'inYT 355 v5.0', 'Low level configuration #': 3, 'Log version': 66, 'Pack type': 10, 'Snapshot UTC Time': '2025-02-08 23:36:20.405225+00:00', 'Status Block': ['0x3', '0x1', '0xe', '0x1', '0xc', '0x0', '0x0', '0x0', '0x17', '0x0', '0x9', '0x0', '0x3', '0x1', '0x0', '0xc', '0x2', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0xe', '0x0', '0x0', '0x1', '0x0', '0x1', '0x2', '0x0', '0x1', '0x1e', '0xc', '0xc', '0x0', '0x0', '0x4', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x11', '0x0', '0x1', '0x30', '0x14', '0x78', '0x78', '0x2', '0x14', '0x2', '0x4', '0x8', '0x28', '0x28', '0x0', '0x73', '0x1', '0xe', '0x2', '0xe4', '0x0', '0x1', '0x20', '0x1', '0x0', '0x0', '0x1', '0x0', '0x0', '0xf', '0x2', '0xe5', '0x0', '0x2', '0x0', '0x2', '0x1', '0x1', '0x3', '0x3', '0x3', '0x5', '0x5', '0x4', '0x4', '0x1', '0x3', '0x6', '0x14', '0xf', '0x18', '0xf', '0xf', '0x14', '0x6', '0x6', '0x6', '0x6', '0x6', '0x6', '0xf4', '0x0', '0x2', '0x0', '0x1', '0x3', '0x0', '0x0', '0x14', '0x0', '0x1e', '0x1', '0xa', '0x30', '0x40', '0x80', '0x0', '0x83', '0x0', '0x0', '0x0', '0x0', '0x1', '0x0', '0x0', '0x0', '0x4', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x2', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x12', '0x0', '0x0', '0x0', '0x0', '0x0', '0x5', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0xff', '0xff', '0xff', '0x0', '0x2', '0x1', '0xe', '0x1', '0xe', '0x0', '0x0', '0x0', '0x40', '0x0', '0x0', '0x0', '0x40', '0xe', '0x0', '0xa', '0x4b', '0x0', '0x3d', '0x37', '0x0', '0x41', '0x42', '0x1', '0x63', '0x5', '0x0', '0x0', '0x1d', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x15', '0x18', '0x1', '0x0', '0x0', '0x0', '0x1', '0x18', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x2', '0x49', '0x1', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0xff', '0x24', '0x0', '0x0', '0x0', '0x0', '0x16', '0x81', '0x26', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x7f', '0xff', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x1', '0x2', '0x3', '0x4', '0x5', '0x6', '0x7', '0x8', '0x9', '0xa', '0xb', '0xc', '0xd', '0xe', '0xf', '0x10', '0x11', '0x12', '0x13', '0x14', '0x15', '0x33', '0x34', '0x35', '0x36', '0x97', '0x98', '0x99', '0x9a', '0xff', '0xff', '0xff', '0x69', '0x6e', '0x59', '0x54', '0x5f', '0x43', '0x36', '0x35', '0x2e', '0x78', '0x6d', '0x6c', '0x0', '0x0', '0x0', '0x0', '0x69', '0x6e', '0x59', '0x54', '0x5f', '0x53', '0x36', '0x36', '0x2e', '0x78', '0x6d', '0x6c', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0', '0x0'], 'intouch version EN': '70 v14.0', 'intouch version CO': '69 v11.0', 'Config version': 65, 'Integration Version': '0.1.18'}

@gazoodle
Copy link
Owner

Some spa packs seem to have min/max settings which I’ll use, others don’t. I might also put a config UI setting in to allow other ranges.

@BenJewell
Copy link
Author

Thanks! I updated from 0.1.18 to 0.1.25 and unfortunately the issue is still there.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants