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

Support assigning maya look to multiple separate assets inside an Arnold aiStandin with alembic files. #167

Conversation

BigRoy
Copy link
Contributor

@BigRoy BigRoy commented Oct 29, 2024

Changelog Description

  1. Do not force remove assignments on aiStandin look assignments, but allow updating/adding assingments so that assigning looks of multiple assets inside one aiStandin shape is supported.

  2. This also fixes a bug where cbId attribute exported from Houdini was not detected correctly and its value was incorrectly interpreted from the alembic property.

  • Maya seems to export as alembic.Abc.IArrayProperty
  • Houdini seems to export as alembic.Abc.IScalarProperty
  1. You can now also list assets and assign look to gpuCache if mtoa (Arnold) renderer is loaded and is set as the current renderer.

  2. This also fixes a bug where ALL assets inside an aiStandin are assigned even if only one of them was picked from the asset list in the look assigner.

We now support both.

Additional info

Fixes #160

Testing notes:

  1. Create a pointcache containing geometry from multiple folders (assets)
  2. Load the Alembic pointcache via AYON to aiStandin.
  3. Assign the look.

It should detect the multiple assets inside the aiStandin and allow assigning each of them without applying only the last asset.

…low updating/adding assingments so that assigning looks of multiple assets inside one `aiStandin` shape is supported
@BigRoy BigRoy added the type: enhancement Improvement of existing functionality or minor addition label Oct 29, 2024
@BigRoy BigRoy requested review from antirotor and moonyuet October 29, 2024 15:23
@BigRoy BigRoy self-assigned this Oct 29, 2024
@BigRoy BigRoy added the sponsored This is directly sponsored by a client or community member label Oct 29, 2024
Copy link
Member

@moonyuet moonyuet left a comment

Choose a reason for hiding this comment

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

Tested successfully in Maya:
image

@LiborBatek LiborBatek self-requested a review October 30, 2024 10:34
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.

I have used 3 different assets for pointcache and loaded it as a standin and using Look assigner correctly resulting into all looks applied to each entity inside! Really cool!

LGTM

Screenshot 2024-10-30 115249

And after using assign look:

Screenshot 2024-10-30 115403

@BigRoy BigRoy merged commit 165adaf into ynput:develop Oct 30, 2024
1 check passed
@BigRoy BigRoy deleted the enhancement/160-ay-7024_look-assigner-to-combined-aistanind-issue branch October 30, 2024 12:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
sponsored This is directly sponsored by a client or community member type: enhancement Improvement of existing functionality or minor addition
Projects
None yet
Development

Successfully merging this pull request may close these issues.

AY-7024_Look Assigner to combined aiStanInd issue
3 participants