InfluxDB: parallel banks are receiving first bank's voltages

[jeronimo]

diyBMSv4Code Release-2021-03-29-15-35
InfluxDB shell version: 1.6.4

[stuartpittaway]

Oh, that looks like an issue. I'm quite busy at the moment so may take a few days to investigate.

[jeronimo]

Just wrote a quick script to send data from Victron Smartshunt via Ruby to InfluxDB and have started to look at diybms code as well. It seems that in

diyBMSv4Code/ESPController/src/main.cpp

Line 725 in 87f6de1

poststring = poststring + "cells," + "cell=" + String(bank) + "_" + String(i) + " v=" + String((float)cmi[i].voltagemV / 1000.0, 3) + ",i=" + String(cmi[i].internalTemp) + "i" + ",e=" + String(cmi[i].externalTemp) + "i" + ",b=" + (cmi[i].inBypass ? String("true") : String("false")) + "\n";

array indexes should be i+bank though not sure if cmi array already has all the values.
If it is not a quick fix I will make a PR at some point :)

[stuartpittaway]

Hello @jeronimo just checking that you are reporting this issue against the ESP8266 version of the DIYBMS controller?

This issue was resolved in the ESP32 version.

[jeronimo]

Yes, I am still using old controller with Wemos D1 Mini ESP8266.

[rolfbartels]

Just confirming that this issue still exists

[stuartpittaway]

Although @rolfbartels is using the ESP32 version which has a different code base

[rolfbartels]

Sorry, just to calrify I am using the old controller with the Wemos D1 Mini ESP8266

[stuartpittaway]

Ah, in which case that's a whole different ball game!

[stuartpittaway]

@rolfbartels just updated the code, do you want to try that one out?

[rolfbartels]

@stuartpittaway I can confirm, this has fixed the error, thank you