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.

Sign up for GitHub

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

Jump to bottom

DataTypeEncoding object pointed by the "HasEncoding" links should have node class Object instead of ObjectType #1549

Closed
erossignon opened this issue Jan 14, 2024 · 1 comment · Fixed by #1550
Closed

DataTypeEncoding object pointed by the "HasEncoding" links should have node class Object instead of ObjectType #1549

erossignon opened this issue Jan 14, 2024 · 1 comment · Fixed by #1550

Comments

@erossignon
Copy link

erossignon commented Jan 14, 2024 •
edited

Describe the bug

Found while investigating node-opcua/issue1232

opcua-asyncio/asyncua/common/manage_nodes.py

Line 401 in f48b4f5

addnode.NodeClass = ua.NodeClass.ObjectType

should be Object and not ObjecType

To Reproduce

Use example.py

Expected behavior

Screenshots

actual
image
The DataTypeEncoding instance is wrongly set to nodeClass=ObjectType
image

Expected ( like in any other companion specification )
The DataTypeEncoding instance should be nodeClass=Object

image

Version

Python-Version: 3.x
opcua-asyncio: 1.0.6
@schroeder- schroeder- mentioned this issue Jan 14, 2024
Merged
@erossignon erossignon changed the title DataEncoding pointed by the "HasEncoding" links should have node class Object instead of ObjectType DataTypeEncoding object pointed by the "HasEncoding" links should have node class Object instead of ObjectType Jan 14, 2024
@rempAut
Copy link

rempAut commented Feb 16, 2024

Looks like also fixes #1524

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Change Encoding Object to Type schroeder-/opcua-asyncio
2 participants
@erossignon @rempAut