openlvc / disco

Java library for Distributed Interactive Simulation
11 stars 3 forks source link

Update to disco to be more compliant with FOM module loading #68

Open jarrodgreene opened 5 months ago

jarrodgreene commented 5 months ago

Disco was previously throwing an exception if two FOM modules defined the same parameter for a class. However according to the spec, this is fine, as long as the definition is the same. This now allows duplicate parameters.

NOTE: currently assuming that parameters with the same name ARE the same, which is not necessarily the case.

jarrodgreene commented 5 months ago

@michaelrfraser these are the updates for getting the VR Forces FOM modules to load. I haven't tested outside of checking CNR starts up - so probably don't want to actually merge this (at least not yet) - just putting it in a PR so you can see the changes