View Issue Details [ Jump to Notes ] | [ Print ] | ||||||||
ID | Project | Category | View Status | Date Submitted | Last Update | ||||
0009741 | VTK | (No Category) | public | 2009-10-20 09:43 | 2016-08-12 09:55 | ||||
Reporter | David Doria | ||||||||
Assigned To | Sean McBride | ||||||||
Priority | normal | Severity | minor | Reproducibility | always | ||||
Status | closed | Resolution | moved | ||||||
Platform | OS | OS Version | |||||||
Product Version | |||||||||
Target Version | Fixed in Version | ||||||||
Summary | 0009741: vtkPLYReader can't read files without faces | ||||||||
Description | Current 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. | ||||||||
Tags | No tags attached. | ||||||||
Project | |||||||||
Type | |||||||||
Attached Files | ![]() ![]() | ||||||||
Relationships | |
Relationships |
Notes | |
(0018367) Sean McBride (developer) 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 (reporter) 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 (developer) 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 (administrator) 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. |
Notes |
Issue History | |||
Date Modified | Username | Field | Change |
2009-10-20 09:43 | David Doria | New Issue | |
2009-10-20 09:43 | David Doria | File Added: vtkPLYReader.patch | |
2009-10-20 10:03 | Sean McBride | Status | backlog => tabled |
2009-10-20 10:03 | Sean McBride | Assigned To | => Sean McBride |
2009-11-06 17:14 | Sean McBride | Note Added: 0018367 | |
2009-12-05 20:08 | David Doria | File Added: vtkPLYReader2_Input.ply | |
2009-12-05 20:10 | David Doria | Note Added: 0018700 | |
2009-12-11 16:46 | Sean McBride | Note Added: 0018828 | |
2011-06-16 13:11 | Zack Galbreath | Category | => (No Category) |
2016-08-12 09:55 | Kitware Robot | Note Added: 0037111 | |
2016-08-12 09:55 | Kitware Robot | Status | expired => closed |
2016-08-12 09:55 | Kitware Robot | Resolution | open => moved |
Issue History |
Copyright © 2000 - 2018 MantisBT Team |