search

Azure / iot-edge-modbus

Modbus protocol module for use with the Azure IoT Edge
Other
90 stars 71 forks source link

Polling interval #18

Closed thegunshow closed 6 years ago

thegunshow commented 6 years ago

This seems to have no effect on the module. Jus reads as fast as possible.

Would be good to timestamp the output.

Thanks

stephenctw commented 6 years ago

Hi, Polling interval works on my machine. Could you provide your configuration?

Also, there is timestamp in the output.

{
    "DisplayName":"Voltage",
    "HwId":"PowerMeter-0a:01:01:01:01:01",
    "Address":"400001",
    "Value":"19775",
    "SourceTimestamp":"2017-11-17 08:43:50"
  }

Thanks.

surabhshah commented 6 years ago

Hi @thegunshow, are you using the docker image and setting the latest configuration? If so, then you need to provide the older key for "PollingInterval" i.e. simply "Interval" in the configuration to see the changes, because I guess the public docker image is not yet updated with latest changes. In the meanwhile, you could probably build the source and create a custom docker image and use it.

Thanks, Surabh

thegunshow commented 6 years ago

Hi Surabh

I am using the public image so this probably is the problem, I will build the image from source and see what happens.

Maybe microsoft can update the public image as well?

Thanks Brendan

On Wed, Dec 13, 2017 at 12:46 PM, Surabh Shah notifications@github.com wrote:

Hi @thegunshow https://github.com/thegunshow, are you using the docker image and setting the latest configuration? If so, then you need to provide the older key for "PollingInterval" i.e. simply "Interval" in the configuration to see the changes, because I guess the public docker image is not yet updated with latest changes. In the meanwhile, you could probably build the source and create a custom docker image and use it.

Thanks, Surabh

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/Azure/iot-edge-modbus/issues/18#issuecomment-351354323, or mute the thread https://github.com/notifications/unsubscribe-auth/APV4f0gG1XbEloLdybFrsHRG_S-MaK34ks5s_6r2gaJpZM4Q_ksc .

-- Kind regards Brendan Gunn

stephenctw commented 6 years ago

Yes, @surabhshah is correct. I will update the public image soon.

stephenctw commented 6 years ago

Try to use microsoft/azureiotedge-modbus-tcp:1.0-rtu-preview. This version has Polling Interval working properly and also Modbus RTU functionality.

thegunshow commented 6 years ago

Thanks, I will give it a try.

On Thu, Dec 14, 2017 at 12:06 PM, Stephen notifications@github.com wrote:

Try to use microsoft/azureiotedge-modbus-tcp:1.0-rtu-preview. This version has Polling Interval working properly and also Modbus RTU functionality.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/Azure/iot-edge-modbus/issues/18#issuecomment-351665724, or mute the thread https://github.com/notifications/unsubscribe-auth/APV4f1Z-YSsxcMh4I6pfxt2uQk477OOiks5tAPM6gaJpZM4Q_ksc .

-- Kind regards Brendan Gunn