search

ynput / ayon-houdini

Houdini addon for AYON
Apache License 2.0
11 stars 9 forks source link

Publish-BGEO-cache-using-the-existing-frames #154

Closed krishnaavril closed 4 days ago

krishnaavril commented 3 weeks ago

Changelog Description

Publish using the existing frames in houdini BGEO

Testing notes:

  1. When creating added boolean parameter for using the existing frames
  2. when publishing if the node is turned on-> it should skip the render and use the existing frames
  3. If the node is turned off-> It should render without any issues
krishnaavril commented 2 weeks ago

Hello!

After Roy's feedback, I changed the method and approach similar to the other logic, It is working fine for the below 3 methods:

  1. Local render
  2. Local use existing frames
  3. Farm render

Please validate the code @BigRoy @MustafaJafar

Can you guys hint where we need to tweak and skip the caching for the "Farm use existing frames" working method? Appreciate your time!

krishnaavril commented 2 weeks ago

The same technique must also be used in the vdb cache as well

MustafaJafar commented 5 days ago

Hello @krishnaavril, Many thank for your contribution. and in turn I've created another PR https://github.com/ynput/ayon-houdini/pull/180 to support publishing Bgeos with different render targets. It reuses some of the logic of render instances.

IMO, supporting more different render targets like "farm_no_render": "Use existing frames (farm)" should have a separate issue.

BTW, I think Use existing frames (farm) should only skips the Houdini job submission. i.e. only submit the AYON publish job. but let's use a dedicated issue for this discussion.