pablode / guc

glTF to USD converter with MaterialX support
Apache License 2.0
107 stars 6 forks source link

First UV set in USD is named "st" #22

Closed JGamache-autodesk closed 1 year ago

JGamache-autodesk commented 1 year ago

Small but important fix in makeStSetName(). The index should only be appended if the index is > 0.

pablode commented 1 year ago

Hi! Afaik 'st' is a convention and there's no guarantee that the implementation will use it. This is why I always emit explicit UsdPrimvarReader_float2 and geompropvalue nodes. I can understand that some applications do not support these - did you experience any particular issue?

JGamache-autodesk commented 1 year ago

Indeed "st. st1, st2" is but one convention. ArKit uses another (uv0, uv1, uv2). Using UsdPrimvarReader and geompropvalue is indeed the right way to handle that so I will close the issue.