search

tolwi / hassio-ecoflow-cloud

EcoFlow Cloud Integration for Home Assistant
284 stars 48 forks source link

Slave battery not showing #172

Closed sirdir1972 closed 5 months ago

sirdir1972 commented 7 months ago

Hi! I don’t get any info for my slave battery. As I’ve seen the same problem with another implementation, maybe you’re expecting a different path where the data is to be found. In my case it’s ‘bms_slave_bmsSlaveStatus_1’ on a Delta 2 Max.

sirdir1972 commented 7 months ago

I found out the slave battery is completely commented out… so no wonder why it doesn't show the correct data. Anyways, after putting the path I mentioned in the last message in, it works. Also, there's another important value:

LevelSensorEntity(client, "bms_bmsStatus.soh", const.SOH),

State of health.

tolwi commented 5 months ago

@sirdir1972 attach the diagnostic dump please

sirdir1972 commented 5 months ago

OK, but keep in mind I changed your integration. I removed the commenting out of the slave battery and added lines for SOH config_entry-ecoflow_cloud-f3bdfc7fd62690ba42d13db36784d4f4.json.txt

tolwi commented 5 months ago

well, maybe you can create a PR with your changes?

sirdir1972 commented 5 months ago

PR? push? Sorry I'm not that much into github.

sirdir1972 commented 5 months ago

Here are the files I changed. the slave battery isn't commented out anymore, so if you want to use it, take that in mind. I updated the changes for the slave batteries that were missing (missing comma, volt instead of millivolts etc) changes.zip