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 https://github.com/stuartpittaway/diyBMSv4Code/blob/87f6de125b10ed8b0e9e4830e15a79ecb33127d8/ESPController/src/main.cpp#L725 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