search

mpepping / solarman-mqtt

Send Solarman Smart PV inverter metrics to MQTT
Apache License 2.0
61 stars 21 forks source link

Suggestion: HA add-on #99

Open austinmroczek opened 9 months ago

austinmroczek commented 9 months ago

Has anyone made this into a Hass OS add-on ? https://developers.home-assistant.io/docs/add-ons/. Seems like you're 90% of the way there since you have a working container.

I'm willing to figure it out and submit a PR. Just wanted to ask first in case someone had already done it. I'm not seeing it, but may be using the wrong search terms.

austinmroczek commented 9 months ago

I'm going to play around with this at https://github.com/austinmroczek/addon-solarman-mqtt, following https://github.com/home-assistant/addons-example

austinmroczek commented 8 months ago

I have a working prototype at https://github.com/austinmroczek/addon-solarman-mqtt if anyone wants to give it a try.

mpepping commented 8 months ago

Nice work, @austinmroczek! I was experimenting with this as well.

My thought was, when this becomes an add-on, then the MQTT part becomes obsolete, as it is possible to create/update native HA sensors directly. So, where MQTT now is used to get the Solarman data in HA, an add-on should provide a more native way of doing the same.

austinmroczek commented 8 months ago

We could make it into an HA Integration, but then we would need to know how to handle various kinds of Solarman logger equipment. Or we would have to make integrations for each type of equipment. For example, I'm doing this to read data from my NeoVolta battery. I think for at least the near future, we'll want to keep MQTT and have users create template sensors like you currently have set up.

austinmroczek commented 8 months ago

I think there is a lot we could do to simplify setup for users. All of those curl calls can probably be put into python and we could find stationId and other settings automatically.