HALRobotics / Beta

HAL Robotics Framework beta release and associated documentation.
17 stars 3 forks source link

Notification about speed limit unclear #31

Closed carneaup closed 5 years ago

carneaup commented 6 years ago

Hello,

You'll find attached a zip file with a tool path where I play with the speed of the robot. The file is working and the procedure is solving without error but there is a notification (INFO) about the speed that cannot be reached which is unclear. This message appears also if i don't give any speed setting to the robot.

The same message also appeared on a different file that we actually used on the robot without any problem (it appeared after I updated HAL last time).

Best regards, Paul Speed limit unclear.zip

sebandraos commented 6 years ago

Thank you for your feedback. Could you please explicit what is unclear about the message? I appreciate the second sentence is cut short but the former is complete and implies that we don't believe you can get to the defined speed due to some mechanical or trajectory constraint. It is purely informational so shouldn't have any negative impact on the solving or execution.

carneaup commented 6 years ago

My concern is that this message appears now on all the files I'm working on and since the second sentence is cut short, I don't know what seems to be the problem. In addition it doesn't seem to be affected by the speed settings or acceleration settings. Although this doesn't affect the solving of the procedure and the robot manage to execute the tool path correctly.

On 22 June 2018 at 16:51, Sebastian Andraos notifications@github.com wrote:

Could you please explicit what is unclear about the message? I appreciate the second sentence is cut short but the former is complete and implies that we don't believe you can get to the defined speed due to some mechanical or trajectory constraint. It is purely informational so shouldn't have any negative impact on the solving or execution.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/HALRobotics/Beta/issues/31#issuecomment-399469355, or mute the thread https://github.com/notifications/unsubscribe-auth/AXw3_EXrRKaUA3KbvP_waZZjrgaphtOzks5t_QR9gaJpZM4UzejA .

sebandraos commented 6 years ago

We are continually rewriting our solvers and this message was based on a condition which should have been superseded. You shouldn't receive this error unnecessarily from today's update. Apologies for the confusion

sebandraos commented 6 years ago

It seems I spoke too soon. Certain targets in your demo file are still invoking the message and it doesn't look like the joints are the limiting factor.

tristangobin commented 5 years ago

Fixed in current version. HAL.McNeel.V5 1.2.6-beta