Skip to content

USD to SDF: Added visuals#897

Merged
scpeters merged 114 commits intosdf12from
ahcorde/usd_to_sdf_visuals
Apr 11, 2022
Merged

USD to SDF: Added visuals#897
scpeters merged 114 commits intosdf12from
ahcorde/usd_to_sdf_visuals

Conversation

@ahcorde
Copy link
Copy Markdown
Collaborator

@ahcorde ahcorde commented Mar 22, 2022

🎉 New feature

Summary

Added visuals with materials

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.

ahcorde and others added 30 commits March 9, 2022 13:22
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>
Signed-off-by: ahcorde <ahcorde@gmail.com>
Signed-off-by: ahcorde <ahcorde@gmail.com>
Signed-off-by: ahcorde <ahcorde@gmail.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: 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>
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: 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>
Signed-off-by: ahcorde <ahcorde@gmail.com>
Signed-off-by: ahcorde <ahcorde@gmail.com>
ahcorde added 4 commits April 6, 2022 00:06
Signed-off-by: ahcorde <ahcorde@gmail.com>
Signed-off-by: ahcorde <ahcorde@gmail.com>
@ahcorde ahcorde marked this pull request as ready for review April 6, 2022 08:34
@ahcorde ahcorde requested review from azeey and scpeters as code owners April 6, 2022 08:34
ahcorde added 3 commits April 6, 2022 11:03
Signed-off-by: ahcorde <ahcorde@gmail.com>
Signed-off-by: ahcorde <ahcorde@gmail.com>
Signed-off-by: ahcorde <ahcorde@gmail.com>
@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Apr 7, 2022

Codecov Report

Merging #897 (d8de8ba) into sdf12 (256fc6b) will decrease coverage by 0.97%.
The diff coverage is 32.33%.

@@            Coverage Diff             @@
##            sdf12     #897      +/-   ##
==========================================
- Coverage   87.70%   86.73%   -0.98%     
==========================================
  Files         105      106       +1     
  Lines       15190    15451     +261     
==========================================
+ Hits        13323    13402      +79     
- Misses       1867     2049     +182     
Impacted Files Coverage Δ
usd/src/usd_parser/USDMaterial.cc 59.44% <ø> (ø)
usd/src/usd_parser/polygon_helper.cc 0.00% <0.00%> (ø)
usd/src/usd_parser/USDLinks.cc 45.79% <34.16%> (-49.21%) ⬇️
usd/src/usd_parser/USDWorld.cc 77.77% <40.00%> (-2.96%) ⬇️
usd/src/usd_parser/USDTransforms.cc 65.54% <0.00%> (-2.03%) ⬇️

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 256fc6b...d8de8ba. Read the comment docs.

adlarkin and others added 3 commits April 7, 2022 14:34
Signed-off-by: Ashton Larkin <42042756+adlarkin@users.noreply.github.com>
Signed-off-by: ahcorde <ahcorde@gmail.com>
Base automatically changed from ahcorde/usd_to_sdf_links to sdf12 April 8, 2022 20:22
Signed-off-by: Ashton Larkin <42042756+adlarkin@users.noreply.github.com>
@adlarkin adlarkin force-pushed the ahcorde/usd_to_sdf_visuals branch from ed92b92 to d7590f9 Compare April 11, 2022 02:06
Comment thread usd/src/usd_parser/USDLinks.hh
Comment thread usd/src/usd_parser/polygon_helper.hh Outdated
Comment thread usd/src/usd_parser/polygon_helper.hh Outdated
Comment thread usd/src/usd_parser/polygon_helper.hh Outdated
Comment thread usd/src/usd_parser/polygon_helper.hh Outdated
Comment thread usd/src/usd_parser/USDLinks.cc Outdated
Comment thread usd/src/usd_parser/USDLinks.cc Outdated
Comment thread usd/src/usd_parser/USDLinks.cc Outdated
Comment thread usd/src/usd_parser/USDLinks.cc Outdated
Comment thread usd/src/usd_parser/USDLinks.cc Outdated
ahcorde and others added 3 commits April 11, 2022 10:49
Signed-off-by: Alejandro Hernández <ahcorde@gmail.com>
Signed-off-by: Alejandro Hernández <ahcorde@gmail.com>
Signed-off-by: Ashton Larkin <42042756+adlarkin@users.noreply.github.com>
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 a few updates in b0085d0. Overall, LGTM, but I do have one last question before approving/merging.

Comment thread usd/src/usd_parser/USDLinks.cc
Signed-off-by: Alejandro Hernández <ahcorde@gmail.com>
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.

LGTM as long as CI comes back green 👍

@ahcorde ahcorde requested a review from adlarkin April 11, 2022 17:56
@ahcorde ahcorde enabled auto-merge (squash) April 11, 2022 20:22
@scpeters scpeters disabled auto-merge April 11, 2022 21:19
@scpeters scpeters merged commit a0b022d into sdf12 Apr 11, 2022
@scpeters scpeters deleted the ahcorde/usd_to_sdf_visuals branch April 11, 2022 21:21
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