ollo69 / ha-smartthinq-sensors

HomeAssistant custom integration for SmartThinQ LG devices configurable with Lovelace User Interface.
Apache License 2.0
1.16k stars 159 forks source link

Add Washer Detergent Level and Softener Level #494

Open kristofferR opened 1 year ago

kristofferR commented 1 year ago

Is your feature request related to a problem? Please describe. It would be nice if Detergent Level and Softener Level could be added to HA, like in the LG app.

As you can see, a bunch of other things visible in the APP are also missing from the integration (all the ones with OFF in the screenshot), but they are frankly not used by me.

kristofferR commented 1 year ago

From my diagnostics file, it seems like it should be pretty easy to implement.

"ezDetergentState": "EZCSDT_NOT_EMPTY", "ezSoftenerState": "EZCSSO_NOT_EMPTY",

Remove _NOT if it's actually empty.

smartthinq_sensors-431b2feb5c68a76c54ceef50d20aa298-Vaskemaskin-7b687f598e43f08e5a7c75ac870025a5.json.txt

ollo69 commented 1 year ago

As you can see, a bunch of other things visible in the APP are also missing from the integration (all the ones with OFF in the screenshot), but they are frankly not used by me.

All this should be already available, both as attribute and as dedicated sensor. Most of this sensors are just disable by default and must be enabled if you need them.

From my diagnostics file, it seems like it should be pretty easy to implement.

Why also exist a ezLinkDetergentState? You know what does it means? From diagnostic I see that is possible to directly monitor available quantity (ezDetergentAmount and ezSoftenerAmount). It is not better to directly monitor this instead of the state?

kristofferR commented 1 year ago

All this should be already available, both as attribute and as dedicated sensor. Most of this sensors are just disable by default and must be enabled if you need them.

Ah, I see now that they are attributes. Nevermind then :)

Why also exist a ezLinkDetergentState? You know what does it means?

Unfortunately I have no idea. The app doesn't mention anything about Link.

From diagnostic I see that is possible to directly monitor available quantity (ezDetergentAmount and ezSoftenerAmount). It is not better to directly monitor this instead of the state?

Those values seem to be the dosing amount per wash, as the values match the dosing amount set in the app perfectly.

ollo69 commented 1 year ago

This 2 binary sensors (and corresponding attributes) are available in last release. Please remember that those sensors (as many others) are disabled by default, so you must enable them in HomeAssistant device page. Please test new release and provide here your feed-back

kristofferR commented 1 year ago

This 2 binary sensors (and corresponding attributes) are available in last release. Please remember that those sensors (as many others) are disabled by default, so you must enable them in HomeAssistant device page. Please test new release and provide here your feed-back

I just ran out of detergent now, and the detection worked great, but the strings doesn't make much sense. Instead of On/Off it should be Normal/Low.

I think the sensors should probably be called "Detergent/Softener Level" instead of just "Detergent/Softener", as "Detergent: Normal" (or especially "Detergent: On") can be misconstrued into thinking it has to do with the type of detergent or that detergent dispensing is even activated.

nicoodeimos commented 4 months ago

I seem to have the same "issue". Are the two published binary sensors reflecting the "isEmpty" state of the containers?