-
Notifications
You must be signed in to change notification settings - Fork 854
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
Connectors/opcua/high load #1501
Conversation
Refactored OPC-UA connector
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looks good to me. the PR contains all improvements discussed.
The following error is through It's related to this kind of messages it worked in the current master branch BuildInfo(ProductUri='http://xxxxxxxx.com/xxxxxxx/v3.1.6.0', ManufacturerName='xxxxxx', ProductName='xxxxxxx', SoftwareVersion='3.1.6.0', BuildNumber='3.1.6', BuildDate=datetime.datetime(2024, 7, 8, 9, 1, 2, 607366, tzinfo=datetime.timezone.utc)) Here is the config
|
We found another types: 2024-08-14 15:04:55 - |ERROR| - [tb_gateway_service.py] - tb_gateway_service - __send_to_storage - 1103 - Object of type datetime is not JSON serializable Range corresponds with this sample value: [Range(Low=0.0, High=0.0)] |
Started the GW with both connectors disabled. This was the output
Enabled both connectors and here is the output:
We also detected in TB that configuration was lost in the first connector "MB107_UDC1" but not in the second one |
build docker script added. requirement for asyncua added
…essing of non primitive types
docker python12 and cleanup
Steps followed and their outputs:
Full log
|
Full log
|
Steps followed:
|
I got many errors but basically these 3:
|
@ashdam do you have logs like: "Unsupported data type: ..., will be processed as a string."? |
@samson0v no, just those errors over and over on the log |
@ashdam added additional log because we can't understand what object type we got in the converter. |
@samson0v got this error stack from this commit ed0fe1e
|
They seem all asyncua OPCUA DataTypes isn't it? |
@ashdam I reproduced situtation with config cutting. |
…, optimized polling and scanning methods calling
…tion parameter, hardcoded for now to be 1MB
…ling (removed disconnection if connector is not stopped), minor fixes in logger
This PR contains optimization for data reading from the OPC-UA server.
read_values
methodawait sleep(.05)