victronenergy / venus

Victron Energy Unix/Linux OS
https://github.com/victronenergy/venus/wiki
574 stars 70 forks source link

Lynx parallel BMS: Do not aggregate BMSes with different nominal voltage levels. #1345

Open ReinvdZee opened 3 days ago

ReinvdZee commented 3 days ago

The Lynx smart BMS paralleling feature automatically aggregates ALL Lynx BMSes with the same N2K device instance. This also means that BMSes managing totally different battery banks will be aggregated if they have the same N2K device instance. This leads to confusion when updating Venus OS from a version without the feature to a version with the feature, as two independent BMSes on different voltage levels will suddenly be aggregated.

The Lynx parallel BMS feature should be changed such that BMSes with different (nominal) voltage levels are not aggregated, even if they have the same N2K instance.

nmbath commented 2 days ago

While I think this is a great idea. I think it will lead to further challenges and issues.

It is working around how a system should be configured and that will then give further issues. I have a system that has a 12v and 24v (*2) BMS devices. It is annoying, but the configuration of the device prioriyu is easy.

Consideration should be taken into what else would fail and other challenges integrating an environment. It would be better to just pop a message advising the user configures the system correctly and changes on of the BMS devices to a different data instance.

ReinvdZee commented 2 days ago

@nmbath, Would there be any case in which you'd want to aggregate BMSes of different nominal voltage levels? What challenges and issues do you see here?

nmbath commented 2 days ago

@ReinvdZee my view there is a broader issue. VenusOS is really designed around a single voltage. So from a DVCC perspective you can only control one battery voltage. From a display in seeing what is happening is also really designed for single voltage. Therefore, I would expect users with multi bank voltages to go down a route of a single VenusOS device per voltage. This is what I have done.

If a single VenusOS device is to be considered then at a minimum the DVCC needs to operate on all voltages being monitored.

As an example I have two OrionXS devices charging my 12v bank. Using a single VenusOS I can't control them as well as the devices on the 24v bank.

The UI component is a second order issue as it does allow multiple devices to be seen and monitored and the overview screen becomes a what is the system as a whole doing. However, once you go to two VenusOS devices any VE.Can connected devices (Such as CAN connected MPPTs and NMEA 2k out from the other VenusOS device) a mass of extra devices materialise and confusion rains.

As an example my 12v VenusOS devices sees the 24v MultiPlus from N2K out on the other Cerbo and wants to start controlling it !

A lot of thought is needed on the design and operation of multi voltage systems. I hope this has given some useful insights.

ReinvdZee commented 2 days ago

I understand that controlling two different battery banks with DVCC is currently not possible. Nevertheless, I'm told that there are quite some systems which have e.g. a main 24V bank (controlled by DVCC) and a secondary 12V bank for small appliances. Currently, when users update such a system from an older venus OS version to a version with the paralleling feature, the unrelated battery banks are suddenly combined into one.

AFAIK, there are no cases in which you'd want to combine banks of different voltage levels into 1, so we might as well prevent that. @nmbath I agree that we should think about how to handle different battery banks with respect to DVCC, but that is a much broader issue than this.