victronenergy / node-red-contrib-victron

MIT License
90 stars 18 forks source link

Node to read the "Active SoC Limit" (not the minimum SOC) #75

Closed IoTPlay closed 2 years ago

IoTPlay commented 3 years ago

Hi, does a node exist to read the Active State of Charge limit?

Screenshot 2021-04-30 at 10 35 39

mpvader commented 3 years ago

The solution for this is to add it to the ESS-output node. For which I think we already have an issue or todo item; assigning to dirkjan.

dirkjanfaber commented 3 years ago

Within the Venus gui, the Active SOC limit is only shown if the Mode is set to Optimized (with BatteryLife).

The question is if we want to replicate the same logic here or just always have Active SOC limit as a selectable input and let the user decide if it contains actual useful information. Always showing options is easier to implement and requires less maintaining, but it probably will lead to some questions about the validity of the information.

dirkjanfaber commented 3 years ago

On the faberd/bms-75 branch, I've added a selectable property for the path object, allowing to set the option to be selectable or not. This results in an unselectable grayed out option to be shown if the Mode is set to something else than Optimized (with BatteryLife):
image

mpvader commented 3 years ago

Nice! So apparently node-red itself already supports this for properties? Or is all of that configuration window UI defined in our own code?

dirkjanfaber commented 3 years ago

It is all defined in our own code. (The buildEditPanel function in src/nodes/victron-nodes.html to be precise)