-
Notifications
You must be signed in to change notification settings - Fork 322
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
Updated MaterialXTest to improve MDL support #1277
Updated MaterialXTest to improve MDL support #1277
Conversation
krohmerNV
commented
Mar 13, 2023
- changed the MDL test renderer to be used with Example DXR (shipped with the MDL SDK)
- with that, added a gltf sphere and a rotated hdri to get a closely matching scene setup
- extended the tests_to_html script for a three-way-comparison
- added a preprocessing step in the test to allow mdlgen to use flattenFilenames for application side resource handling
- replaced element->getActiveSourceUri() by doc->getSourceUri() because of failures with standard_surface_look_* tests
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for all the improvements and fixes. 👍 Just a few small comments.
9c621a9
to
176b00a
Compare
- changed the MDL test renderer to be used with Example DXR (shipped with the MDL SDK) - with that, added gltf sphere and a rotated hdri to get a closely matching scene setup - extended the tests_to_html script for a three-way-comparison - added a preprocessing step in the test to allow mdlgen to use flattenFilename for application side resource handling - replaced element->getActiveSourceUri() by doc->getSourceUri() because of failures with standard_surface_look_* tests
We can revisit this as developers begin integrating MDL into their standard render tests. Signed-off-by: Jonathan Stone <jstone@lucasfilm.com>
176b00a
to
35a0a84
Compare
Signed-off-by: Jonathan Stone <jstone@lucasfilm.com>
Signed-off-by: Jonathan Stone <jstone@lucasfilm.com>
Signed-off-by: Jonathan Stone <jstone@lucasfilm.com>
I believe I addressed all issues with this PR.
|
Thanks for those updates, @krohmerNV, and this changelist looks ready to go. I'd just wanted to run some local tests before merging, but otherwise this looks good to me. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks good to me, thanks @krohmerNV!
abc1f83
into
AcademySoftwareFoundation:main
…on#1277) - changed the MDL test renderer to be used with Example DXR (shipped with the MDL SDK) - extended the tests_to_html script for a three-way-comparison - added a preprocessing step in the test to allow mdlgen to use flattenFilenames for application side resource handling - replaced element->getActiveSourceUri() by doc->getSourceUri() because of failures with standard_surface_look_* tests