Skip to content

Commit ec1c118

Browse files
acmorrowurfeex
andauthored
Apply suggestions from code review
Co-authored-by: Felix Exner <feex@universal-robots.com>
1 parent c0f380c commit ec1c118

2 files changed

Lines changed: 5 additions & 5 deletions

File tree

include/ur_client_library/control/reverse_interface.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -53,7 +53,7 @@ enum class TrajectoryControlMessage : int32_t
5353
TRAJECTORY_START = 1, ///< Represents command to start a new finite trajectory of declared length.
5454
TRAJECTORY_STREAM_START = 2, ///< Represents command to start an open-ended streaming trajectory.
5555
TRAJECTORY_STREAM_END = 3, ///< Represents command to terminate an open-ended streaming trajectory.
56-
///< The point_number argument must equal the total number of spline points
56+
///< The point_number argument must equal the total number of segments
5757
///< the caller wrote on the trajectory socket since TRAJECTORY_STREAM_START.
5858
};
5959

resources/external_control.urscript

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1198,11 +1198,11 @@ while control_mode > MODE_STOPPED:
11981198
kill thread_trajectory
11991199
# The streaming session, if any, has been forcibly ended; clear
12001200
# the flag so a subsequent stray STREAM_END is a no-op.
1201-
if trajectory_streaming:
1202-
# In streaming mode, a cancel should come with the total count, just like a STREAM_END.
1203-
trajectory_points_left = params_mult[3] - (STREAMING_SENTINEL - trajectory_points_left)
1201+
if trajectory_streaming:
1202+
# In streaming mode, a cancel should come with the total count, just like a STREAM_END.
1203+
trajectory_points_left = params_mult[3] - (STREAMING_SENTINEL - trajectory_points_left)
12041204
trajectory_streaming = False
1205-
end
1205+
end
12061206
request_trajectory_cleanup()
12071207
stopj(STOPJ_ACCELERATION)
12081208
wait_for_trajectory_cleanup()

0 commit comments

Comments
 (0)