Toolchefs / atomsGaffer

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

Generator not respecting the incoming points at all #28

Open andrewkaufman opened 5 years ago

andrewkaufman commented 5 years ago

If I transform the points between the CrowdReader and the CrowdGenerator, I expect that to move the agents origins, but it is not currently happening.

andrewkaufman commented 5 years ago

I suspect the points are currently being used just as a "visualization" of the underlying metadata stashed on the atoms:agents attribute. This should not be the case, the points should be in the driver's seat, as that is what users interact with and will manipulate with other gaffer tools (eg frustum cull).

Unless we have a reason to maintain 2 copies of the data, we should remove the metadata section from the atoms:agents attribute, and update the CrowdGenerator to read that data from the points directly.