victronenergy / venus

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

managed batteries: Log blocking status of MOSFETs to VRM #755

Closed izak closed 3 years ago

izak commented 3 years ago

For managed batteries that support it, these paths have been available on dbus and the gui for some time now. Log also to VRM:

/System/NrOfModulesBlockingCharge
/System/NrOfModulesBlockingDischarge
/System/NrOfModulesOffline
/System/NrOfModulesOnline

TODO

izak commented 3 years ago

Implementation of a warning path is in can-bus-bms branch blocking-warnings, but is presently left unmerged until firmware support is fully verified for BYD.

mpvader commented 3 years ago

Changing this to v2.70.

With BYD now confirmed as having those fields not working properly (and us thus removing them from being visible), and also BYD being the only one that we're aware of that even has those fields; its all non-urgent anymore.

mpvader commented 3 years ago

@izak whats the status of this?

all done and can be closed? and whats the status with BYD about this? Do you know?

izak commented 3 years ago

Presently this is explicitly unsupported on BYD batteries, even though the Premium LV sends this information. That's because the information is incorrect in earlier firmware versions.

I tested a new firmware version where it works correctly. I do now know if that firmware is released yet. There were other issues with that firmware too. I will take it up with them and revert here.

mpvader commented 3 years ago

ok but our side works right? And Pylon has official support for this, right? @izak

if yes, please close this

izak commented 3 years ago

Our side works correctly. Pylontech is in a similar boat to BYD: Wrapping up last minute changes. I'll go ahead and close it as this is being handled elsewhere.