-
Notifications
You must be signed in to change notification settings - Fork 0
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
Improve addmethodnode #36
Conversation
The failing testset works on [email protected] ([email protected]), but not with 1.3.11 - weird. And obviously not with 0.2.0 and 1.4.x. I am suspecting that the problem is in the upstream library. Will investigate (and post an issue there), but likely mark these tests as broken right now. |
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #36 +/- ##
==========================================
+ Coverage 42.28% 45.33% +3.04%
==========================================
Files 13 13
Lines 4377 4601 +224
==========================================
+ Hits 1851 2086 +235
+ Misses 2526 2515 -11 ☔ View full report in Codecov by Sentry. |
…h/open62541.jl into improve-addmethodnode
Ok, all issues worked around; should be ready for review @martinkosch |
This introduces high level interface functions for convenient adding and calling of method nodes.
Short summary below: