victronenergy / node-red-contrib-victron

MIT License
90 stars 18 forks source link

Add an option for native dbus instead of using dbus over TCP #9

Closed pkronstrom closed 5 years ago

pkronstrom commented 5 years ago

Using the dbus over TCP works well on external node-red installations. This extension should connect directly to the system dbus on Venus installations in order to skip the dbus-over-tcp configuration step.

mpvader commented 5 years ago

Note that running node-red external is nice for your development now - only. It will not be a use case later on.

mpvader commented 5 years ago

dbuslistener already has (working) code for this, its here: https://github.com/victronenergy/node-red-contrib-victron/blob/master/src/services/dbus-listener.js#L31

Same functionality is used in the Venus-beta-builds with Signalk: I'm real sure it worked in the project from which you took the dbuslistener.

pkronstrom commented 5 years ago

Changes in https://github.com/victronenergy/node-red-contrib-victron/commit/8b438e9c344cbb8a1bcf02868ddeae29c88171c6