gvigroux
commented
1 year ago I just did it. Can you please test?
Closed ndom91 closed 1 year ago
gvigroux
commented
1 year ago I just did it. Can you please test?
ndom91
commented
1 year ago Hmm so many of the sensors are no longer available anymore now.
I got a second device added for my 1 washing machine with many fewer sensors:
This is the previous one:
All of these entities are now "no longer provided by the integration" :thinking:
I was previously on commit 83f6ea8
gvigroux
commented
1 year ago Can you please try to delete and reinstall the integration ?
ndom91
commented
1 year ago Deleted and readded the integration. Now I only have the 1 device with the 4 entities:
ndom91
commented
1 year ago 2 minutes later the 4 washing machine sensor values updated:
gvigroux
commented
1 year ago Are more sensors coming after 1 min ? I don't understand why they are not visible.
I've made some fix elsewhere, can you update and try ? If the same, I need to analyse some logs to understand. Please add this to the configuration.yaml and restart HA
logger:
logs:
custom_components.hon: infoAfter go to the log page '/config/logs' and click to access all logs.
Share any of the logs starting with:
[custom_components.hon.hon] {"payload":
Thanks!
ndom91
commented
1 year ago Ah found the issue,
The totalElectricityUsed on hon/sensor.py:129 is a float, not int. Therefore it failed setting up any sensors after that
Got all this after the change:
ndom91
commented
1 year ago In addition, I have htis error message in the logs:
Entity sensor.washing_machine_current_water_used (<class 'custom_components.hon.sensor.HonBaseCurrentWaterUsed'>) is using state class 'measurement' which is impossible considering device class ('volume') it is using; expected None or one of 'total_increasing', 'total'; Please update your configuration if your entity is manually configured, otherwise report it to the custom integration author.
ndom91
commented
1 year ago As well as an issue with the onOffStatus key
ndom91
commented
1 year ago Doesn't look like I have an onOffStatus key. Here's the whole payload:
EDIT:
I copied your code from the binary_sensor.py onOffStatus conditional definition to the sensor.py:601 onOffStatus definition. Basically:
if( "onOffStatus" in self._coordinator.data ):
self._on = self._coordinator.data["onOffStatus"]["parNewVal"] == "1"
else:
self._on = self._coordinator.data["category"] == "CONNECTED"In addition, in sensor.py:317 you're defining time only if the device type is WASH_DRYER. Leading to errors for only WASHING_MACHINE type devices when you refer to it again at the bottom of this callback.
EDIT: Seems that last self._attr_native_value = time is just a left over. Going to comment it out
ndom91
commented
1 year ago Ah works! Middle of a run cycle:
Although startTime is not right, thats the start time of the homeassistant instance (restarted). Looks like that might just be a side-effect of me redefining it in the middle of a washing cycle.
Anyway, sorry about the "stream of consciousness" bug reporting here :sweat_smile:. But after making all the changes I referenced in this thread, we seem to be up and running again. No more errors being reported :+1:
EDIT: Just reread your message, just wanted to be clear - I did not pull your latest update before making these changes. I was still on 0ed1185. So anyway, sorry if you already got around to these fixes :sweat_smile:
ndom91
commented
1 year ago And the actual issue raised in this GH issue is also fixed :joy:
Remember, the water volume use seems to be wrong type of statistic still though (https://github.com/gvigroux/hon/issues/62#issuecomment-1483816119)
ndom91
commented
1 year ago Final question - is there any way to also control the devices through the API endpoints used for this integration?
Because logging in via the integration seems to kick me out of the android app, meaning I have to signin freshly again every time I want to start a cycle. Slightly annoying. Would be great if I could stay signed in only via hass and also initiate wash cycles via the integration.
gvigroux
commented
1 year ago Wow :) Can you create a pull request with all the changes? If not I will try and ask you test.
There is a service to start a washing program I think. Did you try?
ndom91
commented
1 year ago @gvigroux yeah no problem. Here we go :grin: - https://github.com/gvigroux/hon/pull/64.
Looks like the float/int var type for totalElectricityUsed was already fixed, right.
Ah I see there service now! Is there a possibility to set programType instead of manually specifying water temperature, spin cycles, etc.?
EDIT: Yeah looks like you can pass an id theoretically. I often use prCode: 136, for example. And can easily be observed by starting the desired program once from the app and checking the appropriate entity in HA.
https://github.com/gvigroux/hon/blob/f0a944373fa7df8cb543ed4030d7d64eb48d96f0/__init__.py#L183
EDIT 2: I'll post this as a "feature request" and someone can maybe get to it when they have time :wink: :+1:
EDIT 3: Feature request issue is up: https://github.com/gvigroux/hon/issues/65
gvigroux
commented
1 year ago I did the merge :) Thank you!
So first of all, the latest update is great! I finally got all available fields from my washingmachine!
However, when trying to add electricity usage to the built-in HA energy dashboard, the sensor is not available.
They link to a statistics docs page for developers:
Looks like you have to set the sensor
state_classto one ofmeasurement,totalortotal_increasing. I guess this one would betotal, right?Referring to this sensor: