laetificat / haanna

Plugwise Anna API to use in conjunction with Home Assistant.
MIT License
5 stars 5 forks source link

Additional sensor info from Anna/plugwise #19

Closed AussiSG closed 5 years ago

AussiSG commented 5 years ago

Missing the extra info which could be read out from Anna/plugwise.

Regarding: Actual Boiler temperature. Actual flux rating measured by thermostat. Actual pressure in system.

The needed to pull info is posted on following page. https://community.home-assistant.io/t/advice-how-to-load-sensor-info-from-plugwise-anna-thermostat-web-interface-xml/87801?page=6

bouwew commented 5 years ago

I've updated haanna to include getting the three requested values.

I quickly added getting the values to my private code: image Looks like it's working.

How to make sensors out of these values is another matter, I could use some guidance on this topic :)

bouwew commented 5 years ago

Implemented and released to Pypi.

AussiSG commented 5 years ago

Thank you will check them later this week :D Don't think it will be possible to really use the info to trigger your boiler/CV because Anna is already trying to do that.

I more use it to check if pressure is below 1,8 Bar and use illumination info to switch some lighting

bouwew commented 5 years ago

You're welcome. Yes, please do no test the changes today, we're still fixing some other bugs.

bouwew commented 5 years ago

On Pypi, haanna is updated to version 0.12.3, the requested changes have been implemented in this version.

AussiSG commented 5 years ago

Hi Bouwew, Thanks for the update. I am running the plugwise component on Home automation ( raspberry pi ). I have tried adding you extra line of codes in the climate.py ( looks similar to your haanna) but it doesn't seem to do the trick. Am I doing it wrong. Kind of a python newbie ;-)

bouwew commented 5 years ago

Just replace all the code in haanna.py and climate.py files. You can get the clean code via the Raw- button.

AussiSG commented 5 years ago

Should I be having both the haanna.py and the climate.py files in the custom_component folder? Thought only one should do that trick? Sorry for asking ( so much )

bouwew commented 5 years ago

First you will need to update haanna, this can be done by changing the number in manifest.json. to 0.12.3. Then update the contents of climate.py with the code in the anna-ha repository.

AussiSG commented 5 years ago

Thank you ! Clear thing will give it a go tomorrow. Will let you know.

bouwew commented 5 years ago

Some quick advice: install the Visual Studio Code add-on from the ADD-ON STORE. With this editor it is very easy to edit your HA files.

And here you can find the clean code to put in the climate.py file: https://raw.githubusercontent.com/laetificat/anna-ha/master/custom_components/plugwise_dev/climate.py

Just copy and paste it after deleting to existing code. Or, if you want to play it safe, make a backup of the existing file/code. Can also be done via the Visual Studio Code editor, it has some file management functions.

AussiSG commented 5 years ago

Have updated the climate.py code with the code you linked above. And also changed the number inside the .json file as followed:

{
  "domain": "plugwise_dev",
  "name": "Plugwise Development Anna",
  "documentation": "https://github.com/laetificat/anna-ha",
  "dependencies": [],
  "codeowners": ["@laetificat","CoMPaTech", "@bouwew"],
  "requirements": ["haanna==0.12.3"]
}

but when I look in the develepment tools -> states part. I dont see the extra information. Do I need to trigger something else. Also did a restart of HASS after changing the code ;-)

bouwew commented 5 years ago

That's strange, it works for me and CoMPaTech. Try reloading the States page. What type of HA installation are you using? Can you check in some way if haanna was updated to the required version?

bouwew commented 5 years ago

Also, any information in the Development Tools - Log?

CoMPaTech commented 5 years ago

Not sure, but make sure you only load the dev plugwise and not (also) the actual one that is inside HA If you do that I think that will override back to the earlier haanna

Op 11 okt. 2019, om 22:28 heeft Bouwe Westerdijk notifications@github.com het volgende geschreven:

