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

AC telling "inactive" while it's working when temperature Is as set point #77

Closed
h4de5 opened this issue Jul 19, 2022 · 3 comments · Fixed by #80
Closed

AC telling "inactive" while it's working when temperature Is as set point #77

h4de5 opened this issue Jul 19, 2022 · 3 comments · Fixed by #80

Comments

@h4de5
Copy link
Owner

h4de5 commented Jul 19, 2022

Here everything is back working with 2022.7.1. This morning wasn't.

The only thing I noticed Is AC telling "inactive" while it's working when temperature Is as set point, not working like this before

thats right. this is something that was not release before. I tested some stuff, it should return idle state, so you can check in automations for this. as the AC unit does not go into idle mode, this does not reflect reality. Ill create a new issue for this.

Originally posted by @h4de5 in #72 (comment)

@dario-brombal
Copy link

It says "Inattivo", in italian can be traslated as inactive or idle. Maybe this issue Is related also to #55 ?

Anyway in past days, when turning off AC from cooling, It briefly went to off, for something like a second, then goes to cleaning, so there is actually some information about current state

@h4de5
Copy link
Owner Author

h4de5 commented Jul 19, 2022

yes cleaning is provided as an overall flag - and its overrides the current state the unit is in. I also asked if the state can be extraced from the api and it seems it cannot. see: KaSroka/Toshiba-AC-control#31

so this idle state will be removed again.

@tetienne
Copy link
Contributor

tetienne commented Jul 19, 2022

@h4de5 The whole method hvac_method should be removed if we want to apply HA core rules. HA does not allow computed state. See this comment.

It was the topic of this issue.

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

Successfully merging a pull request may close this issue.

3 participants