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

Major rewrite + improvements in many areas #8

Open
wants to merge 191 commits into
base: master
Choose a base branch
from

Conversation

mbs38
Copy link

@mbs38 mbs38 commented Oct 22, 2018

Hi,
first off: Of course I do not expect this to be merged.

My little rewrite has brought some improvements:

  • more abstraction when writing to coils/registers using mqtt. Writing is now possible without having to know slave id, reference, function code etc.
  • specific coils/registers can be made read only
  • multiple slave devices on one bus are now fully supported
  • polling speed has been increased sgnificantly. With modbus RTU @ 38400 baud more than 80 transactions per second have been achieved.
  • switched over to pymodbus which is in active development

I've made so many changes that compatibility is broken BUT my changes might be very useful considering the issues of modbus2mqtt.

So if you're interested: https://github.com/mbs38/spicierModbus2mqtt

But beware: this is a beta or maybe even an alpha version Can be considered stable now.

mbs38 and others added 30 commits October 22, 2018 14:45
phe95 and others added 30 commits October 4, 2022 14:06
Replace failing MQTT host iot.eclipse.org with working host mqtt.eclipseprojects.io
Implemented signed Int32-values
* Ignore the root warning for the install

* fixed docker build issue

* ensure correct indent

* using plain pip again

* added changelog entrry

Co-authored-by: Stefan Fischer <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.