victronenergy / node-red-contrib-victron

MIT License
90 stars 18 forks source link

solar charger node needs updating for mppt rs #79

Closed mpvader closed 2 years ago

mpvader commented 3 years ago

The mppt rs has multiple trackers, and such multiple paths with PV information.

https://github.com/victronenergy/venus-private/issues/135

dirkjanfaber commented 2 years ago

Merged via https://github.com/victronenergy/node-red-contrib-victron/pull/88.

mpvader commented 2 years ago

Hi @dirkjanfaber : sorry,, but you need to remove /Pv/I and its similar paths, since we'll also be removing those from the D-Bus as well.

The PV array related paths to support in node-red are: '/Pv/0/V' '/Pv/1/V' '/Pv/2/V' '/Pv/3/V' '/Pv/0/P' '/Pv/1/P' '/Pv/2/P' '/Pv/3/P'

and: `'/Pv/V'

and: `'/Yield/Power'

I've added it also to the docs: https://github.com/victronenergy/venus/wiki/dbus#solar-chargers

And where did you learn about "2": "Error", being a valid option for /Mode? Its new to me that thats possible, so want to make sure that actually exists.

mpvader commented 2 years ago

ps. above change is in progress in Venus OS at the moment. Thiemo finished it for vedirect-interface yesterday. And installed that manually on this system: https://vrm.victronenergy.com/installation/55013/dashboard. So if you want to see it, login there.

dirkjanfaber commented 2 years ago

Adjusted the services.json file to accommodate for the d-bus changes.