-
Notifications
You must be signed in to change notification settings - Fork 23
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[BUG] Opc.Ua.Sample/TestData not working #266
Comments
please test with this PR #267 |
Thanks for the quick PR.
Nothing is imported |
I've tested with https://github.com/OPCFoundation/UA-.NETStandard-Samples/blob/master/Samples/Opc.Ua.Sample/TestData/TestData.NodeSet2.xml, you are using a xml with an old schema which is not supported by the nodesetloader. |
Hi, I think would be better to add an error message or something to warn users about unsupported schema. I tried NodeSet2.xml but I get the following error:
Thanks |
have you built the open62541 with full namespace 0? |
I think was compiled with the reduced namespace. Is nodeset-loader working with the full configuration only? |
seems that the TestData.NodeSet2.xml has references to nodes from namespace 0, which are not compiled in with the reduced configuration. |
In this case would be good to log the problematic node without aborting the whole import operation. Or at least have this option. Another improvement can be about the logging, because the LogLevel 0 is not part of the UA_LogLevel enum, I have to use this code:
But I think that logging at LogLevel 100 instead than 0 should be okay and easier to use. |
I'm gonna test with the full namespace 0 today |
I can't compile with the full namespace, i get this exception from the branch 1.4: Exception: Reference ns=1;i=15001<--[ns=0;i=47]--ns=0;i=11715 has an unknown target note: reduced namespace is compiling fine
|
Now that the FULL ns0 is compiling I tried again the PR. TestData.NodeSet2.xml is now imported, with some errors:
The strange thing for these nodes is the namespace, the xml file is mentioning ns=1 while the warning is mentioning ns=2. Example:
|
Please see open62541/open62541#6867
The text was updated successfully, but these errors were encountered: