MantisBT - VTK
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0009741VTK(No Category)public2009-10-20 09:432016-08-12 09:55
ReporterDavid Doria 
Assigned ToSean McBride 
PrioritynormalSeverityminorReproducibilityalways
StatusclosedResolutionmoved 
PlatformOSOS Version
Product Version 
Target VersionFixed in Version 
Project
Type
Summary0009741: vtkPLYReader can't read files without faces
DescriptionCurrent vtkPLYReader does not allow reading files that do not contain faces (i.e. files with only points or points+normals). It simply quits with an error "Cannot read geometry". I modified this class to allow this type of file to be read, along with making the error output a bit more specific in the case that there actually is a problem. In the process I think the whole section that searches for properties became a bit clearer - as previously it was all embedded in a single if statement.
Steps To Reproduce
Additional Information
TagsNo tags attached.
Relationships
Attached Filespatch vtkPLYReader.patch (1,985) 2009-10-20 09:43
https://www.vtk.org/Bug/file/7571/vtkPLYReader.patch
? vtkPLYReader2_Input.ply (4,577,407) 2009-12-05 20:08
https://www.vtk.org/Bug/file/7705/vtkPLYReader2_Input.ply
Issue History
Date ModifiedUsernameFieldChange
2009-10-20 09:43David DoriaNew Issue
2009-10-20 09:43David DoriaFile Added: vtkPLYReader.patch
2009-10-20 10:03Sean McBrideStatusbacklog => tabled
2009-10-20 10:03Sean McBrideAssigned To => Sean McBride
2009-11-06 17:14Sean McBrideNote Added: 0018367
2009-12-05 20:08David DoriaFile Added: vtkPLYReader2_Input.ply
2009-12-05 20:10David DoriaNote Added: 0018700
2009-12-11 16:46Sean McBrideNote Added: 0018828
2011-06-16 13:11Zack GalbreathCategory => (No Category)
2016-08-12 09:55Kitware RobotNote Added: 0037111
2016-08-12 09:55Kitware RobotStatusexpired => closed
2016-08-12 09:55Kitware RobotResolutionopen => moved

Notes
(0018367)
Sean McBride   
2009-11-06 17:14   
I've confirmed that this patch A) does not break current unit tests B) does not break my application's use of PLY files.

David, can you attach a PLY file to this bug that did not work before but does after your patch?
(0018700)
David Doria   
2009-12-05 20:10   
I have attached a file which I believe does not work before and does work after the patch. It has another curiosity too, the colors are named diffuse_red, etc. instead of simply "red". I tried looking up the ply spec but it was pretty foggy - it looks like you can set diffuse_red, specular_red, etc, but it doesn't say if it is valid to simply have the main point colors named "diffuse_red" rather than "red". This would get pretty hard to handle generically I imagine - any ideas? I think this is a very common format that we should try to support as good as possible.
(0018828)
Sean McBride   
2009-12-11 16:46   
I have the impression that not many VTK users are using PLY actually.

Seems reasonable to check for either. Even if "diffuse_red" is not valid, it won't prevent valid files from opening. Could you attach your patch to support "diffuse" to this bug please?
(0037111)
Kitware Robot   
2016-08-12 09:55   
Resolving issue as `moved`.

This issue tracker is no longer used. Further discussion of this issue may take place in the current VTK Issues page linked in the banner at the top of this page.