krbaker / hass-sunpower

Home Assistant SunPower Integration using the local installer ethernet interface.
Apache License 2.0
49 stars 22 forks source link

Set various entities with 'diagnostic' entity category? #38

Open kpfleming opened 1 year ago

kpfleming commented 1 year ago

There are quite a few entities which are not measurements that a user wants to monitor normally, they are diagnostic indicators. These include most of the "PVS" entities, but could include others.

I'm happy to send a PR to add entity_category: diagnostic to these entities if that would be welcome. A side effect of this is that these entities would no longer appear in the default dashboard, so users would need to go to Settings -> Devices -> PV Supervisor to see them (or of course they can be explicitly added to a dashboard).

krbaker commented 1 year ago

Happy to take a PR with that, seems very reasonable. A quick read here: https://developers.home-assistant.io/blog/2021/10/26/config-entity/#entity-categories makes it seem like a great addition.