proposal: Should allow //pose rotation to be specified in degrees?
#252
Assignees
Comments
|
Release target is SDFormat 1.8 (est. Dec. 1, 2020). Could also incorporate quaternions, Denavit-Hartenberg, etc. But we should avoid pre-Cambrian explosims 💥... Other option: Require degrees, full-stop. |
This was referenced Jul 30, 2020
|
Punting on this for SDFormat 1.8 |
|
Per VC, Addisu mentioned that Aaron is looking into whether XSD and/or I am fine w/ compromise (e.g. Operating plan is to target Fortress (Sept. 2021): https://ignitionrobotics.org/docs/all/releases |
aaronchongth
added a commit
that referenced
this issue
Jun 2, 2021
Signed-off-by: Aaron Chong <[email protected]>
8 tasks
aaronchongth
added a commit
that referenced
this issue
Sep 2, 2021
* Added rotation type attribute to pose, handled parsing in param Signed-off-by: Aaron Chong <[email protected]> * Using degrees boolean attribute in pose instead, added tests and test sdf Signed-off-by: Aaron Chong <[email protected]> * Fix code check Signed-off-by: Aaron Chong <[email protected]> * Changed attribute to @rotation_type, modified tests, added delimiters Signed-off-by: Aaron Chong <[email protected]> * Passing attributes into param value parsing, reverted Utils loadPose function Signed-off-by: Aaron Chong <[email protected]> * Reverted location of parsing nested sdf Signed-off-by: Aaron Chong <[email protected]> * Added parent element for param, more tests Signed-off-by: Aaron Chong <[email protected]> * Const ref for GetAttributes and it's relevant calls Signed-off-by: Aaron Chong <[email protected]> * Reverting clone and copy behavior without reallocating memory, using make_shared Signed-off-by: Aaron Chong <[email protected]> * Compared parent element pointers instead of just the names Signed-off-by: Aaron Chong <[email protected]> * Removed redundant reset, checked for null parent before printing error Signed-off-by: Aaron Chong <[email protected]> * Moved tests from integration/param_parent_element to Param_TEST Signed-off-by: Aaron Chong <[email protected]> * Reverting urdf.sdf, adding new pose_without_frames.sdf, using that in actor and inertial Signed-off-by: Aaron Chong <[email protected]> * Fixed typo, added explicit named default variable, made rotation type errors clearer Signed-off-by: Aaron Chong <[email protected]> * Added comment in pose_1_9.sdf that each pose is independent Signed-off-by: Aaron Chong <[email protected]> * Removed use of parsing by string, modified tests to reflect changed behavior Signed-off-by: Aaron Chong <[email protected]> * Reverted changes made to schema for poses that do not handle relative_to frames Signed-off-by: Aaron Chong <[email protected]> * ComCompare degrees more explicitly in tests instead of using IGN_DTOR Signed-off-by: Aaron Chong <[email protected]> * Using consistent notation for reporting rotation_type attribute, modifying test Signed-off-by: Aaron Chong <[email protected]> * Using instead of typedef for forward declaration, error strings include full value Signed-off-by: Aaron Chong <[email protected]> * Using RedirectConsoleStream and ScopeExit, using Root and Root::LoadSdfString in tests Signed-off-by: Aaron Chong <[email protected]> * Added Reparse function for Param, relevant tests regarding rotation_type manipulation, cleaned up Signed-off-by: Aaron Chong <[email protected]> * Reducing changes to just the degrees attribute, modifying tests because of behavior changes in Set Signed-off-by: Aaron Chong <[email protected]> * Fixed stale brief and comment, breaking for loops early Signed-off-by: Aaron Chong <[email protected]> * Changed 'required' of @relative_to and @Degrees to 0, and modified description of pose to be more explicit Signed-off-by: Aaron Chong <[email protected]> * Modified reparsing tests in Param_TEST, added another test using Pose Signed-off-by: Aaron Chong <[email protected]> * Using alias Pose instead of full type name, added check for parent element pointer, fixed typo Signed-off-by: Aaron Chong <[email protected]> * Support printing pose elements in degrees, added tests Signed-off-by: Aaron Chong <[email protected]> * Removed const for SetParentElement Signed-off-by: Aaron Chong <[email protected]> * Adding additional documentation for the behavior between Set<T> and Reparse Signed-off-by: Aaron Chong <[email protected]> * Storing parsed tokens in array instead of vector Signed-off-by: Aaron Chong <[email protected]> * Used explicit Get<boo> for degrees attribute Signed-off-by: Aaron Chong <[email protected]> * Fixed missing array include for windows build Signed-off-by: Aaron Chong <[email protected]> * Check for valueIndex, instead of array size Signed-off-by: Aaron Chong <[email protected]>
8 tasks
|
Closing given merge of #689 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Will make proposal with suggested syntax. Degrees is my main interest, but we can leave space for other stuff.
Came about recently (as well as longer ago) when discussing here:
RobotLocomotion/drake#13036 (review)
Steps:
//pose[@rotation_type]#589Update document to better reflect proposed and accepted implementations sdf_tutorials#56ign sdfSupport printing sdf poses in degrees and allow snapping to commonly used angles #689Relates: origin: Support degrees in
rpyros/urdfdom#124(Moved from gazebosim/sdf_tutorials#8)
The text was updated successfully, but these errors were encountered: