Backport GridConfig improvements to Citadel's Grid3D #363
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.
🎉 New feature
Part of gazebosim/gz-sim#1254
Summary
#324 made multiple improvements to
GridConfig
andGrid3D
, and also consolidated them into the same plugin. That went into Fortress.On Citadel,
GridConfig
is onign-gazebo
, whileGrid3D
is onign-gui
(GridConfig
was moved toign-gui
in #236 ). So what this PR does is copy theGridConfig
code fromign-gui6
intoign-gui3
'sGrid3D
.Test it
Load the
Grid3D
plugin from the menu onign-gazebo
, for example.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.🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸🔸