Master::ping Function should have a longer timeout when using Broadcast ID

ROBOTIS-GIT / Dynamixel2Arduino

DYNAMIXEL protocol library for Arduino

Apache License 2.0

88 stars 55 forks source link

Master::ping Function should have a longer timeout when using Broadcast ID #106

Closed LegacyMecha closed 2 years ago

LegacyMecha commented 2 years ago

When using broadcast ID, if there's no dynamixel ID within #1 or #2 IDs, the ping seems to fail (tested on 2mb baud). So if I have ID #1 and #5, it seems to works fine, but if I only have ID #5 if fails to find it. I ran same ping with a 50ms timeout and it worked. I think the default timeout should be increased correctly for broadcast ID so we don't have to manually set it when using Broadcast ID on ping.

ROBOTIS-Will commented 2 years ago

@LegacyMecha Thank you for reporting the issue. I'll look into this!

ROBOTIS-Will commented 2 years ago

@LegacyMecha A delay of 3ms per each ID (therefore, total 3*253 = 759ms) timeout will be applied when running the ping with broadcast ID. Please let me know if this fix works fine.