search

victronenergy / venus

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

Trigger GX alarm relay also on Lynx Smart BMS warnings #1330

Open ReinvdZee opened 3 months ago

ReinvdZee commented 3 months ago

Currently, the GX alarm relay only triggers on alarms. Should there be an exception for Lynx Smart BMSes (and possibly other devices as well) to trigger the alarm relay on warnings? The purpose of this is to notify the user of low cell voltage before the system shuts down.

And do we then trigger on all warnings? Or only on low cell voltage?

@mpvader @vejpasop

mpvader commented 3 months ago

All warnings of the lynx smart bms & NG. And relay on the gx needs to be triggered as well.

and during implementation pls check what effect this has on mgd batteries like Pylontech

msigman commented 3 months ago

Super happy to see this @ReinvdZee, this is something I have been missing and will be greatly appreciated.

I suspect users would appreciate having a toggle in Settings to enable/disable this feature. If you wanted to get fancy you could make it a dropdown, so the user can choose which relay to trigger for them (None/Relay1/Relay2).

Of course, I don't want to over-complicate it. My personal use case would be to use the default behavior (feature enabled, same relay as other alarms).

I am happy to help test this if/when needed.