Issue 50 in cortex-vfx: Nuke MeshToNukeGeometryConverter can not handle FaceVarying "N"

[corte...@googlecode.com]

Status: New
Owner: ----

New issue 50 by ymco...@gmail.com: Nuke MeshToNukeGeometryConverter can not
handle FaceVarying "N"
http://code.google.com/p/cortex-vfx/issues/detail?id=50

What steps will reproduce the problem?
1. use FromMayaMeshConverter to create a MeshPrimitive, which has primitive
variable called "N" which type is FaceVarying.
2. write it out to disk cob file
3. use Reader to read this MeshPrimitive and use
MeshToNukeGeometryConverter to convert it to Nuke geometry and "N" is not
converted because the Nuke converter only handles Vertex type normals,
while not FaceVarying ones.

What is the expected output? What do you see instead?
The converter should handle both FaceVarying and Vertex "N", so basically
if "N" is FaceVarying, then get the average of all the normals attached to
the same vertex.

[corte...@googlecode.com]

Comment #1 on issue 50 by ymco...@gmail.com: Nuke

MeshToNukeGeometryConverter can not handle FaceVarying "N"
http://code.google.com/p/cortex-vfx/issues/detail?id=50

The Issue 50 and Issue 51 are same. Sorry about that.

Attachments:
50.patch 2.5 KB

[corte...@googlecode.com]

Comment #2 on issue 50 by thehaddo...@gmail.com: Nuke

MeshToNukeGeometryConverter can not handle FaceVarying "N"
http://code.google.com/p/cortex-vfx/issues/detail?id=50

Thanks for the patch - it'll be great to see support for FaceVarying
normals in Nuke. I'm not sure converting them to vertex normals is the
correct way to go though - did you consider using Group_Vertices when
creating the nuke attribute rather than Group_Points? There's an example of
that in MeshToNukeGeometryConverter where the uvs are being specified.
Perhaps the same approach can work for normals, and allow hard edges to be
preserved?

[corte...@googlecode.com]

Comment #3 on issue 50 by ymco...@gmail.com: Nuke

MeshToNukeGeometryConverter can not handle FaceVarying "N"
http://code.google.com/p/cortex-vfx/issues/detail?id=50

Oh, I see your idea. Please check out the new patch file. Now it is using
Group_Vertices instead of Group_Points, and the hard edges can be perserved.
I have tested it in our pipeline geo loader and it worked.

Attachments:
50.patch 2.0 KB