-
-
Notifications
You must be signed in to change notification settings - Fork 39
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
Add support for "Floor Heating Airflow" #105
base: master
Are you sure you want to change the base?
Conversation
@droscy I get this message when I select the Floorheating (and any other option in the swingmode: |
Hi @rploeg, I cannot replicate your error... Actually I don't have Floor Heating, what I did is taking the PR and replacing Did you restarted Home Assistant after replacing the integration? Could you enable debug log adding the following lines in
Then restart HA and try again. You should see in the log file something like the following when you enable the floor heating
|
Yes, had rebooted. Here the logfile:
So I don't see directly floorheatingairflow that you mention in your example. But above is also the failed service that I see in my interface |
The error originates from horizontal swing (ATTR_HSWING_MODE) that you don't have, indeed your
Have you ever tried to set "Vertical" swing mode without the PR? If you have not, could you try? Then, with PR applied, replace file |
@droscy Sorry didn't had the time to go back to oirginal (time issue), so I have replaced the daikin_base.py with yours. I see this back in the logs (doesnt update):
|
That log is ok, I posted exaclty the same previously (see the end of the post). |
I'll try to replicate your environment and test... what's your HA version? I need your full debug log of the integration, so please:
Thanks |
@rploeg, please, try again the PR with latest commit. |
thanks I can confirm that it is working now. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Code is clean, i've tested it and it seems to work
@rospogrigio can this be merged please? |
Hello, |
@krazar I've included this in my fork, swap to that whilst you wait if you want |
Hi guys I tried out this fork but home assistant would not accept it. For a n00nb, please what code or files should I use to test the floor heating? I recovered the files from the backup to make it work again |
Hi @domerich, if you installed this integration manually you could simply replace every file inside If you face any problems please enable debug log adding the following code to
then restart Home Assistant and send us the new log file (remove sensible data from the log before sending, i.e. lines with |
ok thanks that zip file works for me. For automations the mode cannot be selected from a drop down menu but has to be entered manually but it works. Example
Unfortunatelly it is not possible to start the device with the top flap closed so it always hits my desk. I think I will just have to remove it alltogether. |
I think this is a HA Core limitation isnt it? Are you talking about Swing Mode? If someone can point me at the API docs that show otherwise I'll patch it accordingly |
about the automations I do not know, sometimes you can choose from a drop down menu so I think this is fixable |
Can you try https://github.com/jwillemsen/daikin_residential_altherma/releases/tag/v3.1.0, made some changes to support more swing modes today |
I have created a PR to my forked version to add floor heating airflow, maybe you can test this, see jwillemsen/daikin_onecta#35 |
No description provided.