open62541 / open62541-nodeset-loader

Library for loading opc ua nodesets from xml and sorting the nodes
Mozilla Public License 2.0
27 stars 23 forks source link

Added SecondChance algorithm to allow you to retry importing nodes th… #247

Closed xydan83 closed 11 months ago

xydan83 commented 11 months ago

…at could not be imported because they did not find their parents. This may be the case if we have a node type of Object which has its own parent node type of Variable. For example, this could be a HasHistoricalConfiguration reference type. Also, the status about how many nodes we have imported shows numbers that include nodes that were not imported due to some error.

matkonnerth commented 11 months ago

Great job, thanks for the PR. I've added a comment, feel free to ask questions.

xydan83 commented 11 months ago

Hello! I changed some details and included the container in the addNodeImpl function. NL_Node no longer has StatusCode. Look here please.

xydan83 commented 11 months ago

WindowsBuildOnly is using an old version of open62541 (open62541/1.3.1) and we are getting an error.

image

xydan83 commented 11 months ago

Added a couple of tests.

matkonnerth commented 11 months ago

Thank you, good job!