piotrva / esphome_gree_ac

GNU General Public License v3.0
3 stars 2 forks source link

Wrong current temperature reading #3

Open lokomoti opened 1 month ago

lokomoti commented 1 month ago

Hi Piotr,

I got your component working on TTGO display. Sometimes it does not react to the command but also in my case I am getting wrong reading of current temperature.

image

My AC is Sinclair MS-H09AIZ.

Appreciate your work on this. Thank you, Jakub

piotrva commented 1 month ago

Hi!

Could you please collect and attach log messages that shall include raw data transferred from your AC to the device?

Together with the temperature you except to see. Maybe the encoding is somehow different for your device...

lokomoti commented 1 month ago

I am including logs from uart i managed to set up. Please let me know if this is sufficient. logs_gree-ac-1_logs.txt

Current temperature should read 25. I can only check by switching the unit display to actual temperature so I am not sure if this is exact.

Jakub

piotrva commented 1 month ago

Hi, seems like logs are complete - that is what I needed.

[20:31:27][D][uart_debug:114]: <<< 7E:7E:2F:31:00:00:40:00:10:90:02:02:00:00:00:00:00:00:00:00:00:00:00:00:00:00:00:00:19:00:00:00:00:00:00:00:00:00:00:00:00:00:00:00:00:00:00:00:00:5D
[20:31:27][D][uart_debug:114]: >>> 7E:7E:2F:01:00:00:00:00:10:90:02:02:00:00:00:08:00:00:00:00:00:00:00:00:00:00:00:00:00:00:00:00:00:00:00:00:00:00:00:00:00:00:00:02:00:00:00:00:00:DE

First look at the log suggests that your report frames (from AC to WiFi module - marked by <<<) are shorter than mine. Third byte indicates frame length. My frames are 0x30, yours are 0x2F - exactly one byte shorter.

So maybe your AC does not report the temperature reading? Did your AC ever reported current temperature that was different from temperature requested? Can you collect the frames from AC to WiFi while ambient temperature around the unit changes by 2-3 Celcius? Can you maybe identify the sensor itself (a thermistor) somewhere in the enclosure?

As I was identifying this field I used a "compressed air" from a can to cool down the sensor.

If we confirm that your unit does not report the temperature - I will try to make a fix for this - shall be an easy one.

Sometimes it does not react to the command

This is already known issue #2 - for now I am scripting it in Home Assistant - need to find additional time for this.

lokomoti commented 1 month ago

So maybe your AC does not report the temperature reading?

It might be it. Previously I tried using some node.js mqtt bridge and the message didnt contain a current temperature reading. The same thing was when I used the HA Gree integration. The unit has to measure the temperature somehow to get feedback but it just seems like it doesnt report it.

Oddly if i change the mode on the unit led display to current temperature instead of setpoint (even the remote does not have button for that) it shows current reading.

Would it be possible to not read the temperature from the serial comms if the payload lenght is shorter?