Skip to content
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

Error when //inertial/pose/@relative_to is non-default #480

Merged
merged 8 commits into from
Feb 3, 2021

Conversation

jennuine
Copy link
Collaborator

Closes #380

Signed-off-by: Jenn Nguyen <jenn@openrobotics.org>
@github-actions github-actions bot added the 🏢 edifice Ignition Edifice label Jan 28, 2021
@jennuine jennuine requested a review from azeey January 28, 2021 23:58
@jennuine jennuine self-assigned this Jan 28, 2021
Copy link
Collaborator

@azeey azeey left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good. Left some minor comments.

src/Link.cc Outdated Show resolved Hide resolved
src/Link_TEST.cc Outdated Show resolved Hide resolved
src/Link_TEST.cc Outdated Show resolved Hide resolved
src/Link_TEST.cc Outdated Show resolved Hide resolved
Signed-off-by: Jenn Nguyen <jenn@openrobotics.org>
sdf/1.7/inertial.sdf Outdated Show resolved Hide resolved
sdf/1.7/inertial.sdf Outdated Show resolved Hide resolved
@jennuine jennuine force-pushed the jennuine/inertial_pose_relative_to branch from f5faa3d to 78bd773 Compare January 29, 2021 23:31
Signed-off-by: Jenn Nguyen <jenn@openrobotics.org>
@codecov-io
Copy link

codecov-io commented Jan 29, 2021

Codecov Report

Merging #480 (655b83f) into master (105bf22) will increase coverage by 0.03%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #480      +/-   ##
==========================================
+ Coverage   87.67%   87.70%   +0.03%     
==========================================
  Files          65       65              
  Lines        9223     9223              
==========================================
+ Hits         8086     8089       +3     
+ Misses       1137     1134       -3     
Impacted Files Coverage Δ
src/parser.cc 80.11% <0.00%> (+0.33%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 105bf22...655b83f. Read the comment docs.

Signed-off-by: Jenn Nguyen <jenn@openrobotics.org>
Signed-off-by: Jenn Nguyen <jenn@openrobotics.org>
Signed-off-by: Jenn Nguyen <jenn@openrobotics.org>
Migration.md Outdated
@@ -317,9 +301,12 @@ but with improved human-readability..

### Deprecations

1. **inerial.sdf** `//inertial/pose/@relative_to` attribute is deprecated
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you change this to say removed instead of deprecated?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done 655b83f

Signed-off-by: Jenn Nguyen <jenn@openrobotics.org>
@azeey azeey merged commit 93f8c56 into master Feb 3, 2021
@azeey azeey deleted the jennuine/inertial_pose_relative_to branch February 3, 2021 17:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🏢 edifice Ignition Edifice
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Inertial <pose relative_to="some_frame"> has no effect
4 participants