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

Maya: Redshift Volume Loader Implement update, remove, switch + fix vdb sequence support #3197

Merged
merged 2 commits into from
Jul 1, 2022

Conversation

BigRoy
Copy link
Collaborator

@BigRoy BigRoy commented May 17, 2022

Brief description

Improves Redshift Volume VDB loader to actually support updating and removing + also support single file or multiple frames in a sequence

Additional info

The Loader still loads the volume without! a material assigned by default. I was thinking of potentially auto-assigning a simple Volume Material but wasn't sure whether we wanted that in the same namespace + actually auto-remove that material when removing the volume as well.

For now you'll need to just manually assign the shader you'll want to render it with after loading the volume.

Testing notes:

  1. Load a VDB volume into Maya

Tested with Maya 2022 + Redshift 3.0.67

@BigRoy BigRoy self-assigned this May 17, 2022
@mkolar mkolar requested a review from m-u-r-p-h-y May 25, 2022 15:21
Copy link
Member

@m-u-r-p-h-y m-u-r-p-h-y left a comment

Choose a reason for hiding this comment

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

I'm probably doing something wrong but it seems like path resolution is not working in my case.

  • published vdb sequence from houdini
  • loaded vdb to Maya
>>> Finding presets for maya:load ...
// openpype.pipeline.load.utils : Running 'LoadVDBtoRedShift' on 'shVDB'
// Warning:  [Redshift] Failed to find volume file D:\PROJECTS\murphy_22101\shots\shVDB\publish\vdbcache\vdbcacheDefault\v001\22101_shVDB_vdbcacheDefault_v001.1001.vdb\22101_shVDB_vdbcacheDefault_v001.0001.vdb. Volume will be empty // 
// Warning:  [Redshift] Failed to find volume file D:\PROJECTS\murphy_22101\shots\shVDB\publish\vdbcache\vdbcacheDefault\v001\22101_shVDB_vdbcacheDefault_v001.1001.vdb\22101_shVDB_vdbcacheDefault_v001.1001.vdb. Volume will be empty // 
Good bye

published vdb sequence is
D:\PROJECTS\murphy_22101\shots\shVDB\publish\vdbcache\vdbcacheDefault\v001\22101_shVDB_vdbcacheDefault_v001.####.vdb

@BigRoy
Copy link
Collaborator Author

BigRoy commented May 27, 2022

How did you publish the VDB file? :) Just so I can try to test the same.

@m-u-r-p-h-y
Copy link
Member

  • Just downloaded sample vdb sequence
  • created FX task for the shot
  • started Houdini in FX context
  • in Houdini created loader to load vdb
  • created VDB instance in OP
  • published

@antirotor antirotor self-requested a review June 7, 2022 12:35
Copy link
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 happened to me too,

openpype/hosts/maya/plugins/load/load_vdb_to_redshift.py Outdated Show resolved Hide resolved
Copy link
Member

@m-u-r-p-h-y m-u-r-p-h-y left a comment

Choose a reason for hiding this comment

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

managing vdb caches works as expected.

  • loading
  • up versioning
  • removing

Maya2023 Redshift 3.5.04

image

could not test rendering as we ran out of trial :(

@mkolar mkolar added the type: enhancement Enhancements to existing functionality label Jul 1, 2022
Copy link
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.

I've just tested it. The volume loads correctly. Strange thing is that render doesn't match the points in the viewport - it seems like the rendered volume is scaled up, but that might be just my test data - as I can't see anything realted to scale in the code.

@antirotor antirotor merged commit 6e1cba8 into ynput:develop Jul 1, 2022
@BigRoy BigRoy deleted the maya_redshift_vdb_loader branch March 20, 2024 15:21
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
host: Maya type: enhancement Enhancements to existing functionality
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants