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

[BUG] [FEATURE] Command confirmation for switches ModbusRTU and RTUoverTCP #1178

Open
maufc opened this issue May 4, 2024 · 1 comment
Open

Comments

@maufc
Copy link

maufc commented May 4, 2024

Describe the bug
Modbus RTU and RTUoverTCP switches should wait for the response (confirmation) in order to confirm the new state of the switch.
The switch button should wait the response of the device. At this moment, if the command was not executed, the states changes until a new regular requests is make.
I found also that the commands can me send at the same time the regular requests are made making crossing modbus queries to the device.
The problem origin could come from this problem: #1148 waiting for a response.

To Reproduce
Just define a tag (coil status) of a modbus rtu connection and assign it as a property to a switch.
Go to Lab, and try to change the coil status with the switch.
See the app log and the port log.

Screenshots

Captura desde 2024-05-04 12-17-19

Captura desde 2024-05-04 12-18-35

Environment

  • FUXA version: V.1.1.19-1587
  • Node.js version: 16.16.0
  • npm version: 8.11.0
  • Platform/OS: Ubuntu 22.04
  • Browser: Firefox 124.0.2 (64-bit)
@rvbatista
Copy link
Contributor

ModbusRTU should be used for serial communication from the server, and RTUoverTCP should be used with serial device using transparent gateway. I have to check if there is some kind of mechanism to handle concurrent requests, but I think that neither Fuxa nor Modbus-serial have this. And it's kind of critical for serial devices.

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

No branches or pull requests

2 participants