search

hristo-atanasov / Tasmota-IRHVAC

Home Assistant platform for controlling IR Air Conditioners via Tasmota IRHVAC command and compatible hardware
196 stars 67 forks source link

Climate card does not update with the state_topic #26

Closed dedalusjinja closed 3 years ago

dedalusjinja commented 3 years ago

The card is not updated if I control the air conditioner with the remote control, but I can correctly control it from HA. I have tried both Daikin and Gree but it doesn't change, I get the message correctly if I listen to MQTT. Some idea? Thanks

hristo-atanasov commented 3 years ago

Do you use some of the releases or the master branch?

dedalusjinja commented 3 years ago

The master branch

hristo-atanasov commented 3 years ago

can you see any error or warnings in HA log that has in common with tasmota_irhvac?

dedalusjinja commented 3 years ago

no kind of error in the log. I tried with several receivers, I got the files and configurations of a friend who has it working but I do not modify the card

hristo-atanasov commented 3 years ago

Hmm .. Interesting .. Can you write the "vendor:" or "protocol:" in lowercase (with small letters) in your config, restart HA and try again? I mean like the following, but with your protocol or vendor. Instead of "ELECTRA_AC", you write "electra_ac". protocol: "electra_ac", or vendor: "electra_ac"

dedalusjinja commented 3 years ago

It does not change. Maybe this can help, I have created a mqtt sensor

sensor:

and not even this sensor acquires the remote control code of the air conditioner, but that of the TV does. While if I listen to Mqtt I see both codes

dedalusjinja commented 3 years ago

this error came to me when checking config for version 118: Testing configuration at /tmp/config WARNING:homeassistant.components.climate:ClimateDevice is deprecated, modify TasmotaIrhvac to extend ClimateEntity

hristo-atanasov commented 3 years ago

I'm no shure what the problem could be. If anyone can share any oppinion here, it would be great. Looks like some issue with MQTT, or the MQTT broker .. Not sure.

tigers75 commented 3 years ago

The only thing I noticed is that also the operating modes have to be written all lowercase in configuration.yaml, while in Tasmota they appear with the first letter capitalized (in tasmota I see "Heat", but in config I must write "heat"). Since all home assistant names must be lowercase I think maybe that's related to it? Anyway if it works it's not a big problem, but I'd point it out in the examples.

hristo-atanasov commented 3 years ago

@tigers75 When modes come from Tasmota, they are explicitly set to lowercase. Also there was a bug that even the Vendor was lowercased too (and it shouldn't). Now it is fixed. This has any chance to be the reason for this issue, but not sure. When I read about the MQTT sensor, that does not catch the irreceived result from using the remote, I thing the problem is bigger and outside the TasmotaIRHVAC scope. Could be related to Tasmota. There is a limit about how big can be a single topic message. Many remote codes are higher than this limit. I'm talkin about #define MQTT_MAX_PACKET_SIZE 1500 // Tasmota v8.1.0.8 Any message bigger than this size woud be truncated or skiped .. This size could be set to bigger number, but you need to recompile it.

hristo-atanasov commented 3 years ago

@tigers75 I got it from the first message. No need to constantly comment about it. Read this:

https://github.com/hristo-atanasov/Tasmota-IRHVAC/issues/28

Master branch is updated already. And put the vendor exactly as you see it in Tasmota console.

dedalusjinja commented 3 years ago

with the latest update of the custom_component everything works perfectly! thanks