-
-
Notifications
You must be signed in to change notification settings - Fork 3.1k
[USD] Integrate "tinyusdz" project #5628
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[USD] Integrate "tinyusdz" project #5628
Conversation
|
Forgive some ignorance here but I was just wondering why the tinyusdz library and not OpenUSD for what I would assume would offer more support as its their format? Love your work by the way its been so useful with O3DE! Thank you. |
|
@Christian-Prather On the other hand, the TinyUSDZ project self-advertises as being self-contained and potentially easier to integrate: |
|
@tellypresence |
|
@Christian-Prather
P.S. It's nice of you to offer to help, do you happen to have any contacts on the |
"rendermesh-refactor" was merged to "dev" around 9 May 2024 but merge was not obvious from commit messages
(cherry picked from commit d89fe8f034c353cc5cc5b3ac78cd8845e006de38)
assimp github PR auto-CI checks clone/build the tinyusdz code, and reject PR due to compiler warnings in the 3rd party external tinyusdz project
|
Lets see. Thanks a lot for your great work!!! |
|
Thank you for the great work! |
Unfortunately I do not :) but if needed happy to help out in any other way, super excited to see this merged! |
|
Excited to see this! Going to try it out in O3DE. |
Overview
This PR provides limited support for loading USD model files
usdusdausdcusdzby leveraging the external
tinyusdzproject.Closes #5547
Details
During build the entire
tinyusdzrepo is automatically cloned under thecontrib/tinyusdzfolder at a specific commit hash defined incode/CMakeLists.txt; currently the hash points to the latest commit on the main (dev) branch of thetinyusdzproject. The source code is built alongside the rest of the assimp source code and becomes part of the output library file (.a/.so etc)How to test this PR
ASSIMP_BUILD_USD_IMPORTER**the USD import feature has to be explicitly disabled by default. If enabled, assimp's github CI PR automatic checks will
tinyusdzproject codeDiscussion
#1332 "Support USD," 2 Jul 2017: insightful technical discussion
Appendix
Original PR was #5551 but got destroyed following force push due to confusion between the assimp branch named
masterand the assimp tag namedmaster