We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
The related codes are in line 151~162 of https://github.com/stepjam/PyRep/blob/master/pyrep/objects/joint.py, where it specifies interval[0] is the joint minimum allowed value, interval[1] is the joint range (the maximum allowed value is interval[0]+interval[1].
interval[0] is the joint minimum allowed value, interval[1] is the joint range (the maximum allowed value is interval[0]+interval[1]
The related codes are in line 102 of https://github.com/stepjam/PyRep/blob/master/pyrep/robots/end_effectors/gripper.py, where it goes like joint_range = joint_intervals[:, 1] - joint_intervals[:, 0], appearing to treat joint_intervals[1] as the maximum allowed value instead of the joint range.
joint_range = joint_intervals[:, 1] - joint_intervals[:, 0]
joint_rage = joint_intervals[:, 1]
The text was updated successfully, but these errors were encountered:
No branches or pull requests
Description of the bug
The specification of joint intervals
The related codes are in line 151~162 of https://github.com/stepjam/PyRep/blob/master/pyrep/objects/joint.py, where it specifies
interval[0] is the joint minimum allowed value, interval[1] is the joint range (the maximum allowed value is interval[0]+interval[1]
.The incorrect use of joint intervals
The related codes are in line 102 of https://github.com/stepjam/PyRep/blob/master/pyrep/robots/end_effectors/gripper.py, where it goes like
joint_range = joint_intervals[:, 1] - joint_intervals[:, 0]
, appearing to treat joint_intervals[1] as the maximum allowed value instead of the joint range.My suggestions on improvement
joint_rage = joint_intervals[:, 1]
.The text was updated successfully, but these errors were encountered: