Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Allow switching relay(s) when CCL=0 or DCL=0 sent by managed battery #737

Open
izak opened this issue Nov 23, 2020 · 1 comment
Open

Allow switching relay(s) when CCL=0 or DCL=0 sent by managed battery #737

izak opened this issue Nov 23, 2020 · 1 comment
Assignees

Comments

@izak
Copy link
Collaborator

izak commented Nov 23, 2020

Background: Some batteries are losing the physical allow-to-charge and allow-to-discharge relays. They use only CAN-bus communications for this.

In such systems, where there might be other equipment that still relies on an analog switch to control them.

Solution: Make it possible to switch the relays on the GX device so that these can be used instead.

@izak izak self-assigned this Nov 23, 2020
izak added a commit to victronenergy/dbus-systemcalc-py that referenced this issue Feb 1, 2021
This is for batteries that no longer have their own
dry contacts. CAN-bus signals can be relayed to the
relay(s) of the GX device to signal devices that
only support analog signalling.

victronenergy/venus#737
jhofstee pushed a commit to victronenergy/meta-victronenergy that referenced this issue Feb 5, 2021
Add support for switching relay contacts based on BMS signals
    For batteries that no longer have their own relays, switch the
    GX relay(s) based on CCL=0 or DCL=0 commands on the can-bus.
    victronenergy/venus#737
Support more than two relays (only on dbus, only useful on unofficial platforms such as Raspberry Pi).
@mpvader
Copy link
Contributor

mpvader commented Feb 10, 2021

There are two open questions about this:

  1. As implemented now, this feature only works with a BMS which sends CCL & DCL values. Which is confusing, since the gui now shows the settings always - even if you only have a BMV.
  2. This code depends on CCL being zero, but, we are all the time telling BMS makers that they should never set CCL to 0...

And for my self: why is this feature relevant? Some use cases:

  1. DC-Loads: while AC-Loads are taken of since the inverter or multi will be switched off, DC-loads are not. So for an off-grid system having DC-Loads, you could switch a Smart BatteryProtect with a GX relay, quite nice. about this feature is that (with systems having such BMS-es)
  2. For example switch other charge sources that are not CAN controlled, such as a wind generator or alternator and so forth.

Lastly, thinking about this, there are pure Victron systems that need our attention as well, perhaps better to focus on that first?

  1. A system with a smallBMS (ATC and ATD outputs, no comms port) a GX Device and an MPPT: since the VE.Direct port of the MPPT is connected to the GX Device, there is no way to have that switched on and off by the BMS. Solution: wire the digital outputs of the smallBMS into the digital inputs of the Cerbo GX, and control the MPPT by software.
  2. A system with a VE.Bus BMS (ATC and ATD outputs, and VE.Bus), a GX Device and a MPPT. same problem, but here you could consider using the VE.Bus BMS connection via the inverter/charger, rather than the digital input wiring. But if thats a good idea, I don't know: what happens if the inverter/charger is off. .. deadlock situation etc.

izak added a commit to victronenergy/dbus-systemcalc-py that referenced this issue Mar 15, 2021
jhofstee pushed a commit to victronenergy/meta-victronenergy that referenced this issue Mar 16, 2021
- Remove feature to control relays using BMS signals
    This needs more thought and perhaps should not be done at all.
    victronenergy/venus#737
- Fix SCS bug where adding a VE.Can solarcharger would disable SCS
    Broken since Venus 2.40 when SCS was introduced
    Closes victronenergy/venus#792
- Publish service name of BMS on dbus
mpvader pushed a commit to victronenergy/meta-victronenergy that referenced this issue Mar 17, 2021
Add support for switching relay contacts based on BMS signals
    For batteries that no longer have their own relays, switch the
    GX relay(s) based on CCL=0 or DCL=0 commands on the can-bus.
    victronenergy/venus#737
Support more than two relays (only on dbus, only useful on unofficial platforms such as Raspberry Pi).

(cherry picked from commit 67ce4be)
mpvader pushed a commit to victronenergy/meta-victronenergy that referenced this issue Mar 17, 2021
- Remove feature to control relays using BMS signals
    This needs more thought and perhaps should not be done at all.
    victronenergy/venus#737
- Fix SCS bug where adding a VE.Can solarcharger would disable SCS
    Broken since Venus 2.40 when SCS was introduced
    Closes victronenergy/venus#792
- Publish service name of BMS on dbus

(cherry picked from commit 92da1b8)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants