Skip to content

sdf -> usd: Added sensor#830

Merged
ahcorde merged 32 commits intosdf12from
ahcorde/sdf_to_usd_sensors
Feb 25, 2022
Merged

sdf -> usd: Added sensor#830
ahcorde merged 32 commits intosdf12from
ahcorde/sdf_to_usd_sensors

Conversation

@ahcorde
Copy link
Copy Markdown
Collaborator

@ahcorde ahcorde commented Jan 26, 2022

Signed-off-by: ahcorde ahcorde@gmail.com

🎉 New feature

Parse SDF sensor and convert them in USD

  • Camera
  • Lidar
  • IMU

Added test too

Checklist

  • Signed all commits for DCO
  • Added tests
  • Added example and/or tutorial
  • Updated documentation (as needed)
  • Updated migration guide (as needed)
  • codecheck passed (See contributing)
  • All tests passed (See test coverage)
  • While waiting for a review on your PR, please help review another open pull request to support the maintainers

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.

Signed-off-by: ahcorde <ahcorde@gmail.com>
Signed-off-by: ahcorde <ahcorde@gmail.com>
Signed-off-by: ahcorde <ahcorde@gmail.com>
@ahcorde ahcorde requested a review from adlarkin January 26, 2022 15:38
@ahcorde ahcorde self-assigned this Jan 26, 2022
@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Jan 26, 2022

Codecov Report

Merging #830 (8903f48) into sdf12 (5e3605f) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##            sdf12     #830   +/-   ##
=======================================
  Coverage   90.88%   90.88%           
=======================================
  Files          78       78           
  Lines       12636    12636           
=======================================
  Hits        11484    11484           
  Misses       1152     1152           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 5e3605f...8903f48. Read the comment docs.

@adlarkin adlarkin mentioned this pull request Jan 26, 2022
8 tasks
ahcorde and others added 6 commits January 26, 2022 23:10
Signed-off-by: ahcorde <ahcorde@gmail.com>
Signed-off-by: Ashton Larkin <42042756+adlarkin@users.noreply.github.com>
Signed-off-by: Teo Koon Peng <koonpeng@openrobotics.org>
Signed-off-by: Teo Koon Peng <koonpeng@openrobotics.org>
ahcorde and others added 9 commits February 10, 2022 17:10
Signed-off-by: ahcorde <ahcorde@gmail.com>
Signed-off-by: ahcorde <ahcorde@gmail.com>
Signed-off-by: ahcorde <ahcorde@gmail.com>
Signed-off-by: ahcorde <ahcorde@gmail.com>
…rapped sdf errors

Signed-off-by: Ashton Larkin <42042756+adlarkin@users.noreply.github.com>
Signed-off-by: Ashton Larkin <42042756+adlarkin@users.noreply.github.com>
Signed-off-by: Ashton Larkin <42042756+adlarkin@users.noreply.github.com>
Signed-off-by: Ashton Larkin <42042756+adlarkin@users.noreply.github.com>
@scpeters scpeters added the usd label Feb 21, 2022
Base automatically changed from ahcorde/sdf_to_usd_visuals to sdf12 February 24, 2022 09:42
Copy link
Copy Markdown
Contributor

@adlarkin adlarkin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I made some updates in 10d7da1 and have one question, but other than that, LGTM!

Comment thread usd/src/sdf_parser/Sensor.cc Outdated
Signed-off-by: ahcorde <ahcorde@gmail.com>
@ahcorde ahcorde requested a review from adlarkin February 25, 2022 08:58
Copy link
Copy Markdown
Contributor

@adlarkin adlarkin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@scpeters or @azeey, would you like to take a look at this before we merge it?

Comment thread usd/src/sdf_parser/Sensor.cc Outdated
Comment thread usd/src/sdf_parser/Sensor.cc Outdated
Signed-off-by: ahcorde <ahcorde@gmail.com>
@ahcorde ahcorde merged commit 4dd71a5 into sdf12 Feb 25, 2022
@ahcorde ahcorde deleted the ahcorde/sdf_to_usd_sensors branch February 25, 2022 22:06
@osrf-triage
Copy link
Copy Markdown

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants