aldas / modbus-tcp-client

PHP client for Modbus TCP and Modbus RTU over TCP (can be used for serial)
Apache License 2.0
191 stars 55 forks source link

Allow unit ID to be in range of 0-255. #160

Closed aldas closed 6 months ago

aldas commented 6 months ago

Older MODBUS specification limited unit ID (slave id) in range of 0-247). See "Modicon Modbus Protocol Reference Guide PI–MBUS–300 Rev. J" page 19. https://modbus.org/docs/PI_MBUS_300.pdf

In newer spec that limitation is removed.

Fixes #159

codecov[bot] commented 6 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 98.96%. Comparing base (6704b45) to head (d4a7345). Report is 1 commits behind head on master.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #160 +/- ## ========================================= Coverage 98.96% 98.96% Complexity 936 936 ========================================= Files 73 73 Lines 2426 2426 ========================================= Hits 2401 2401 Misses 25 25 ``` | [Flag](https://app.codecov.io/gh/aldas/modbus-tcp-client/pull/160/flags?src=pr&el=flags&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Martti+T.) | Coverage Δ | | |---|---|---| | [unittests](https://app.codecov.io/gh/aldas/modbus-tcp-client/pull/160/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Martti+T.) | `98.96% <100.00%> (ø)` | | Flags with carried forward coverage won't be shown. [Click here](https://docs.codecov.io/docs/carryforward-flags?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Martti+T.#carryforward-flags-in-the-pull-request-comment) to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.