-
Notifications
You must be signed in to change notification settings - Fork 281
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
Implement Pose3d with common widget pose #1571
Conversation
Signed-off-by: youhy <[email protected]>
Signed-off-by: youhy <[email protected]>
Signed-off-by: youhy <[email protected]>
Signed-off-by: Jenn Nguyen <[email protected]>
Signed-off-by: youhy <[email protected]>
Signed-off-by: youhy <[email protected]>
Codecov Report
@@ Coverage Diff @@
## ign-gazebo3 #1571 +/- ##
============================================
Coverage 77.76% 77.76%
============================================
Files 250 250
Lines 14443 14443
============================================
Hits 11231 11231
Misses 3212 3212 Continue to review full report at Codecov.
|
Signed-off-by: youhy <[email protected]>
Signed-off-by: youhy <[email protected]>
Signed-off-by: youhy <[email protected]>
are there any plans to port this change to fortress? there are conflicts when I try to merge this forward, and I'm not sure how to resolve them |
@scpeters Fortress has a plotting feature for pose that is not supported in Citadel so there are conflicts when you forward-port. I am working a PR in gz-gui6 to add the feature and after a release for gui6 we should be good to forward-port. |
thank you! |
New feature
Summary
Implement
Pose
in Component Inspector with common widget GzPose introduced in gz-gui#424.Behavior should be exactly the same as before.
Test it
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.