search

alengwenus / ha-sma-ev-charger

SMA EV Charger integration for Home Assistant
Apache License 2.0
18 stars 2 forks source link

Request to add additional features #2

Closed Xrlwyzz closed 11 months ago

Xrlwyzz commented 1 year ago

I just installed it and everything seems to be running great. Many thanks for this integration, that is a lot better than building everything manually!

The first ideas for improvement came up already: Can you please also add the sensor for "Status of connected vehicle" Measurement.Operation.EVeh.Health? That would be great to use for some automations! And also, the parameter "Initiate device restart" Parameter.Sys.DevRstr might be helpful sometimes.

alengwenus commented 1 year ago

I added the vehicle status and the charging station status in 1.0.5. Restarting the device would need a service call. But why would we ever have to restart the device?

Xrlwyzz commented 1 year ago

Well, unfortunately I have to restart it fairly often. After a while, the EV Charger has hundreds of communication errors a day, which sometimes also stops charging for a few minutes (not shifting between one and three phases). Nobody found the error so far, but restarting helps… There might be other reasons for a restart as well.

alengwenus commented 1 year ago

I have to think about how to implement the restart function properly. The problem is, that if the device is restarted, the integration has to re-authenticate. At the moment this would mean that the integration needs to be reloaded. That's not easy to automize unless you completely restart HomeAssistant. Let's see what's possible.

Xrlwyzz commented 12 months ago

As long as that is only my request and nobody else needs it, please do not waste your energy on this issue. I did not expect it to be more compliacated than all the other settings. But your explanation makes sense. I can help myself by just flicking the fuse as well, like I did before. Not everything has to be automated :-)

alengwenus commented 12 months ago

I implemented a device action to restart the device. Re-authentication should also work as intended. It needs some re-work as I am not very satisfied with the current implementation. But it should work. The new release is 1.0.6.

You can trigger the action in any automation by selecting "device" (in the actions tab), select your SMA EV Charger device and select the "Initiate restart" action.

Xrlwyzz commented 12 months ago

Thanks a lot!

Another issue that I just realized: The sensor.wallbox_charging_station_meter_reading is displayed like this: image That is actually 7045.821 kWh, but interpreted by HA like 7045821.00 kWh. So it is off by factor 1000. For the sensor.wallbox_charging_session_energy, however, everything is fine - since that has Wh as unit.

alengwenus commented 12 months ago

Thanks! Should be fixed in 1.0.7.