Added more sensor properties to scene/info topic #1344
Conversation
Signed-off-by: ahcorde <[email protected]>
Codecov Report
@@ Coverage Diff @@
## ign-gazebo3 #1344 +/- ##
===============================================
+ Coverage 77.74% 77.82% +0.07%
===============================================
Files 248 248
Lines 14239 14342 +103
===============================================
+ Hits 11070 11161 +91
- Misses 3169 3181 +12
Continue to review full report at Codecov.
|
| components::AirPressureSensor>(_entity); | ||
| if (airPressureComp) | ||
| { | ||
| sensorMsg->set_type("air_preasure"); |
There was a problem hiding this comment.
| sensorMsg->set_type("air_preasure"); | |
| sensorMsg->set_type("air_pressure"); |
Signed-off-by: ahcorde <[email protected]>
|
This pull request has been mentioned on Gazebo Community. There might be relevant details there: https://community.gazebosim.org/t/new-ignition-releases-2022-03-01-citadel-edifice-fortress/1313/1 |
|
This pull request has been mentioned on Gazebo Community. There might be relevant details there: https://community.gazebosim.org/t/new-ignition-releases-2022-04-13-fortress-edifice/1367/1 |
Signed-off-by: ahcorde [email protected]
🎉 New feature
Summary
Topic
/world/<world_name>/scene/infois sending some data about the sensor, but it's just the id, parent, name and pose. There is no data about the sensor itself. I don't know if this is intended.This PR add property data to:
Test it
Checklist
codecheckpassed (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-bymessages.