[vtk-developers] vtkPLYReader patch to allow points only

[David Doria]

On Tue, Oct 20, 2009 at 1:03 PM, Berk Geveci <berk.geveci at kitware.com>wrote:

> I assigned it to Sean. It is up to the developer/manager to assign a
> particular bug.
>
> -berk
>

While I was looking at this patch, I also found some PLY files I had that
still could not be read. The problem is that the header is as follows:

ply
format ascii 1.0
element vertex 182323
property float x
property float y
property float z
property float nx
property float ny
property float nz
property uchar diffuse_red
property uchar diffuse_green
property uchar diffuse_blue

If I change diffuse_red, diffuse_green, diffuse_blue to simply red, green,
blue, everything works again. There doesn't seem to be a hard fast "rule"
about what to name the colors... should we check for any vertex property
that resembles a color array name and read it into RGBPoints? Or should we
create a DiffuseRGBPoints?

Should we read the normals (nx,ny,nz)?

I think any time that data is skipped/ignored we should throw a warning. If
I was the user and I loaded the data and my normals were missing without any
errors/warnings, I'd be very confused. If I got a warning something like
"vtkPLYReader does not support vertex property diffuse_red", etc I would at
least know that there is a known lack of support for something I am trying
to read.

Things are are/aren't supported should also be VERY clearly explained in the
documentation.

Thoughts/comments?

Thanks,

David
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://public.kitware.com/pipermail/vtk-developers/attachments/20091108/480846c7/attachment.html>