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.

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

Protocol binding for Modbus/TCP - Second candidate #194

wants to merge 9 commits into
base: master


Copy link

wiresio commented Mar 20, 2020

Protocol binding for Modbus/TCP - Second candidate

Signed-off-by: Christian Glomb

wiresio added 9 commits Mar 20, 2020

This comment has been minimized.

Copy link
Member Author

wiresio commented Mar 27, 2020

@relu91 - Here is a

Feature comparison

Feature relu91
Encoding of Modbus function New modbus:function and modbus:entity field in form URL
Register number New modbus:range field in form URL
Unit ID modbus:unitID URL
Subscriptions modbus:pollingTime not implemented
Timeout modbus:timeout not implemented
Protocol URL modbus:// modbus+tcp://
Connection pooling TODO yes
Register pooling no yes
Conversion based on DataSchema no yes
Tests yes no
  • Connection pooling: use open connection for multiple requests instead of always opening a new one
  • Register pooling: Merge requests for registers that are next to each other - increases performance
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

1 participant
You can’t perform that action at this time.