tolwi / hassio-ecoflow-cloud

EcoFlow Cloud Integration for Home Assistant
359 stars 63 forks source link

Solar In Volts and In Amps gone? #273

Closed shimfish closed 3 days ago

shimfish commented 1 month ago

I've just noticed that these entities are no longer available on my Delta 2 Max integration.

Did they go in the latest update or did EF stop providing them?

foxthefox commented 1 month ago

Since the new version >1.0 is using the official API, it might not get redistributed. I would guess that the DM2 still sends it. I have the feeling that the official API is only delivering a few of the available datapoints. On the other hand for DM2 at least the documentation says that in is there mppt.inVolt, mppt.inAmp.

Nid01 commented 1 month ago

@shimfish I checked the file changes of delta2_max.py, but can't find anything which shows that such data existed. Can you provide me more details so I can take another look?

shimfish commented 1 month ago

@Nid01 Not sure what I can provide you with.

All I can say is that when I first set it up, it gave me solar in volts and solar in amps for both mppt inputs and now it says "not provided". My history shows a graph for those values and then one day it just stops.

image
shimfish commented 1 month ago

So I went through my backup files and found this in my version of delta2_max.py

           InWattsSensorEntity(client, "mppt.inWatts", const.SOLAR_1_IN_POWER),
            InWattsSensorEntity(client, "mppt.pv2InWatts", const.SOLAR_2_IN_POWER),
            InMilliVoltSensorEntity(client, "mppt.inVol", const.SOLAR_1_IN_VOLTS),
            InMilliVoltSensorEntity(client, "mppt.pv2InVol", const.SOLAR_2_IN_VOLTS),
            InAmpSensorEntity(client, "mppt.inAmp", const.SOLAR_1_IN_AMPS),
            InAmpSensorEntity(client, "mppt.pv2InAmp", const.SOLAR_2_IN_AMPS),

I think I must have added them and then completely forgotten about it. I don't even know how I managed to work out that I could add them in the first place.

Anyway, um, if anyone feels like merging that in to the plugin, er, I guess that might be handy for people.

shimfish commented 3 days ago

Have finally got it together to submit a PR for this change.