OmegaModr / home-assistant-metron-ev

This integration exposes your Metron EV charger local websockets API data as Home Assistant sensors.
MIT License
0 stars 1 forks source link

Additional sensor values #57

Open LeoAlioth opened 3 weeks ago

LeoAlioth commented 3 weeks ago

Checklist

Is your feature request related to a problem? Please describe.

/

Describe the solution you'd like

I see that the integration already parses most of the data that the EVSE provides, but it does not expose all of it to the home assistant as a sensor. Specifically, i am missing total home power and total solar power, to have an easy way to use the measurements for showing whole home consumption data, import and export.

Describe alternatives you've considered

do you plan on adding that, or should i make a fork and add the missing info, and open a pull request? Also, it might be useful to have some of the entities disabled by default, and then have an option in HA to just enable the ones you want to see.

Additional context

/

OmegaModr commented 3 weeks ago

Oh hey, an unexpected visitor :D

I mean I can add it, but not sure when i will find the time - if your comfortable enough with the sloppy code I'll gladly accept a PR.

LeoAlioth commented 3 weeks ago

Well, ive already added the two mentioned manually in the custom components on my install. So it is not a big deal to make a pull request to the repo. I'll first go through the other values and add the rest of the missing ones though.