-
Notifications
You must be signed in to change notification settings - Fork 201
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
MAYA-103736 Clear frameSamples set #326
Conversation
…when the flag is used.
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.
lgtm! thanks.
@@ -93,6 +93,7 @@ UsdMayaExportTranslator::writer(const MFileObject &file, | |||
theOption[1].split(',', filteredTypes); | |||
} | |||
else if (argName == "frameSample") { | |||
frameSamples.clear(); |
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.
The default value of 0.0 for frameSample will always be listed in the options. If the flag is explicitly set then we'll parse it a second time and add in all of the associated values, so we need to clear the set before parsing it that second time.
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.
For future - We should avoid code duplication and extract the common components into the core library.
* MAYA-127016 : Support viewport XRay shading mode * MAYA-127016 : Add a comment for the hardcoded value of opacity and use a struct for the constructor of MayaHydraMaterialNetworkConverter
Clear frameSamples set so that we don't add in the default value when the flag is used.