plugwise / plugwise-beta

Custom-component / development version of the Plugwise Home Assistant platform
MIT License
24 stars 8 forks source link

Legacy Anna: climate entity missing #49

Closed CoMPaTech closed 4 years ago

CoMPaTech commented 4 years ago

Hi Gents I am currently running 0.2.0 on HA on version 219. Got everything running. But I dont get the climate entity?

I do get 2 entities called "Anna" and "Smile_Anna" but they only contain information. ( i.e. sensors )

image

Originally posted by @AussiSG in https://github.com/laetificat/haanna/issues/19#issuecomment-620732495

bouwew commented 4 years ago

@AussiSG, please respond in this topic.

bouwew commented 4 years ago

@AussiSG, do you see any Error related to plugwise-beta in Developer Tools --> Logs?

If not, can you please activate debug-logging for plugwise-beta?:

logger:
  default: warn
  logs:
    custom_components.plugwise-beta: debug

And restart HA after this change?

After the restart, at Developer Tools --> Logs, load the FULL HA LOG. There should be many plugwise-beta related messages, can you please show them to us?

AussiSG commented 4 years ago

Yes, I have update to the beta version of HA (Home Assistant 0.109.0b4).

Here is a full copy of the HA log:

https://pastebin.com/nMLqeHpX

Dont mind the error of the serial / usb thingy still struggling with hehe

bouwew commented 4 years ago

Thanks! I can see what went wrong: Single master thermostat = None. Should be True in your system as you have a legacy Anna, correct?

AussiSG commented 4 years ago

Correct

CoMPaTech commented 4 years ago

(possible) Code improvements underway

CoMPaTech commented 4 years ago

We're deliberating, my PR is an improvement but probably won't fix it. Can you check http://192.168.x.y/core/locations.xml on your Smile? It's probably pretty empty like (this one)[https://github.com/plugwise/Plugwise-Smile/blob/master/tests/legacy_anna/core.locations.xml]

The second question is what firmware are you running according to the interface? Is this 1.8 or a 3.x or 4.x?

We're readying the PR anyway and will be dropping a pre-release here - if you use HACS make sure you tick the dots right top of the integration and enable 'beta'.

CoMPaTech commented 4 years ago

@AussiSG besides the above request (mainly which firmware you are on according to you app or Smile itself) - 0.2.1a1 should be visible from HACS - please leave your logging set to debug, below Gateway is <some long alphanumeric string> there should be some relevant information.

The debug logging might be stating 1.8.0, 3.3.9 or something but we're really looking for what your app tells you.

AussiSG commented 4 years ago

Goodmorning.

Anna is currently running the following software/hardware, found on the internal website image

Complete log after restart HA: https://pastebin.com/svaEYDUv

And

Here is snip of my /locations.xml image Looks like it isnt there?

CoMPaTech commented 4 years ago

My bad, its http://w.x.y.z/core/locations (without the xml - but it still should be empty). We'll check your pastebin, it still says it didn't find a master thermostat even though we already should have 1.8 fw working.

AussiSG commented 4 years ago

image That has some more content indeed.

Yea thats wat I spotted aswell in the logs

CoMPaTech commented 4 years ago

Well this explains a lot (now) ... the assumption on test input was 1.8 didn't have locations, so if it has some of the logic is now bypassed. Seeing your output I assume (new one) that the other 1.8 didn't have outdoor_temp monitored and hence got no location.

Tnx, 'll get back to you

CoMPaTech commented 4 years ago

0.2.1a2 should be ready to install

AussiSG commented 4 years ago

Gonna try it out. Altho the outdoor temp sensor dont make sense... It always states that it's 21.01 degree outside.....

AussiSG commented 4 years ago

This is more like it 👍 Great sorting guys. Thanks for the support of the extreme legacy anna hehe

image

Edit: The additional data however is not there: image

CoMPaTech commented 4 years ago

Great to see it working - in retro, maybe the 1.8 indeed shouldn't have locations but yours just got it anyway ... Leaving the issue open so @bouwew is also informed

bouwew commented 4 years ago

@AussiSG Can you please post the log-output of the last HA start, with 0.2.1a2 loaded?

AussiSG commented 4 years ago

Yea I have been messing with different Anna gateways because of getting a busted one from Marketplace.....

But : after deleting the integrations and looking it back up the additional sensor seem to be coming in 👍

bouwew commented 4 years ago

Ah Ok, remnant devices... Indeed then better to delete the integration and add it again. What do you see now? Can you post the Integration-screen showing the 2 main devices? And then per device the attached entities? So we can double-check everything is OK?

AussiSG commented 4 years ago

Sure thing. Here are the devices: image

And here some detailed info of the added entities: image image

AussiSG commented 4 years ago

Having a closer look on the entities. I see the preset_mode state "null" is back. I think we had this sorted in the version 0.1.18

bouwew commented 4 years ago

Please post the log-output of the last start showing the plugwise-beta debug messages, there should be a clue for us why it's null.

AussiSG commented 4 years ago

Yes a fresh one after a fresh restart:

https://pastebin.com/7ZpE250M

bouwew commented 4 years ago

I see in your output: active-preset : None, this is causing the null-results.

It would help us enormously if you could capture the XML-data from your Anna/Smile, see here: https://github.com/plugwise/Plugwise-Smile/tree/master/tests how to do it, scroll down to see the README-text. Next, put the files in a zip-file and add it to a new message here.

AussiSG commented 4 years ago

Hmm I have picked up all the request XML files. But I cant find any of the mentioned variables that need to be replaced due to privacy. I hope I am doing it in the right way?

AussiSG commented 4 years ago

Here is the zip file with the data. I hope its good for you. I didnt find any privacy related stuff in it strangely enough Anna.zip

CoMPaTech commented 4 years ago

I don't see any details in there, I think the 1.8 fw doesn't have anything like smile-id's in there. Added your files to our testsetup, but will continue tonight on the auxiliary stuff unless @bouwew already want to tinker with it. The testfiles have been added to a PR in the module repo, tnx!

bouwew commented 4 years ago

@AussiSG Can you please test v0.2.1a3? This update should solve the preset_mode showing "null". Instead it should show "none" when no preset is selected. Can you also activate a preset to see if that works too?

AussiSG commented 4 years ago

Yes , just updated and check the presets they all seem to be working:

image

CoMPaTech commented 4 years ago

💯

AussiSG commented 4 years ago

Thanks for the superb work guys to also make this possible on my 'old' Anna 👍 👍

bouwew commented 4 years ago

@AussiSG Can you disable the preset? Then preset_mode should shown "none" instead of null. When you report this result back, then the final issue is fixed :)

AussiSG commented 4 years ago

Si :D image

bouwew commented 4 years ago

Nice! Thanks for your confirmation. Please close this issue as your final action :)

AussiSG commented 4 years ago

Love too ;-)

Edit:

Wait I didnt open it :D @CoMPaTech you may close it ;)

bouwew commented 4 years ago

True, I close it then.

bouwew commented 4 years ago

@AussiSG other legacy Anna users report that they can see a tap-icon on the screen when they are tapping hot water. Is this also true for your legacy Anna? Or not? I seem to remember that in your XML-files "domestic_hot_water_state" to be always "on". Then maybe you do not see a tap-icon?

bouwew commented 4 years ago

@AussiSG, please update to version 0.2.4rc0 and let me know if on climate.anna(?) hvac_mode is showing the heating and idle states correctly and if binary_sensor.auxiliary_dhw_state is showing the state of tapping hot water correctly.

AussiSG commented 4 years ago

@bouwew just updated and heating up shows good image And also the DHW works properly: image

indeed I see a warm water tapping icon on my anna.

Good job :+1: :+1:

bouwew commented 4 years ago

@AussiSG, that looks good! Another user pointed me to the heating + hot water tapping case, that case is not correctly shown yet, will try to fix that in v0.2.4rc1.

AussiSG commented 4 years ago

I think both the icons show on the Anna when tapping hot water ( heating symbol and tapping symbol ) But I am not sure. Need me to verify anything in that case?

bouwew commented 4 years ago

Yes, please.

AussiSG commented 4 years ago

Both the icon( heating symbol and tappig symbol ) show on the Anna unit. But the states on the entities show it the proper way in my case image

bouwew commented 4 years ago

The coming update, v0.2.4rc1 should show Mode: Heating and W.W.Afname: on. But, it will depend on the connected cv-ketel whether this case can be properly shown, or not.

AussiSG commented 4 years ago

Alright. Will update you when I have installed 2.4rc1 and check that situation

bouwew commented 4 years ago

@AussiSG version 0.2.4rc1 is available on HACS, please test.

AussiSG commented 4 years ago

Morning, just tested but the HVAC mode did not change to heating. I have an Intergas

bouwew commented 4 years ago

So, version rc0 was ok for you but rc1 is not? Did you wait long enough, it could take as much as 60 seconds before a change is visible on HA?

Update: please look at the climate-entity under Developer Tools -- States directly. I don't know how you have configured the Simple Thermostat Card so the info from that card is less reliable to me :) What does the climate-entity show when you are heating only, tapping hot water only and doing both?

AussiSG commented 4 years ago

Also in version rc0 the HVAC state didn't change to heating when tapping hot water. Yea it took quite a while before the taping mode went from 'off' to ' on'

bouwew commented 4 years ago

Hot water state is shown via binary_sensor.auxiliary_dhw_state. Also, when only tapping hot water only, hvac_mode is supposed to show idle.