Changed mesh orientations, added normals to triangle mesh, clarified … #80
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
…camera. To make it easier for tools which don't yet read both types of USD meshes, this commit changes the geometry to be rightHanded. There should be no visual differences for renderers/software that recognize meshes could have either winding order. (Issue usd-wg#70) The triangulated variant now has `normals` authored as well now (Issue usd-wg#69) Also re-authored the Camera transform using XformCommonAPI, rather than a 4x4 Matrix, so the translate/rotate values are clear.
|
Not sure why Logged as #82 |
Changed displayColor to be constant-interpolated, removed some unused primvars, and authored extents/extentsHint attributes in a few places that were missing it.
|
yeah, sadness, I'm guessing there are reasons why mtlx isn't included in usd-core (which I believe is what we're using here) |
|
Two reasons, boiling down to distro from pypi problems. The first is having a zoo of dylibs; that'll be resolvable more easily when USD is able to use the static/monolithic builds of MaterialX; not all the CMake changes have landed in Mx or USD for that though. The other tricky bit is the MaterialX node library itself. I don't know that anyone's investigated how to get that pip installable yet. |
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.
…camera.
Cleaned up some Houdini/Karma ApiSchemas or properties.
To make it easier for tools which don't yet read both types of USD meshes, this commit changes the geometry to be rightHanded. There should be no visual differences for renderers/software that recognize meshes could have either winding order. (Issue #70)
The triangulated variant now has
normalsauthored as well now (Issue #69)Also re-authored the Camera transform using XformCommonAPI, rather than a 4x4 Matrix, so the translate/rotate values are clear.