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

Not working in HA 12.0 update #50

Closed
Lauriz7 opened this issue Dec 11, 2021 · 12 comments
Closed

Not working in HA 12.0 update #50

Lauriz7 opened this issue Dec 11, 2021 · 12 comments

Comments

@Lauriz7
Copy link

Lauriz7 commented Dec 11, 2021

Not working anymore, mqtt issues seen in log.

https://pastebin.com/56gh0zWW (log)
https://pastebin.com/dxdLBBrw (yaml)

Running Tasmota-ir 10.0

@tjhorner
Copy link

Can confirm. Please fix this issue ASAP, I am very cold.

@tjhorner
Copy link

tjhorner commented Dec 12, 2021

It looks like this issue was caused by mqtt.async_publish turning into a coroutine. It needs to be awaited or otherwise dealt with as a coroutine rather than a normal function call.

@stibbzy
Copy link

stibbzy commented Dec 12, 2021

Also confirming I have the same issue since upgrading to 2021.12...

@stibbzy
Copy link

stibbzy commented Dec 12, 2021

I have (at least as far as I can tell) fixed the issue by changing line 1006 of climate.py to self.hass.async_create_task(mqtt.async_publish(self.hass, self.topic, payload))

@nao-pon
Copy link
Collaborator

nao-pon commented Dec 13, 2021

I solved it by making the following changes in my repository.

nao-pon@f968c92

@akdri
Copy link

akdri commented Dec 13, 2021

also work's changing the last line of climate.py
mqtt.async_publish(self.hass, self.topic, payload) --> mqtt.publish(self.hass, self.topic, payload)

@hristo-atanasov
Copy link
Owner

Ok, I see there is a problem .. I'll read the change log .. And I have to update to the newest HA version too .. :)

@chicaneau
Copy link

also work's changing the last line of climate.py mqtt.async_publish(self.hass, self.topic, payload) --> mqtt.publish(self.hass, self.topic, payload)

can confirm this works for me too, thanks for the fix. This appears to be the cleanest way to fix the problem

@Lauriz7
Copy link
Author

Lauriz7 commented Dec 13, 2021

Ok, I see there is a problem .. I'll read the change log .. And I have to update to the newest HA version too .. :)

Can I also ask you to include fix AC power off in the new version? I think it was relevant for Panasonic/Daikin AC's

@kzagatti
Copy link
Contributor

also work's changing the last line of climate.py mqtt.async_publish(self.hass, self.topic, payload) --> mqtt.publish(self.hass, self.topic, payload)

Confirmed this works here too! Thanks

@hristo-atanasov
Copy link
Owner

I solved it by making the following changes in my repository.

nao-pon@f968c92

The new version fixes the problem. Thanks to @nao-pon! I want to apologize to everyone experiencing problems because of my slow fixes.

@nao-pon
Copy link
Collaborator

nao-pon commented Dec 14, 2021

I will soon create a PR that may solve some of the issues reported in Issues of this repository.
I would like to ask everyone to test.

@nao-pon nao-pon closed this as completed Dec 14, 2021
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

8 participants