LumaPictures / maya-to-hydra

This repo is no longer updated. Please see https://github.com/Autodesk/maya-usd
Other
10 stars 3 forks source link

Attribute for selecting renderer (defaultRenderGlobals.mtohRenderer) should be string, not enum #48

Closed pmolodo closed 5 years ago

pmolodo commented 5 years ago

If it's an enum, then it's value when it's saved out to file is unreliable - the list of available renderers may be different when it's opened, and the value is saved as an int - so "1" when it's saved may be different than when it's loaded.

Instead, it should be a string, which is validated against the current set of renderers, and use the default renderer if it's invalid (after printing a warning)

sirpalee commented 5 years ago

Closing this in favor of #44 .