-
Notifications
You must be signed in to change notification settings - Fork 91
New issue
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
sdf/1.10/joint.sdf: add screw_thread_pitch #1168
Merged
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Gazebo-classic interprets this parameter with units of radians / meter with a positive value corresponding to a left-handed thread. Signed-off-by: Steve Peters <scpeters@openrobotics.org>
Deprecate thread_pitch parameter and recommend new screw_thread_pitch parameter with different units and sign convention. Signed-off-by: Steve Peters <scpeters@openrobotics.org>
Signed-off-by: Steve Peters <scpeters@openrobotics.org>
This adds ScrewThreadPitch and SetScrewThreadPitch APIs for getting and setting the thread pitch using the units and sign convention of //joint/screw_thread_pitch. During Joint::Load, the screw_thread_pitch variable is preferred, while thread_pitch is used otherwise. Signed-off-by: Steve Peters <scpeters@openrobotics.org>
Signed-off-by: Steve Peters <scpeters@openrobotics.org>
Signed-off-by: Steve Peters <scpeters@openrobotics.org>
Signed-off-by: Steve Peters <scpeters@openrobotics.org>
Codecov Report
@@ Coverage Diff @@
## sdf13 #1168 +/- ##
==========================================
- Coverage 87.26% 87.25% -0.01%
==========================================
Files 125 125
Lines 16102 16114 +12
==========================================
+ Hits 14051 14061 +10
- Misses 2051 2053 +2
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. |
azeey
reviewed
Oct 11, 2022
azeey
reviewed
Oct 13, 2022
Signed-off-by: Steve Peters <scpeters@openrobotics.org>
Add param_precedence joint to test file with identical values for thread_pitch and screw_thread_pitch and confirm that the screw_thread_pitch value is used. Signed-off-by: Steve Peters <scpeters@openrobotics.org>
scpeters
added a commit
to scpeters/drake
that referenced
this pull request
Oct 18, 2022
Depends on gazebosim/sdformat#1168 to provide the //joint/screw_thread_pitch parameter in SDFormat 1.10 and corresponding Joint::ScrewThreadPitch() that uses the same units as drake: meters / revolution.
This was referenced Oct 18, 2022
scpeters
added a commit
to scpeters/drake
that referenced
this pull request
Oct 18, 2022
Depends on gazebosim/sdformat#1168 to provide the //joint/screw_thread_pitch parameter in SDFormat 1.10 and corresponding Joint::ScrewThreadPitch() that uses the same units as drake: meters / revolution.
I've opened RobotLocomotion/drake#18127 as a draft pull request that uses this branch. I'll mark it as ready for review once this PR has been merged and released |
azeey
approved these changes
Oct 18, 2022
scpeters
added a commit
to scpeters/drake
that referenced
this pull request
Oct 20, 2022
Depends on gazebosim/sdformat#1168 to provide the //joint/screw_thread_pitch parameter in SDFormat 1.10 and corresponding Joint::ScrewThreadPitch() that uses the same units as drake: meters / revolution.
scpeters
added a commit
to scpeters/drake
that referenced
this pull request
Oct 24, 2022
Depends on gazebosim/sdformat#1168 to provide the //joint/screw_thread_pitch parameter in SDFormat 1.10 and corresponding Joint::ScrewThreadPitch() that uses the same units as drake: meters / revolution.
scpeters
added a commit
to scpeters/drake
that referenced
this pull request
Oct 24, 2022
Depends on gazebosim/sdformat#1168 to provide the //joint/screw_thread_pitch parameter in SDFormat 1.10 and corresponding Joint::ScrewThreadPitch() that uses the same units as drake: meters / revolution.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
🦟 Bug fix
Fixes #1125 and adds alternate parameter
Summary
This clarifies the definition (including units and sign convention) of the
//joint/thread_pitch
parameter in 3f080e6 based on the interpretation of this parameter in gazebo-classic, fixing #1125. Based on the additional discussion in that issue, I have added a new element//joint/screw_thread_pitch
with units of meters / revolution that is positive for right-handed threads, and deprecates//joint/thread_pitch
in SDFormat 1.10. C++ and python APIs for getting and setting the thread pitch using the new units / sign convention are added as well, while preserving the functionality of the existing API.Checklist
codecheck
passed (See contributing)Note to maintainers: Remember to use Squash-Merge and edit the commit message to match the pull request summary while retaining
Signed-off-by
messages.