Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Consolidate primitive generators #97

Closed
bentoogood opened this issue Apr 2, 2013 · 4 comments · Fixed by #334
Closed

Consolidate primitive generators #97

bentoogood opened this issue Apr 2, 2013 · 4 comments · Fixed by #334
Assignees
Labels
bug Issues representing bugs docs New user documentation should be created or existing documentation needs to be updated scene Issues with GafferScene

Comments

@bentoogood
Copy link
Contributor

currently 'Plane' is available from Scene->Source->Primitive->Plane and generates a scene graph.

'Sphere' is available from Cortex->Primitive->Sphere and generates an object that requires promotion to a scene entity.

consolidate the approach to primitive generation in some sensible way.

@ghost ghost assigned andrewkaufman May 25, 2013
@bentoogood
Copy link
Contributor Author

We need some primitives for the example scripts in the docs.

This would also provide a useful entry point for developing nodes.

So move up to next week.

Agreement is to add extra primitives alongside the plane we have currently and remove the cortex primitive ops from the gaffer install.

Primitives to add:

  • sphere
  • cube
  • cone

@andrewkaufman
Copy link
Contributor

Is the sphere meant to be a Cortex SpherePrimitive (single point with radius and theta) or a Mesh with controls for horizontal/vertical divisions?

@bentoogood
Copy link
Contributor Author

B

@johnhaddon
Copy link
Member

For now I think just a SpherePrimitive will do - that's what the node you're replacing did (the Cortex level one). Later I think it'd be nice for a "type" plug to specify whether it's a sphere-sphere, a mesh-sphere, a geodesic-sphere or a nurbs-sphere...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Issues representing bugs docs New user documentation should be created or existing documentation needs to be updated scene Issues with GafferScene
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants