-
Notifications
You must be signed in to change notification settings - Fork 77
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
[FEATURE] Improve swing mode Home Assistant Integration #588
Comments
I cannot easily test as I don't have that icon/option on my HA for some reason. If you set Let me know if they work, and I can make a beta. |
I was able to test, and seems to work, bus my HA is not showing anything special apart from those names, e.g. I have made a beta release now, so just try a normal update to beta. |
Yeh, I also applied some changes to try and get TCP to work better and concerned it may have upset things. Investigating now. |
My initial testing worked, the beta did not, which is a problem. |
OK, the beta is fixed - do you have the means to solder USB leads to the board, to reflash it? |
This looks like a bug in ESP IDF that has caught me a few times now. |
I just got the Faikin Module (also pulled out the whole indoor unit in the process, which needed some replacement wall anchors apparently). Do you have a wiring diagram for which pin is which for the USB cable? |
Can I directly solder the leads from a USB cable to the PCB? Or do I need to use a programmer? If so, I only have ESP01 USB programmers here, does it work with this kind of programmer? |
OK programming is explained end of https://github.com/revk/ESP32-RevK which can be done using Chrome. This is pin out. |
Thanks, sorry for the hassle. |
So do we think some other config would work? Maybe "Horizontal" not "SWING_HORIZONTAL" or some such? |
If it does not help I'll change back so it is what people are used to. But if you can find a config that works better, let me know. |
@revk somewhat off-topic, but seems I got unlucky with automatic upgrade timing. 2 out 3 of my Faikin boards probably got the upgrade with that bad firmware you mentioned earlier (they are no longer connecting to network). I had beta upgrades enabled at some point which is probably not the best idea in retrospect. I'm assuming the only way out of this is to re-flash the firmware? I saw these instructions for re-flashing, but don't have a USB to serial converter around. I do have a Raspberry Pi though, just don't know if I can use it to flash ESP32 or how to do it. Do you have any pointers? |
We could try "Horizontal", but my guess is that the Homeassistant internal link from SWING_HORIZONTAL to a display name and a icon is not yet made. |
@Floyd-Schramm thanks! I have older version of the board and no soldering tools anyway, which is why I'm hoping to get away with some jumper wires and a RPI. Just need to figure out how... Update: I figured it out! Re-flashing worked. Shared the details here: #530 (comment). |
@Floyd-Schramm: Maybe try
"Comfort" appears to be an operation mode in HAss, which actually somewhat matches how it works on the remote (mutually exclusive with powerful mode, disregards vertical swing config). |
Daikin hardware
Indoor Unit FTXA25C2V1BB, Faikin connected to free S21 port, original WiFi module BRP069C4x installed
Is your feature request related to a problem? Please describe.
After integrating the Faikin modules in Home Assistant I noticed that the swing modes are not translated correctly. As seen in the below screenshots, the modes available are named "H" instead of "Horizontal" for example. This difference to the HA default swing also causes the problem that there is no icon available for the swing mode "H".
Describe the solution you'd like
A better, Home Assistant specific, integration using the default swing mode values described in the official HA docs
SWING_OFF
,SWING_VERTICAL
,SWING_HORIZONTAL
,SWING_BOTH
.I am not sure how to integrate the comfort mode as there is no default value in HA, but all other standard Daikin swing modes should be implemented as stated above.
Describe alternatives you've considered
Live with the current situation, as renaming or overwriting within HA is too much trouble :D
The text was updated successfully, but these errors were encountered: