Support printing sensors using ign model
#1157
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
added 2 commits
November 1, 2021 14:49
ign model [options]
Available Options:
--list Get a list of the available models.
-m [--model] arg Select the model to be shown.
-p [--pose] Print the pose of the model.
-l [--link] arg Select a link to show its properties.
If no arg is passed all links are printed
Requires the -m option
E.g. to get information about the
caster link in the diff_drive world, run:
ign model -m vehicle_blue -l caster
-s [--sensor] arg Select a sensor to show its properties.
If no arg is passed all sensors are printed
Requires the -m and -l options
E.g. to get information about the
imu sensor in the sensors world, run:
ign model -m sensors_box -l link -s imu
-j [--joint] arg Select a joint to show its properties.
If no arg is passed all joints are printed
Requires the -m option
E.g. to get information about the
caster_wheel joint in the diff_drive
world, run:
ign model -m vehicle_blue -j caster_wheel
-h [--help] Print this help message.
--force-version <VERSION> Use a specific library version.
--versions Show the available versions.
Signed-off-by: Nate Koenig <[email protected]>
Signed-off-by: Nate Koenig <[email protected]>
Codecov Report
@@ Coverage Diff @@
## ign-gazebo6 #1157 +/- ##
===============================================
- Coverage 63.69% 62.18% -1.52%
===============================================
Files 256 256
Lines 20116 20355 +239
===============================================
- Hits 12813 12657 -156
- Misses 7303 7698 +395
Continue to review full report at Codecov.
|
ahcorde
requested changes
Nov 2, 2021
Signed-off-by: Nate Koenig <[email protected]>
ahcorde
requested changes
Nov 3, 2021
added 2 commits
November 4, 2021 08:29
Signed-off-by: Nate Koenig <[email protected]>
ahcorde
requested changes
Nov 4, 2021
src/ModelCommandAPI_TEST.cc
Outdated
Comment on lines
93
to
94
| ignition::common::joinPaths(std::string(PROJECT_SOURCE_PATH) + | ||
| "/test/worlds/static_diff_drive_vehicle.sdf")); |
There was a problem hiding this comment.
Suggested change
| ignition::common::joinPaths(std::string(PROJECT_SOURCE_PATH) + | |
| "/test/worlds/static_diff_drive_vehicle.sdf")); | |
| ignition::common::joinPaths(std::string(PROJECT_SOURCE_PATH), "test", "worlds", "static_diff_drive_vehicle.sdf")); |
There was a problem hiding this comment.
Right, thanks. Done.
added 2 commits
November 4, 2021 10:41
Signed-off-by: Nate Koenig <[email protected]>
Signed-off-by: Nate Koenig <[email protected]>
ahcorde
requested changes
Nov 4, 2021
ahcorde
approved these changes
Nov 4, 2021
8 tasks
Signed-off-by: Nate Koenig <[email protected]>
WilliamLewww
pushed a commit
to WilliamLewww/ign-gazebo
that referenced
this pull request
Dec 7, 2021
* Support printing sensors using Print information about models.
ign model [options]
Available Options:
--list Get a list of the available models.
-m [--model] arg Select the model to be shown.
-p [--pose] Print the pose of the model.
-l [--link] arg Select a link to show its properties.
If no arg is passed all links are printed
Requires the -m option
E.g. to get information about the
caster link in the diff_drive world, run:
ign model -m vehicle_blue -l caster
-s [--sensor] arg Select a sensor to show its properties.
If no arg is passed all sensors are printed
Requires the -m and -l options
E.g. to get information about the
imu sensor in the sensors world, run:
ign model -m sensors_box -l link -s imu
-j [--joint] arg Select a joint to show its properties.
If no arg is passed all joints are printed
Requires the -m option
E.g. to get information about the
caster_wheel joint in the diff_drive
world, run:
ign model -m vehicle_blue -j caster_wheel
-h [--help] Print this help message.
--force-version <VERSION> Use a specific library version.
--versions Show the available versions.
Signed-off-by: Nate Koenig <[email protected]>
* Added tests
Signed-off-by: Nate Koenig <[email protected]>
* Update documentation
Signed-off-by: Nate Koenig <[email protected]>
* use joinPaths
Signed-off-by: Nate Koenig <[email protected]>
* Use commas
Signed-off-by: Nate Koenig <[email protected]>
* More commas
Signed-off-by: Nate Koenig <[email protected]>
* Moved test to require DRI
Signed-off-by: Nate Koenig <[email protected]>
Co-authored-by: Nate Koenig <[email protected]>
Signed-off-by: William Lew <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Signed-off-by: Nate Koenig [email protected]
🎉 New feature
Summary
Print some sensor information using
ign model. Output is currently limited to altimeter, imu, camera, and rgbd cameras.Test it
ign gazebo -v4 sensors.sdfign model -m sensors_boxChecklist
codecheckpassed (See contributing)Note to maintainers: Remember to use Squash-Merge