dahlb / ha_carrier

Carrier Infinity Integration for Home Assistant
MIT License
42 stars 3 forks source link

Integration showing wrong thermostat state after midnight #33

Closed myevit closed 1 year ago

myevit commented 1 year ago

System Health details

System Information

version core-2023.6.2
installation_type Home Assistant OS
dev false
hassio true
docker true
user root
virtualenv false
python_version 3.11.4
os_name Linux
os_version 6.1.21-v8
arch aarch64
timezone America/Edmonton
config_dir /config
Home Assistant Community Store GitHub API | ok -- | -- GitHub Content | ok GitHub Web | ok GitHub API Calls Remaining | 5000 Installed Version | 1.32.1 Stage | running Available Repositories | 1278 Downloaded Repositories | 14
Home Assistant Cloud logged_in | false -- | -- can_reach_cert_server | ok can_reach_cloud_auth | ok can_reach_cloud | ok
Home Assistant Supervisor host_os | Home Assistant OS 10.3 -- | -- update_channel | stable supervisor_version | supervisor-2023.06.2 agent_version | 1.5.1 docker_version | 23.0.6 disk_total | 28.5 GB disk_used | 9.4 GB healthy | true supported | true board | rpi4-64 supervisor_api | ok version_api | ok installed_addons | File editor (5.6.0), AdGuard Home (4.8.10), Matter Server (4.6.1), Let's Encrypt (4.12.9), NGINX Home Assistant SSL proxy (3.5.0)
Dashboards dashboards | 2 -- | -- resources | 5 views | 5 mode | storage
Recorder oldest_recorder_run | 16 June 2023 at 13:59 -- | -- current_recorder_run | 20 June 2023 at 20:45 estimated_db_size | 316.95 MiB database_engine | sqlite database_version | 3.41.2

Checklist

Describe the issue

I have a scheduling enabled in my thermostat. 22:00 to 7:00 it's in sleep mode with lower temperature set points. Termostat works as expected photo_2023-06-25 11 37 44 While the integration showing different picture after 00:00

Screenshot 2023-06-25 at 11 38 53 AM

Reproduction steps

N/A

Debug logs

N/A

Diagnostics dump

No response

dahlb commented 1 year ago

please attach debug level logs or diagnostics dumps from before, during, and after if possible

myevit commented 1 year ago

here is about 24 hours of logs. By the way, carrier_api still posts serial info into the logs, including PIN and url..... probably needed to be sanitized... carrier_api.log.zip ha_carrier.log.zip

dahlb commented 1 year ago

thanks those logs were illuminating, but looks like it is a problem in the logic of the scheduling periods so I released 1.20.6 with improved ha_carrier.log entries that expose the results of the current logic. please update to 1.20.6 and attach the re log, just the ha_carrier.log is needed this time.

myevit commented 1 year ago

here is new log Archive.zip

dahlb commented 1 year ago

thanks for uploading those logs, I think I found the bug and it should be fixed in 1.20.7

myevit commented 1 year ago

Great! I will give it a go and let you know! Thank for doing this!

myevit commented 1 year ago

Confirming all works as intended. thanks!