Skip to content
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

How to create an instance node the 'right' way? #2783

Open
2 of 5 tasks
larws opened this issue Oct 2, 2024 · 2 comments
Open
2 of 5 tasks

How to create an instance node the 'right' way? #2783

larws opened this issue Oct 2, 2024 · 2 comments
Assignees

Comments

@larws
Copy link

larws commented Oct 2, 2024

Type of issue

  • Bug
  • Enhancement
  • Compliance
  • Question
  • Help wanted

Current Behavior

I am currently trying to achieve the following. I am writing an OPC UA server application with UA-.NETStandard SDK and am loading my type nodes from a nodeset file. So far so good. Now during start up and runtime I would like to create instance nodes from my type nodes depending on the state of my server application.

I am using a node manager dervied from Opc.Ua.Server.CustomNodeManager2 and found the public method CreateNode(ServerSystemContext context, NodeId parentId, NodeId referenceTypeId, QualifiedName browseName, BaseInstanceState instance) which sounds like what I would like to achieve.

My code basically looks like this:

BaseObjectState newDevice = new BaseObjectState(null)
{
    NodeId = new NodeId(101010, index),
    BrowseName = new QualifiedName("Demo Device", index),
    DisplayName = new Opc.Ua.LocalizedText("Demo Device"),
    TypeDefinitionId = deviceTypeId,
};

// this raises an ArgumentNullException
CreateNode(SystemContext, parentId, ReferenceTypeIds.Organizes, newDevice.BrowseName, newDevice);

What I am observing is a ArgumentNullException within AddPredefinedNode() where the node is assigned to the dictionary. m_predefinedNodes[activeNode.NodeId] = activeNode; This is caused by the fact that the New() method of the node manager is the standard implementation, meaning I am not overriding that behavior.

What strikes me as odd is that this doesn't work out of the box. What am I missing here? Any help would be highly appreciated.

On another note, what is the differences between NodeState.Create and CustomNodeManager2.CreateNode are they meant to solve the same problem or do the target different scenarios?

Expected Behavior

Using an instance of CustomNodeManager2 should work out of the box without having to override more functionality than expected.

Steps To Reproduce

No response

Environment

- OS:
- Environment:
- Runtime:
- Nuget Version:
- Component:
- Server:
- Client:

Anything else?

No response

@ThomasNehring
Copy link
Contributor

Hello larws,

sorry for the delay in responding. From what you have written it is not clear to me how you defined the types for which you want to create the instances. It would be easier if you could post a more complete description of what you have done.
If you want to use you own types you have to provide classes which can represent these types at runtime - do you have these in place?

Typically this is done with the help of the model compiler which generates the types, which then have to be loaded into the server. This is done, for example, in the TestData subfolder in the Quickstarts.Servers project which is used in the Console Reference Server. Unfortunately in that example the instances are also defined through the model, so you cannot just proceed as in that case.

(I have to admit I don't know what the CreateNode method in the CustomNodeManager2 is supposed to do - it is used only in three places, as it seems).

Can you please provide some more details on what you have in place already?

BR, Thomas

@larws
Copy link
Author

larws commented Nov 14, 2024

Hi @ThomasNehring,
I was trying to avoid using the generated code from the model compiler and do the following instead.

  • Import all the type nodes required for my server through a Nodeset, using UANodeSet.Import and then adding the resulting NodeStateCollection to the address space.
  • Create instances of my loaded types by calling CreateNode and thus filling my address space with the needed instances.

From what I have found out so far the currently best supported way is the usage of the model compiler. In my opinion is unfortunate since the generated code is not compatible with the current versions of .net.

Am I wrong and there is another way to achieve what I am trying to do without using the model compiler?

Thanks in advance.
BR, Stephan

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants