-
Notifications
You must be signed in to change notification settings - Fork 277
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
minimal_scene.sdf example: add camera_clip params #1166
minimal_scene.sdf example: add camera_clip params #1166
Conversation
This adds example camera_clip parameters to the MinimalScene plugin in the example world file minimal_scene.sdf These parameters were added in gazebosim/gz-gui#309 Signed-off-by: Steve Peters <scpeters@openrobotics.org>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM but should we make an ign-gui6
release before merging?
Codecov Report
@@ Coverage Diff @@
## ign-gazebo6 #1166 +/- ##
============================================
Coverage 62.18% 62.18%
============================================
Files 256 256
Lines 20355 20355
============================================
Hits 12657 12657
Misses 7698 7698 Continue to review full report at Codecov.
|
I tested with the current release of ign-gazebo6 and ign-gui6, and it didn't give any error messages. I think the parameters are just ignored, so I would be fine with merging now, but can also wait if needed |
Signed-off-by: Steve Peters <scpeters@openrobotics.org>
I noticed |
This adds example camera_clip parameters to the MinimalScene plugin in the example world file minimal_scene.sdf These parameters were added in gazebosim/gz-gui#309 Also increase timeout for examples build test Signed-off-by: Steve Peters <scpeters@openrobotics.org> Signed-off-by: William Lew <WilliamMilesLew@gmail.com>
🎉 New feature
Small follow-up to gazebosim/gz-gui#309 and #1162
Summary
This adds example
camera_clip
parameters to theMinimalScene
plugin in the example world fileminimal_scene.sdf
. These parameters were added in gazebosim/gz-gui#309.Test it
If you build against
ign-gui6
from souce, then when you openminimal_scene.sdf
and expand the View Angle widget, you should see the values 0.25 and 25000 in the "Camera's clipping plane distance" section.Checklist
codecheck
passed (See contributing)Note to maintainers: Remember to use Squash-Merge