Skip to content

LiveScene: Support point-based geometry in readObject#1532

Merged
ivanimanishi merged 3 commits into
ImageEngine:RB-10.5from
jaec-ie:39021_Caribou_Nuke_Deep_to_Points_does_not_translate_to_Gaffer
Jun 1, 2026
Merged

LiveScene: Support point-based geometry in readObject#1532
ivanimanishi merged 3 commits into
ImageEngine:RB-10.5from
jaec-ie:39021_Caribou_Nuke_Deep_to_Points_does_not_translate_to_Gaffer

Conversation

@jaec-ie
Copy link
Copy Markdown

@jaec-ie jaec-ie commented May 25, 2026

Generally describe what this PR will do, and why it is needed.

  • Nuke's DeepToPoints node outputs eParticles type primitives, which LiveScene::readObject() did not recognize as point geometry. The previous check only matched eParticlesSprite, causing DeepToPoints output to fall through to MeshFromNuke. eParticlesSprite represent a collection of particles in one primitive, but output of DeepToPoints could be N primitives, now checking if primitives > 0.

Related Issues

Dependencies

Breaking Changes

  • IECoreNuke LiveScene: Fixed point clouds from DeepToPoints to be supported. The geometry was incorrectly treated as a mesh instead of points.

Checklist

  • I have read the contribution guidelines.
  • I have updated the documentation, if applicable.
  • I have tested my change(s) in the test suite, and added new test cases where necessary.
  • My code follows the Cortex project's prevailing coding style and conventions.

jaec-ie added 3 commits May 25, 2026 11:39
The previous check only handled eParticlesSprite with exactly one
primitive, which missed point geometry from nodes like DeepToPoints
that output eParticles primitives. This caused the points to be routed through MeshFromNuke, producing a broken empty mesh that didn't render in Caribou/Gaffer.
@ivanimanishi ivanimanishi merged commit 0e92f04 into ImageEngine:RB-10.5 Jun 1, 2026
3 of 5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants