Possible changes...

[lllucius]

I finally decided to take the Home Assistant plung as I needed to make our Maytag dryer more accessible to my wife. While I'm still working out exactly how that's going to be (voice or tactile), the first thing I needed was an integration that could interact with the dryer.

So, when I figured I might try to enhance the dryer support in this project, but I guess I got a little carried away.

It now presents entities for nearly all attributes for any device. Separate predefined classes aren't required. Mind you, I've only been able to test it with a Maytag dryer as that's all I have, but it should be able to handle any of them. I'm sure there will need to be some sort of handling of specific attributes, but so far, the generic handling seems to be adequate.

You can see the carnage at: https://github.com/lllucius/whirlpool-sixth-sense
The HACS integration is at: https://github.com/lllucius/whirlpool_hacs

I have not submitted a PR since the changes were major, but I'd be happy to do so.

Remember, this whole HA thing is new to me (started research mid-December), so be nice. 😄

I've attached some screenshots from HA to show it actually does work, though I'm SURE there's critters lurking. I'll weed those out as I get further into actually using it.

[pavandave]

Looks awesome! It's been on my to-do list for a while, just never got around to it. I'll try to spin this up my Oven and let you know!

Got it up and it seems to be adding a lot of different sensors, etc. I will dive into the code a bit later to see what is going on.

Ref:

[lllucius]

I do see that the cavity "name" need to be added to the entites like you did in your mods. Will ponder that.

Actually, it's a simple matter of just changing it in the strings.json file. Any changes there are preserved when additional models are added.

[pavandave]

Just pulled the update and looks good. Let me know if you want something tested in more detail. As I improve my integrations, I'll see if any enhancements can be made. There may be some that don't make sense to keep enabled by default, but that's my only commentary. Will try to pull and help polish (for the Oven part).

[lllucius]

If you run across attributes that you don't think should be presented to the user, you can pass the attribute names to the "setup_entities()" using the optional "exclude=[]" argument. There's a few that I should probably add myself.

[lllucius]

So, now there's 3 ways to tame the number of entities presented to the user:

The original "exclude=" arg on the setup_entities() call, which bypasses presenting the attribute to HA completely. Might need this if we find an attribute that could be harmful to the device/user. (Don't know what...maybe firmware updates or something).

And now each entity type (Button, Sensor, ...) can now specify whether the entity should be disabled and/or hidden by default.

[abmantis]

I think it is ok to add all of them, but disabled by default.

[abmantis]

Nice work @lllucius !

I will be more than happy to help you getting those changes merged and into HA!
Could you split it into small PRs so we can iterate over them?

Also, check #61 since it is related to some of your changes.

PS: I've moved this from an Issue to a Discussion.

[abmantis]

Not sure what you would prefer, but maybe starting with the split of WasherDryer into two different classes (and removing WasherDryer)

[pavandave]

I think I've see WasherDryer as a single class before because some devices can serve both functions. Not sure if that's a good reason to keep it together, but some side commentary.

[lllucius]

I did split it up, but left the combined version there since there really is (or was) a combo/all-in-one unit, from Whirlpool at least. But, it might be that if one of those pops up, then 2 separate devices are presented instead of a combo. Would LOVE to get the attributes and datamode from one of those fellas.

[abmantis]

There has been some discussion around that here: #61

[abmantis]

By the way, I found it nice that you even included the diagnostics platform on the integration! 👏