stefanor / signalk-victron-ble

Simple Signal K Node server plugin to read data from Victron devices using Instant Data over BLE
Apache License 2.0
10 stars 3 forks source link

extend SolarCharger output #6

Closed humppafreak closed 1 year ago

humppafreak commented 1 year ago

although panelPower is non-spec (panelVoltage and panelCurrent are...), it's really helpful to see at a glance how well the panel is performing.

loadCurrent is in spec: https://signalk.org/specification/1.7.0/doc/vesselsBranch.html#vesselsregexpelectricalsolarregexploadcurrent and there's no reason not to populate the path if we have the data.

Fixes: #4

stefanor commented 1 year ago

It seems signalk-venus-plugin uses panelPower too. Filed SignalK/specification#653 to add it.

stefanor commented 1 year ago

We could add yieldToday too (as requested in #4)

humppafreak commented 1 year ago

I don't see a reason why we shouldn't add it. If the data is there, just make it available.

I just didn't know what I would do with the yieldToday data when I made this PR, that's why it's not been included.