Also, any information in the Log?

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/laetificat/haanna/issues/19?email_source=notifications&email_token=ABL4CME6XE4IKTWI5P67NATQODOYRA5CNFSM4I4M3YPKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEBBEBIQ#issuecomment-541212834, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABL4CMAWK257LVE74WSLOJDQODOYRANCNFSM4I4M3YPA.

AussiSG commented 5 years ago

The only entry I can find in the log is:

2019-10-12 08:46:26 INFO (SyncWorker_16) [homeassistant.loader] Loaded plugwise from homeassistant.components.plugwise 2019-10-12 08:46:30 INFO (MainThread) [homeassistant.components.climate] Setting up climate.

I have added the following code in the config file to enable the logging but it feels wrong because of not getting any hehe

climate.plugwise: debug

To enable the logging but it feels wrong because of not getting any hehe

bouwew commented 5 years ago

I guess you have configured the Plugwise component that is a part of HA. You can disable that one and enable the Plugwise_dev component by making a change in configuration.yaml: Change

climate:
  - platform: plugwise

To

climate:
  - platform: plugwise_dev

And then restart HA.

bouwew commented 5 years ago

Also, you directory must be named: .../custom_components/plugwise_dev

AussiSG commented 5 years ago

Alright have made another step forward.... :D Now get the following warning

2019-10-12 12:15:46 WARNING (MainThread) [homeassistant.loader] You are using a custom integration for plugwise_dev which has not been tested by Home Assistant. This component might cause stability problems, be sure to disable it if you do experience issues with Home Assistant.

But sadly enough I dont see the extra info. I have also updated the climate.py and .json file fully by downloading and putting them in /custom_components/plugwise_dev

bouwew commented 5 years ago

The warning is normal, so you are indeed making progress.

Can you post an image of what is shown on the States-page? On Windows press Shift-PrintSc, open Paint, Crtl-V, then select the part of the screen you want to show here, Ctrl-C, etc.

bouwew commented 5 years ago

Also, you have one climate.plugwise_dev entity in the States-list? And no climate.plugwise anymore?

AussiSG commented 5 years ago

Yeah only have the DEV one as entity.

image

bouwew commented 5 years ago

Ok, looks like there's more missing, what is the firmware-version of your Anna?

AussiSG commented 5 years ago

Its running on firmware version 1.8.20

bouwew commented 5 years ago

Ah, Ok, that's why. I'm running on 3.1.7 and the additions are written and tested for the 3.x firmware. But, I'm happy to try and get it to work for your version. Can you send me a complete capture of the Anna-xml-data? Please browse to "your-anna-ip"/core/domain_objects, click with your mouse-cursor in the text that is shown, press Crtl-a, Ctrl-c and then paste (Ctrl-v) the data for instance in a pastebin.com page and send me the link.

AussiSG commented 5 years ago

Ah serious. I have read some stuff about that on the HA forum.... :( Firmware version is from 2017-03-13 and HW version = 6539-1302-5002.

hope it worked. This is the link: https://pastebin.com/510RVMTN

bouwew commented 5 years ago

Great, thanks!

Please change the requirements-line in manifest.json from this: "haanna==0.12.3" to this: "https://github.com/bouwew/haanna/archive/master.zip#haanna==0.12.4"

Then restart HA and let me know whether the 3 new parameters are being shown.

AussiSG commented 5 years ago

Wauw quick progress! seems to have done the trick

image

bouwew commented 5 years ago

Great!

I see some other stuff that is not working, let me make some more changes...

bouwew commented 5 years ago

Ok, ready. Please change in the requirements-line in manifest.json this: 0.12.4 to 0.12.5. Change only the number, leave the rest as is.

Then restart HA and show me the result :)

bouwew commented 5 years ago

Some more fixes, please change the number to 0.12.7, restart HA and share the result, please.

AussiSG commented 5 years ago

Hereby the results: image

Dont know if I see any differences ;-)

bouwew commented 5 years ago

I do! Much more is working now: I can see you are running in auto-mode(= schedule is active), the available havc_modes are shown, I can see that you are tapping hot water (dhw), the available preset_modes are now correctly shown.

