-
-
Notifications
You must be signed in to change notification settings - Fork 31.5k
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
Bump pymodbus version to 2.5.0 #48545
Conversation
This works in my environment, but that is a small one. @vzahradnik can you please give this a spin, before approving. Thanks in advance. |
Hey there @adamchengtkc, @vzahradnik, mind taking a look at this pull request as its been labeled with an integration ( |
Thanks for bumping the dependency! However, could you please update the PR description to contain at least one (or multiple) of the following:
This allows us to review upstream changes, which is needed to decide this change is working as intended and/or if we can include it in, for example, a patch release of Home Assistant. Thanks already! 👍 |
Done, I hope I added what you need, otherwise just ask. I would really like the approval from @vzahradnik before merging this, since he has a bigger modbus installation than I will ever get. |
@janiversen I will test your change today. |
Thanks a lot, but no need to stress yourself. With v2.5.0 we can start thinking on trying the async again. In the meantime I am working to reduce the open issues, one by one. |
Please use pyModbus |
Thanks for the advice, however I do not like to upgrade to a release candidate, so I will close this PR and wait until I see 2.5.1 as being stable. |
Breaking change
Proposed change
Bump pymodbus version to 2.5.0 since it now seems stable.
Version 2.5.0 have solved a number of issues (especially error handling) that should make the modbus integration more stable.
Release notes:
https://github.com/riptideio/pymodbus/releases
Compare v2.3.0 to v2.5.0
pymodbus-dev/pymodbus@v2.3.0...v2.5.0
Changelog:
https://github.com/riptideio/pymodbus/blob/master/CHANGELOG.rst
Type of change
Example entry for
configuration.yaml
:# Example configuration.yaml
Additional information
This PR fixes or closes issue: fixes #
fixes Update modbus integration to use pymodbus 2.5.1 when available/stable #41415
This PR is related to issue:
Link to documentation pull request:
Checklist
black --fast homeassistant tests
)If user exposed functionality or configuration variables are added/changed:
If the code communicates with devices, web services, or third-party tools:
Updated and included derived files by running:
python3 -m script.hassfest
.requirements_all.txt
.Updated by running
python3 -m script.gen_requirements_all
..coveragerc
.The integration reached or maintains the following Integration Quality Scale:
To help with the load of incoming pull requests: