Skip to content
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

Removed DisplayColor from ShadingMode export command as the flag has its own command exportDisplayColor #2930

Merged
merged 1 commit into from
Mar 10, 2023

Conversation

AramAzhari-adsk
Copy link
Collaborator

Removed DisplayColor from ShadingMode export command as the flag has its own command exportDisplayColor.

@AramAzhari-adsk AramAzhari-adsk added documentation Improvements or additions to documentation adsk Related to Autodesk plugin labels Mar 9, 2023
@AramAzhari-adsk
Copy link
Collaborator Author

There is still a shadingMode with DisplayColor on the import section. Is that also removed?

| `-shadingMode` | `-shd` | string[2] multi| `useRegistry` `UsdPreviewSurface` | Ordered list of shading mode importers to try when importing materials. The search stops as soon as one valid material is found. Allowed values for the first parameter are: `none` (stop search immediately, must be used to signal no material import), `displayColor` (if there are bound materials in the USD, create corresponding Lambertian shaders and bind them to the appropriate Maya geometry nodes), `pxrRis` (attempt to reconstruct a Maya shading network from (presumed) Renderman RIS shading networks in the USD), `useRegistry` (attempt to reconstruct a Maya shading network from (presumed) UsdShade shading networks in the USD) the second item in the parameter pair is a convertMaterialFrom flag which allows specifying which one of the registered USD material sources to explore. The full list of registered USD material sources can be found via the `mayaUSDListShadingModesCommand` command. |

If so, then both the above and

| `-preferredMaterial` | `-prm` | string | `lambert` | Indicate a preference towards a Maya native surface material for importers that can resolve to multiple Maya materials. Allowed values are `none` (prefer plugin nodes like pxrUsdPreviewSurface and aiStandardSurface) or one of `lambert`, `standardSurface`, `blinn`, `phong`. In displayColor shading mode, a value of `none` will default to `lambert`.

need to be updated.

@seando-adsk
Copy link
Collaborator

@AramAzhari-adsk Sorry saw your questions here after approving. I don't know the answers. Maybe @pierrebai-adsk does.

@AramAzhari-adsk
Copy link
Collaborator Author

@AramAzhari-adsk Sorry saw your questions here after approving. I don't know the answers. Maybe @pierrebai-adsk does.

@seando-adsk it is not required to complete this PR.

@AramAzhari-adsk AramAzhari-adsk added the ready-for-merge Development process is finished, PR is ready for merge label Mar 10, 2023
@seando-adsk seando-adsk removed the documentation Improvements or additions to documentation label Mar 10, 2023
@seando-adsk seando-adsk merged commit fea7549 into dev Mar 10, 2023
@seando-adsk seando-adsk deleted the azharia/MAYA-124881/correcting-command-docs branch March 10, 2023 20:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
adsk Related to Autodesk plugin ready-for-merge Development process is finished, PR is ready for merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants