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

Enhancement: Houdini publish existing caches/frames #5767

Closed
2 tasks done
MustafaJafar opened this issue Oct 13, 2023 · 4 comments
Closed
2 tasks done

Enhancement: Houdini publish existing caches/frames #5767

MustafaJafar opened this issue Oct 13, 2023 · 4 comments
Labels
host: Houdini type: enhancement Enhancements to existing functionality

Comments

@MustafaJafar
Copy link
Contributor

MustafaJafar commented Oct 13, 2023

Is there an existing issue for this?

  • I have searched the existing issues.

Please describe the feature you have in mind and explain what the current shortcomings are?

Problem description:
Houdini simulations need many iterations so artists can keep adding tweaks and once they are comfortable they will consider publishing.
Also, simulations are CPU intensive so most of times it will be a problem to re-render or re-cache to just publish (register data to AYON/OpenPype DB)
However, publishing is not just about registering data to the DB, it's also about running validations to ensure these data imply to studio's specifications.

one solution that we came up with in community to have a toggle on the ROP node that skips the render action so it will just validate these data before publishing.

in this screen shot, alembic pointcache rop is reading a file read sop node.
image

How would you imagine the implementation of the feature?

  1. add use_existing checkbox in creator plugin

  2. use it in extractors so, it skips rendering

if not instance.data["use_existing"]:
  render_rop(ropnode)

Are there any labels you wish to add?

  • I have added the relevant labels to the enhancement request.

Describe alternatives you've considered:

another solution would be to have a universal publish node that's only purpose to publish.

Additional context:

Feel free to add your ideas and suggestions.

[cuID:OP-7229]

@MustafaJafar MustafaJafar added type: enhancement Enhancements to existing functionality host: Houdini labels Oct 13, 2023
@BigRoy
Copy link
Collaborator

BigRoy commented Oct 13, 2023

Instead of a Use existing checkbox I think we're better of making an ENUM attribute similar to Fusion and Nuke does for renders. There you have the option defined here to:

  • Use Existing Frames
  • Local Machine Render
  • Farm Rendering
  • Use Existing Frames - farm

Having an enum makes it easier to understand what's going on and will also make sure that "Existing frames" + "Local or farm" etc can be tackled in unison.

@krishnaavril
Copy link

Need these options same for Review family as well please, we can take advantage of rendering opengl in deadline machines, and publish existing frames, currently there is a limitation of publishing locally, Huge size caches are eating so much of time on this limitation.

@MustafaJafar
Copy link
Contributor Author

I got it! Totally make sense.

Here's a screenshot for reference
image

@MustafaJafar MustafaJafar linked a pull request Nov 2, 2023 that will close this issue
5 tasks
@MustafaJafar
Copy link
Contributor Author

MustafaJafar commented Jul 4, 2024

It's implemented with render but not cache
re-created on ayon ynput/ayon-houdini#19

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
host: Houdini type: enhancement Enhancements to existing functionality
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants