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

NonAtomicRead/Write Definition in AccessLevelEx is wrong #410

Closed
coconut147 opened this issue Mar 21, 2018 · 5 comments
Closed

NonAtomicRead/Write Definition in AccessLevelEx is wrong #410

coconut147 opened this issue Mar 21, 2018 · 5 comments
Assignees
@AlinMoldovean
Labels
bug A bug was identified and should be fixed.

Comments

@coconut147
Copy link

According to Part 3 of UA Spec 1.04 the bits 8 & 9 define the "NonatomicRead/Write" Flags.

In https://raw.githubusercontent.com/OPCFoundation/UA-.NETStandard/0894364ca94d20f298e8480cb871fd76c2190ba6/Stack/Opc.Ua.Core/Stack/Generated/Opc.Ua.DataTypes.cs

The enum AccessLevelExType is wrong and sets

NonAtomicRead to 65536 which is bit 16 (should be 8)
NonAtomicWrite to 131072 which is bit 17 (should be 9)
@coconut147 coconut147 mentioned this issue Mar 21, 2018
Closed
@mregen
Copy link
Contributor

mregen commented Mar 28, 2018

Hi @coconut147, indeed, we need to open an issue in the Nodeset repo and Mantis.

@mregen mregen mentioned this issue Mar 28, 2018
Closed
@mregen mregen added the bug A bug was identified and should be fixed. label Mar 28, 2018
@mregen
Copy link
Contributor

mregen commented Jun 7, 2018

fixed in OPCFoundation/UA-ModelCompiler@a9699ab, would be fixed here in a nodeset update

@coconut147
Copy link
Author

Any time schedule for nodeset updates - or am I able to do this as a pull request!?

@AlinMoldovean AlinMoldovean self-assigned this Jun 14, 2018
@AlinMoldovean
Copy link
Contributor

We intend to provide this update for the nodeset files soon.

@AlinMoldovean
Copy link
Contributor

Fixed in Commit 00bf7b1

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@AlinMoldovean AlinMoldovean

Labels
bug A bug was identified and should be fixed.
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@mregen @AlinMoldovean @coconut147