Toolchefs / atomsGaffer

Atoms Crowd extension for Gaffer
https://atoms.toolchefs.com
27 stars 15 forks source link

VariationReader weight attributes as primitive variables #25

Closed andrewkaufman closed 5 years ago

andrewkaufman commented 5 years ago

It would be good to move the joint* attributes to PrimitiveVariables on the meshes. Its not common to store such a large dataset as attributes in Gaffer, and its less flexible in terms of manipulation downstream.

There are known limitations preventing this currently (eg we'd need a fixed numbers of influences), but there are naming schemes that can accomodate these issues as well.

andrewkaufman commented 5 years ago

If we take this approach, we need to make sure the Generator removes those primvars so they never hit the renderer.