SkyFi fails on init and update #29
Assignees
Labels
SkyFi
Devices using the SkyFi class
Comments
|
just did the Update to 9.1 these are the only logs mentioning daikin |
|
|
Thanks for the logs. These logs are just saying that the unit is slow (there is a fix for this too ;)). Try increasing the loglevel to |
|
@kingy444 do you have time to look at this? |
|
I just tried the update to 9.1 and got these logs |
Looks like another issue.. |
|
I've created a new branch (https://github.com/fredrike/pydaikin/tree/skyfi) where I've added even more logging. Please test with the branch |
fredrike
changed the title
|
hi fredrike sorry for my ignorance, i the above debugging from hassio, ive followed steps login, then done the below.. but get [\e]0;\u@\h:\w\a]\W $ docker exec -it homeassistant pip install --force --no-deps git+https://github.com/fredrike/pydaikin.git@master |
|
please see debug level logs (i hope this is correct) |
|
Hi fredrike, I too am stuck with the "Debugging-from-hassio" procedure. After logging in and installing your script I get Is there something else I need to configure or install? |
|
Thanks for the log, this will be really useful. I'll review the instructions on running this code.. |
|
The instructions works on my machine, a few things, you must have missed the login-part. And you have not changed the branch from I'll upgrade my HassOS instance to 2024.9.2 and see if it still works. |
|
How do we change the branch? |
|
Yep, the newest version of HA also works: Are you running the right Teminal plugin? I guess you have to disable 
|
You run this command (note the skyfi at the end) |
|
i just tried the above and recived same error, i'm not sure what im missing heres the flow ive done |
|
I get exactly the same result. If I then exit and log back in I am straight back to [\e]0;\u@\h:\w\a]\W $. |
|
Strange that you don't get it to work. Did you disable protection mode? |
Protection mode is off, I also tried uninstalling and reinstalling ssh to see if that fixed but it didn’t unfortunately |
|
I've updated to ha core 2024.9.2, set logging to debug, and installed the pydaikin@skyfi branch, here are the logs so far: |
Fredrike, I upgraded to 2024.9.2 and the device would not start. I uninstalled the Daikin integration, but cannot get it going again under 2024.9.2. |
|
Mine is call good - all issues resolved.Running 2024.9.2
On Wednesday 18 September 2024 at 10:48:55 am AEST, RonFac ***@***.***> wrote:
@ciaran4, @RonFac can you test if you can get the new changes in skyfi working?
Fredrike, I upgraded to 2024.9.2 and the device would not start. I uninstalled the Daikin integration, but cannot get it going again under 2024.9.2.
I have decided to purchase a Airbase unit (BRP 15B61) and will see how that goes. Part of the reason is that Daikin no longer support remote (out of home) control for the SkyFi (BRP 15A61) on the phone apps. Will keep you posted on how the Airbase works.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.Message ID: ***@***.***>
|
|
I carried out some further testing with the SkyFi under 2024.9.2 This time, I ran the HA server on WiFi, as does the SkyFi unit. The logs indicate that when pydaikin is calling on the SkyFi unit with an http call, the response is failing around 50% of the time. It looks like there is a timing problem between the SkyFi and HA.... This problem did not exist in versions before and including 2024.7.4 If I run the url ( |
|
For what it is worth I have just been pinging the skyfi controller and have found there is great variation in the response times. Average times ranged from 2ms to 50ms. I also noticed that the controller seems to have a sleep mode and sometimes doesn't respond. |
When we get the SkyFi support back you should not need a new controller, why not just control everything from HA ;)
We have already concluded that the SkyFi is an old and quite low performing module in other threads (perhaps in the HA-forum). I've build a patch in the |
Wow, this really looks promising, no critical errors from pydaikin anymore. It looks like if you don't run in debug-mode you don't see any issues at all (except sensor-update being slow which has been solved in another branch). @jahmai can you send some more logs when changing zone-status and modes (turn off and switch heat->cool) too. |
Merged
|
@fredrike Here are the logs of me flicking through all the modes and zones: Everything seems to be working. HA did add new entities for the Daikin device though which I had to go and delete the old ones and rename the new ones to the old entity id's but wasn't too hard. EDIT: I should say I updated to the latest commit off the skyfi branch before doing the above. |
|
@fredrike i finally managed the debugging from hassio steps to work, i put sudo in front of and it worked so |
|
these are logs after installing skyfi branch |
GOLDEN! 🥇 I would say that we have a winner! I'll merge the code and update HA. |
|
Here is the PR on HomeAssistant: home-assistant/core#126219 |
|
I have finally managed to get this working, albeit only for one unit (i have two) but thats not unexpected one has always had intermittent issues connecting, skyfi is an absolute disgrace of a product by daikin and should never have existed, but thats another story. It took me a while to get working, if this helps someone else, in the end what worked was, update to 9.2, update pydaikin to 2.13.7, delete old main unit entities, rename the new entities to old name, restart. Thanks for working on this @fredrike! The reason i even originally got home assistant 4 years ago was to find a solution to get around how rubbish skyfi was. |
You are welcome, I do take donations via https://buymeacoffee.com/fredrike
You don't need to do anything to revert these changes, HA will overwrite them automatically. The changes are scheduled for the 2024.9.3 release that is due September 20. |
|
Not only is the integration working, but the second unit has now come online with no action from me. In previous versions, if it failed at startup the only method to get it working would be a restart. Excellent work! |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
It seems like SkyFi uses the wrong
parse_response, this code indaikin_base._get_resourcepydaikin/pydaikin/daikin_base.py
Line 180 in df68cd2
should call
pydaikin/pydaikin/daikin_skyfi.py
Lines 92 to 105 in 5b062bf
We need testcases for this.
The text was updated successfully, but these errors were encountered: