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

Maya: Cant assign shaders to the ass file - OP-4859 #4460

Conversation

tokejepsen
Copy link
Member

@tokejepsen tokejepsen commented Feb 14, 2023

Brief description

Support AiStandIn nodes for look assignment.

Description

Using operators we assign shaders and attribute/parameters to nodes within standins. Initially there is only support for a limited mount of attributes but we can add support as needed;

primaryVisibility
castsShadows
receiveShadows
aiSelfShadows
aiOpaque
aiMatte
aiVisibleInDiffuseTransmission
aiVisibleInSpecularTransmission
aiVisibleInVolume
aiVisibleInDiffuseReflection
aiVisibleInSpecularReflection
aiSubdivUvSmoothing
aiDispHeight
aiDispPadding
aiDispZeroValue
aiStepSize
aiVolumePadding
aiSubdivType
aiSubdivIterations

Testing notes

  1. Load an alembic and lookdev, plus publish the look.
  2. Load the same alembic and publish as Arnold Scene Source.
  3. Load the publish Arnold Scene Source and assign the publish look through the LookManager.

Resolves #4535

@tokejepsen tokejepsen added host: Maya type: feature Larger, user affecting changes and completely new things labels Feb 14, 2023
@tokejepsen tokejepsen self-assigned this Feb 14, 2023
@ynbot
Copy link
Contributor

ynbot commented Feb 14, 2023

@tokejepsen tokejepsen marked this pull request as ready for review February 16, 2023 15:16
@tokejepsen tokejepsen assigned LiborBatek and unassigned tokejepsen Feb 21, 2023
@tokejepsen
Copy link
Member Author

Look at #4535

@tokejepsen
Copy link
Member Author

@LiborBatek give this another go. Think I've fixed #4535 now.

…e-ass-file

# Conflicts:
#	openpype/hosts/maya/api/lib.py
#	openpype/hosts/maya/tools/mayalookassigner/arnold_standin.py
#	openpype/hosts/maya/tools/mayalookassigner/lib.py
Copy link
Collaborator

@BigRoy BigRoy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Some notes

openpype/hosts/maya/tools/mayalookassigner/app.py Outdated Show resolved Hide resolved
openpype/hosts/maya/tools/mayalookassigner/commands.py Outdated Show resolved Hide resolved
@LiborBatek
Copy link
Member

@LiborBatek what version of Maya and MTOA are you running?

maya 2023.3 + arnold 5.1.3.3

@tokejepsen
Copy link
Member Author

@LiborBatek could you try this again now?
I've replaced the ASS query with json sidecar, so it should be more stable now.

@LiborBatek LiborBatek self-requested a review March 31, 2023 15:43
LiborBatek
LiborBatek previously approved these changes Mar 31, 2023
Copy link
Member

@LiborBatek LiborBatek left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Now its possible to assign Look asset to the Ass Instance loaded in scene as with other representations of the model.

So working ok.
image

Not sure why but I m not able to graph it in Hypershade editor its not shown as connected shader to the ass asset. Maybe its by its nature just a reminder what observed during testing....could be issue later on.

@tokejepsen
Copy link
Member Author

Not sure why but I m not able to graph it in Hypershade editor its not shown as connected shader to the ass asset. Maybe its by its nature just a reminder what observed during testing....could be issue later on.

Yeah, assigning shaders to aiStandIn is done through operators, which do not follow any native material assignments workflows.; https://help.autodesk.com/view/ARNOL/ENU/?guid=arnold_user_guide_ac_operators_html

@tokejepsen
Copy link
Member Author

@BigRoy you happy with this PR?

…e.py

Co-authored-by: Roy Nieterau <roy_nieterau@hotmail.com>
@tokejepsen
Copy link
Member Author

@moonyuet or @LiborBatek could I get an approval on this?

@LiborBatek LiborBatek self-requested a review April 3, 2023 11:19
Copy link
Member

@LiborBatek LiborBatek left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Seems ok. Being already tested.

@antirotor antirotor merged commit 4f0fed1 into ynput:develop Apr 3, 2023
@ynbot ynbot added this to the next-patch milestone Apr 3, 2023
@tokejepsen tokejepsen deleted the feature/OP-4859_cant-assign-shaders-to-the-ass-file branch April 3, 2023 15:15
@OlivierOsotimehin
Copy link

Hello,
This feature will be so great if it also worked on .abc loaded through the standin. This mean we could load animation caches directly in .abc and apply the look on them without the need to réexport in .ass

@tokejepsen
Copy link
Member Author

@OlivierOsotimehin it does work on abc representations :⁠-⁠)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
host: Maya size/M Denotes a PR changes 500-999 lines, ignoring general files type: feature Larger, user affecting changes and completely new things
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

Maya: Arnold Scene Source (ASS) publish fails
9 participants