Skip to content
This repository was archived by the owner on Sep 20, 2024. It is now read-only.

Maya: Arnold Scene Source overhaul - OP-4865#4449

Merged
antirotor merged 25 commits intoynput:developfrom
tokejepsen:enhancement/OP-4865_static-ass-file-option
Feb 14, 2023
Merged

Maya: Arnold Scene Source overhaul - OP-4865#4449
antirotor merged 25 commits intoynput:developfrom
tokejepsen:enhancement/OP-4865_static-ass-file-option

Conversation

@tokejepsen
Copy link
Copy Markdown
Contributor

@tokejepsen tokejepsen commented Feb 10, 2023 •
edited
Loading

Brief description

General overhaul of the Arnold Scene Source (ASS) workflow.

Description

This originally was to support static files (non-sequencial) ASS publishing, but digging deeper whole workflow needed an update to get ready for further issues. During this overhaul the following changes were made:

  • Generalized Arnold Standin workflow to a single loader.
  • Support multiple nodes as proxies.
  • Support proxies for pointcache family.
  • Generalized approach to proxies as resources, so they can be the same file format as the original.

This workflow should allow further expansion to utilize operators and eventually USD.

Discussion

By supporting proxies for the pointcache family, we can also get rid of the specialized proxyAbc workflow. What do we think of that?
We could also implement/support proxy workflows when it comes to animation and rig, although not sure how much that is used outside of big studios.

Testing notes:

  1. Create and publish Arnold Scene Source or Pointcache instance.
  2. Load and render.
  3. Repeat with proxy workflow to see proxy in viewport and original at render time.

@tokejepsen tokejepsen self-assigned this Feb 10, 2023
@ynbot
Copy link
Copy Markdown
Contributor

ynbot commented Feb 10, 2023

Task linked: OP-4865 static ass file option

@tokejepsen tokejepsen mentioned this pull request Feb 10, 2023
Closed
@tokejepsen tokejepsen marked this pull request as draft February 10, 2023 09:17
Minkiu
Minkiu reviewed Feb 10, 2023
View reviewed changes
@tokejepsen tokejepsen removed the request for review from antirotor February 10, 2023 09:52
@tokejepsen tokejepsen requested a review from antirotor February 10, 2023 09:52
@tokejepsen tokejepsen marked this pull request as ready for review February 10, 2023 10:01
Minkiu
Minkiu reviewed Feb 10, 2023
View reviewed changes
Copy link
Copy Markdown
Contributor

@Minkiu Minkiu left a comment •
edited
Loading

Choose a reason for hiding this comment

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

Haven't tested the changes themselves, and I would be keen in adding more Docstrings, even if they seem redundant.

Overall code looks good ✔️

@moonyuet
Copy link
Copy Markdown
Member

moonyuet commented Feb 10, 2023

I have tested loader and extractor, so far so good. Apart from the question about the second extraction in the extractor, I think it looks ok but it somehow breaks another loader due to the adjustment
image

This is something I saw when I run reference .ass standin with proxy
image

image

@tokejepsen
Copy link
Copy Markdown
Contributor Author

tokejepsen commented Feb 10, 2023 •
edited
Loading

This is something I saw when I run reference .ass standin with proxy

@moonyuet Not sure how you get that Loader since I've removing it with this commit?
9733b07

@moonyuet
Copy link
Copy Markdown
Member

moonyuet commented Feb 10, 2023

I have tested loader and extractor, so far so good. Apart from the question about the second extraction in the extractor, I think it looks ok but it somehow breaks another loader due to the adjustment image

This is something I saw when I run reference .ass standin with proxy image

image

This is something I saw when I run reference .ass standin with proxy

@moonyuet Not sure how you get that Loader since I've removing it with this commit? 9733b07

Sorry, maybe I mixed with other forks. my bad! Let me test again!

@moonyuet
Copy link
Copy Markdown
Member

moonyuet commented Feb 10, 2023

I have tested it again, the loader works fine. Sorry for the inconvenience!
image

The ass will be published successfully. Not sure if the ass with "_proxy" needed to be published. If yes, the ass with "_proxy" is not there yet.
image

@tokejepsen
Copy link
Copy Markdown
Contributor Author

tokejepsen commented Feb 10, 2023

The ass will be published successfully. Not sure if the ass with "_proxy" needed to be published. If yes, the ass with "_proxy" is not there yet.

@moonyuet the proxy version goes into the resources folder atm. Did you not get it publish and loaded?

antirotor
antirotor approved these changes Feb 14, 2023
View reviewed changes
Copy link
Copy Markdown
Member

@antirotor antirotor left a comment

Choose a reason for hiding this comment

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

It works fine, just little style comment to accept or dismiss :)

@tokejepsen @antirotor
Update openpype/hosts/maya/plugins/load/load_arnold_standin.py
6bdbdd4 
Co-authored-by: Ondřej Samohel <33513211+antirotor@users.noreply.github.com>
@antirotor antirotor merged commit d06a77d into ynput:develop Feb 14, 2023
@ynbot ynbot added this to the next-patch milestone Feb 14, 2023
@tokejepsen tokejepsen deleted the enhancement/OP-4865_static-ass-file-option branch February 14, 2023 15:42
@tokejepsen tokejepsen mentioned this pull request Feb 14, 2023
Merged
@tokejepsen tokejepsen mentioned this pull request Mar 30, 2023
Merged
@tokejepsen tokejepsen mentioned this pull request Jan 30, 2024
Merged
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

3 more reviewers

@Minkiu Minkiu Minkiu approved these changes

@antirotor antirotor antirotor approved these changes

@moonyuet moonyuet moonyuet approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

@tokejepsen tokejepsen

Labels

host: Maya

Projects

None yet

Milestone

3.15.1

Development

Successfully merging this pull request may close these issues.

5 participants

@tokejepsen @ynbot @moonyuet @Minkiu @antirotor