MantisBT - ParaView
View Issue Details
0010045ParaViewFeaturepublic2009-12-13 09:462010-03-09 21:42
David Doria 
David Partyka 
normalminorN/A
closedfixed 
 
3.8 
0010045: Paraview does not have OBJ reader
VTK has an OBJ reader, so it should be trivial to add it to the list of readers for Paraview.
No tags attached.
bz2 vtkOBJReader.tar.bz2 (5,385) 2010-02-17 10:09
https://www.vtk.org/Bug/file/7875/vtkOBJReader.tar.bz2
bz2 vtkOBJReader-2.tar.bz2 (4,873) 2010-02-18 10:04
https://www.vtk.org/Bug/file/7877/vtkOBJReader-2.tar.bz2
Issue History
2009-12-13 09:46David DoriaNew Issue
2010-02-17 10:09olesenFile Added: vtkOBJReader.tar.bz2
2010-02-17 10:14olesenNote Added: 0019544
2010-02-18 10:04olesenFile Added: vtkOBJReader-2.tar.bz2
2010-02-18 10:06olesenNote Added: 0019549
2010-03-02 12:09David PartykaStatusbacklog => tabled
2010-03-02 12:09David PartykaAssigned To => David Partyka
2010-03-02 13:45David PartykaNote Added: 0019687
2010-03-02 13:45David PartykaStatustabled => @80@
2010-03-02 13:45David PartykaFixed in Version => 3.8
2010-03-02 13:45David PartykaResolutionopen => fixed
2010-03-09 21:42Alan ScottNote Added: 0019786
2010-03-09 21:42Alan ScottStatus@80@ => closed
2011-06-16 13:10Zack GalbreathCategoryFeature Request => Feature

Notes
(0019544)
olesen   
2010-02-17 10:14   
The change required to Servers/ServerManager/Resources/readers.xml is absolutely trivial. Although it was already posted on the mailing list, I've added it in the tar.bz2 file as well.

Since the vtkOBJReader class already exists, I would categorize the omission of the corresponding readers.xml entry as a bug rather than a feature request.

In the meantime, since it has been reported as a "feature request", I've also added in the missing support for handling line elements in the obj format.
(0019549)
olesen   
2010-02-18 10:06   
Note to vtkOBJReader-2.tar.bz2:

ENH: improve robustness of vtkOBJReader and error messages

- handle commands that do not start on column 1
- handle backslash-newline continuations
- handle OBJ 'p' (point) command
- more stringent checking of vertex count: min 3 for faces, 2 for lines
(0019687)
David Partyka   
2010-03-02 13:45   
Committed these patches to CVS and they passed the continuous dashboard. Thanks for the contribution!
(0019786)
Alan Scott   
2010-03-09 21:42   
Not tested. Please reopen this bug if it has not been fixed.