The only thing left to fix is the schema-stuff. I know from another legacy Anna user that you can have only one schema(=schedule). I can create several schedules and can switch between them.

My state looks like this: image

AussiSG commented 5 years ago

Ahaa nice to know. Didnt know there was a newer version avaible of the anna. ( or the gateway ) Do you know how you can tell the difference between a newier or oldier version?

Thats why your firmware is quite higher

Indeed I can only have one time schedule for the whole week. I do have different modes ( Vacation mode , sleep mode , etc )

bouwew commented 5 years ago

Are you using the climate card? image

Can you please test some more? Turn off the automation by clicking on the flame icon? Does that work? Can you turn the automation back on by clicking the schedule icon? Can you active a preset? Can you manually change temperature? Please note, wait a little while after each change for the card to update.

bouwew commented 5 years ago

I don't think you can see any difference in the hardwares. But the difference in firmware-version makes it clear

bouwew commented 5 years ago

In your attribute-list the presets are shown correctly via preset_mode but they show up again at available_schemas. And in my attribute-list there is an extra key: selected_schema, I will try to make that work also for your Anna.

AussiSG commented 5 years ago

Strangest thing is that when I try to update it states that it is up to date. So there much be some difference somewhere hehe.

Might be that plugwise has released a newer version. Of which year is your hardware version if you check it on your smile gateway site.

Currently I am at my parents house so can't check your question but will try tomorrow

bouwew commented 5 years ago

That's not strange, your hardware probably can't run the new firmware. Or maybe it can but there is a commercial reason then :)

Ok, no problem, looking forward to hear whether the control-functions work for you .

bouwew commented 5 years ago

One more question: the single weekschedule that is available on your Anna, does it have a fixed name or can you rename it? Or delete it and create a new one with a different name?

bouwew commented 5 years ago

Never mind my last question, please update the version number to 0.12.8, restart HA and let me know the result. I think I have fixed the detection of the schedule.

bouwew commented 5 years ago

Please update the version number to 0.12.9, restart HA and show me the state. You should no longer see available schema's but there should be a selected schema key instead. And the schedule-name should be shown on the climate-card.

AussiSG commented 5 years ago

Had some time to check the climate card. in version 0.12.9 it looks as followed image and I get the following message when I press the scheme button. image

AussiSG commented 5 years ago

The Entity looks as followed now image

AussiSG commented 5 years ago

Also the HVAC action nows says DHW. But the system is inactive at the moment. No hot water or usage of warm water for heating

bouwew commented 5 years ago

Hmm, that didn't work, I'll have a look.

bouwew commented 5 years ago

Please put version number 0.12.10 in manifest,json, restart HA and let me know the state-result.

Now I will look into the dhw issue.

AussiSG commented 5 years ago

Alright gonna give it a go in a minute.

PS thank you already for the much effort. And don't forget it's Saturday eve right ;-)

bouwew commented 5 years ago

Not sure why dhw is always on, could be a bug. I will switch it to always off for legacy Anna. I will do that tomorrow, when you have shared the results of 0.12.10. Enjoy your saturday-evening :)

AussiSG commented 5 years ago

Seems that the update gives the following error :D

2019-10-12 22:29:50 ERROR (MainThread) [homeassistant.helpers.entity] Update for climate.plugwise_development_thermostat fails
Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/helpers/entity.py", line 261, in async_update_ha_state
    await self.async_device_update()
  File "/usr/src/homeassistant/homeassistant/helpers/entity.py", line 441, in async_device_update
    await self.hass.async_add_executor_job(self.update)
  File "/usr/local/lib/python3.7/concurrent/futures/thread.py", line 57, in run
    result = self.fn(*self.args, **self.kwargs)
  File "/config/custom_components/plugwise_dev/climate.py", line 348, in update
    self._domain_objects
  File "/usr/local/lib/python3.7/site-packages/haanna/haanna.py", line 165, in get_active_schema_name
    if self.get_schema_state(self, root):
TypeError: get_schema_state() takes 2 positional arguments but 3 were given