gopcua / opcua

Native Go OPC-UA library
MIT License
856 stars 259 forks source link

Add ability to access dynamic datatypes #637

Closed joeACV closed 1 year ago

joeACV commented 1 year ago

Some OPC values are data types defined in the server. This will allow you to register the data type value and read or subscribe to it and be encoding into a map[string] interface{} allowing it to be a dynamic data structure closes #588

magiconair commented 1 year ago

I have a similar requirement for this and wrote some internal code which uses reflect to create a type on the fly. So we can check which approach works better.

A couple of comments on the PR:

In general, please provide more rationale and context for the change. Explain the approach. Think about how you would like to see contributions to your code from someone you haven't seen before. I appreciate your effort and want to review it. However, this library is in critical active production use. It runs in a lot of places. Let your code quality reflect that please.

Thank you Frank

joeACV commented 1 year ago

Thanks Frank for the comments. I will close the pull request until I get all of your recommendations completed.