esphome-daikin-s21
esphome-daikin-s21 copied to clipboard
Improve compatibility and silent mode
First commit improves compatibility (discussed in issue #7) with units that don't respond to RH, RI, Ra and RL queries. This should be a minimally intrusive way to do it, by moving them to a "failable" queries list. This way, we now always expect response from F9 (all of them should according to what I found online), and for units that don't return their external temperature there, we'll get it from Ra query anyway.
Second commit adds silent fan mode. Nothing special, just another value in enum. Tested and working.
Tested and working on my FTXC-B. Didn't got the main branch working due to unknown queries: RH, RI, Ra, RL. F9 does work and gives me indeed an indoor and outdoor temperature.
I've tested also some "experimental" queries and "F2", "F3", "F4", "M" seem to work in addition to "F1", "F5", "Rd". I can possibly check in the future. Currently on my FTXC-B I don't have the coil temp and fan speed (rpm), possibly this isn't an option on mine either.
[14:02:28][D][daikin_s21:367]: ** UNKNOWN QUERIES **
[14:02:28][D][daikin_s21:294]: S21: G2 -> 4:\x80\x80 (4)
[14:02:28][D][daikin_s21:341]: Unknown response G2 -> "4:\x80\x80"
[14:02:28][D][daikin_s21:294]: S21: G3 -> 0000 (4)
[14:02:28][D][daikin_s21:341]: Unknown response G3 -> "0000"
[14:02:28][D][daikin_s21:294]: S21: G4 -> 0 \x800 (4)
[14:02:28][D][daikin_s21:341]: Unknown response G4 -> "0 \x800"
[14:02:28][D][daikin_s21:261]: NAK from S21 for RN query
[14:02:28][D][daikin_s21:261]: NAK from S21 for RX query
[14:02:28][D][daikin_s21:261]: NAK from S21 for RD query
[14:02:28][D][daikin_s21:294]: S21: M -> 1344 (4)
[14:02:28][D][daikin_s21:341]: Unknown response M -> "1344"
[14:02:29][D][daikin_s21:294]: S21: G\x00\x00\x00 -> \x00\x00 (2)
[14:02:29][D][daikin_s21:341]: Unknown response G\x00\x00\x00 -> "\x00\x00"
@joshbenner what about this pull request? Can it be merged? The change request seems to be fixed.
Availability of the silent mode is a must have for me. @joshbenner Will this be merged ?
Is there any chance of this being merged?
Interestingly, my units respond to the silent fan mode command by switching into the automatic fan mode. This seems like an acceptable compromise against the s21 quirks. so I'll get this merged.