USDZExporter: Support metalness/roughness factor and Image combination
See original GitHub issueIs your feature request related to a problem? Please describe. Currently in usdzExporter , the roughnessMap and metalnessMap is only exported if roughness is 1 and metalness 1. https://github.com/mrdoob/three.js/pull/22201 this was due to the limitation that usdz generated does not have support for both a factor and image .
A clear and concise description of what the problem is.
On a metal ring model I exported which had roughness of 0.75 and a dark roughness map , due to the conditions above the image was skipped and the model was exported with just float inputs:roughness = 0.75
which made the usdz model very rough and inconsistent with the three js render
Describe the solution you’d like Since there have been some good updates from @sunag and @yuta-hayashi ,I was wondering if we can revisit this topic and implement a alternate solution
Issue Analytics
- State:
- Created a year ago
- Reactions:1
- Comments:5 (1 by maintainers)
Top GitHub Comments
I was looking for references but only found this one based on
NodeGraph
: Do you know some other reference too?https://github.com/PixarAnimationStudios/USD/blob/dev/pxr/usd/usdMtlx/testenv/testUsdMtlxFileFormat.testenv/baseline/NodeGraphs.usda
@elalish The case of zero textures and just factors like
roughness:0.1
andmetalness:1
should export correctly , like you said that error message can be caused due to wide variety of reasons (one silly case i remember is presence of symbols in the material names )Made a fiddle please test it out, (the usdz file can renamed to .zip and can be opened on any os and the materials info can be checked)