ZWaveJS.NET does not support multiple drivers about zwavejs.net HOT 5 OPEN

@marcus-j-davies , do you want me to take a shot at it, or do you want to do it?

from zwavejs.net.

Ooof!

possible of course, but will need some (quite a bit) of re-work - and thought

From the top of my head

• The Controller class to have an internal Driver reference be set to this within the following code block
  

  ZWaveJS.NET/Visual Studio Projects/ZWaveJS.NET/ZWaveJS.NET/Driver.cs

  Line 769 in 6c3db91

  {

Example

Controller C = JO.SelectToken("result.state.controller").ToObject<Controller>();
C._Driver = this

• Any reference inside the controller to Driver.Instance be pointed at it

• Each ZWaveNode/Endpoint class will also need a reference (somehow) - They are instanciated directly from the init JSON presently (we could use the Node collection somehow)

• Various static properties on the driver class to be made instance only.

  • SchemaVersionID
  • ServerCommunicationPort
  • ServerErrorThrottleTime

  These are used (I think) during Driver class instance creation, so will need to be optional params on the ctor's

• Various static properties in the server class be made instance only

  • this includes static methods

  This means having a separate Server class instance for each Driver instance

There is a lot that surrounds the static approach presently, unless I'm over thinking it 🤔

from zwavejs.net.

Hi @spudwebb

I am not one to reject free labour! 😉
Please by all means have a crack at it! 👍

Thanks

from zwavejs.net.

Is this still open or resolved? I see the latest available is '22 still (from Visual Studio NuGet)

from zwavejs.net.

Hi, this issue is still open, as v4 has not yet been published.

it’s fixed in the v4 branch.
See the PR : #31

from zwavejs.net.