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

Modbus TCP implementation #72

Open
ihategoto opened this issue Jul 26, 2021 · 1 comment
Open

Modbus TCP implementation #72

ihategoto opened this issue Jul 26, 2021 · 1 comment

Comments

@ihategoto
Copy link

Hello,
I'm currently writing a Modbus TCP client using Minimalmodbus utilities.
The task turned out to be fairly easy as Minimalmodbus provides all the utilities to create the Modbus PDU;
all I had to do was writing utilities to create the MBAP header and the socket management.
My goal is to integrate the Modbus TCP implementation with the current one.
I've been testing the code that I wrote and it seems to behave properly.
I'm writing here because I noticed that the Modbus TCP implementation is not planned for future releases, and my intention was, as soon as the code is ready, to submit a pull request to merge my fork and the original one. Might that be a good idea?

PS: feel free to contribute to my code.

@pyhys
Copy link
Owner

pyhys commented Aug 8, 2021

Hi, thanks for your interest in MInimalModbus.
It is planned than in the future modify the internal representation of the data, now that Python2 support is dropped. So instead of storing data as strings, the data will be stored as bytes. That is a major rewrite of the internals.
Any TCP functionality needs to be done after that.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants