View Issue Details [ Jump to Notes ] [ Print ]
IDProjectCategoryView StatusDate SubmittedLast Update
0004062ParaView(No Category)public2006-11-22 17:232009-05-13 13:47
ReporterKen Moreland 
Assigned ToKen Moreland 
PrioritylowSeverityfeatureReproducibilityalways
StatusclosedResolutionfixed 
PlatformOSOS Version
Product Version 
Target Version3.6Fixed in Version 
Summary0004062: Extension independent reader chooser
DescriptionRight now, ParaView uses the file extension to determine what type of reader to use for a file. Unfortunatly, it is not always clear by the file extension what the format for the file is.

It would be much better to give readers the capability to determine for themselves whether they can read a file. For each reader, there could be a mechanism (which may be independent from the actual reader) that could introspect the file (read some header information) and determine if the file is definitely readable, possibly readable, or definitely not readable. If exactly one reader says it can definitely read the file, use that reader. Otherwise, present to the user the subset of readers that can definitely or possibly read the file.
TagsNo tags attached.
Project
Topic Name
Type
Attached Files

 Relationships

  Notes
(0011112)
Berk Geveci (administrator)
2008-04-07 16:13

 Take out all readers that can't read a file from the list that is presented to the user.
(0016249)
Ken Moreland (manager)
2009-04-29 13:32

 Implemented the basic functionality of removing all readers from the list that report they cannot read the file. Of course, the more reader support CanReadFile, the better this functionality works. I'm going to take a look at some of the readers to see if we can easily add CanReadFile before closing this off.

/cvsroot/ParaView3/ParaView3/Qt/Components/pqSelectReaderDialog.cxx,v <-- Components/pqSelectReaderDialog.cxx
new revision: 1.4; previous revision: 1.3
/cvsroot/ParaView3/ParaView3/Qt/Core/pqReaderFactory.cxx,v <-- Core/pqReaderFactory.cxx
new revision: 1.22; previous revision: 1.21
/cvsroot/ParaView3/ParaView3/Qt/Core/pqReaderFactory.h,v <-- Core/pqReaderFactory.h
new revision: 1.9; previous revision: 1.8
(0016251)
Ken Moreland (manager)
2009-04-29 15:06

 Also needed to change a couple of tests.

/cvsroot/ParaView3/ParaView3/Applications/Client/Testing/XML/Plot3DReader.xml,v <-- Testing/XML/Plot3DReader.xml
new revision: 1.19; previous revision: 1.18
/cvsroot/ParaView3/ParaView3/Applications/Client/Testing/XML/SelectReader.xml,v <-- Testing/XML/SelectReader.xml
new revision: 1.14; previous revision: 1.13
(0016272)
Ken Moreland (manager)
2009-05-01 09:28

 Added CanReadFile to several readers. This is probably about as good as it's going to get.

/cvsroot/ParaView3/ParaView3/Applications/Client/Testing/XML/Plot3DReader.xml,v
 <-- Applications/Client/Testing/XML/Plot3DReader.xml
new revision: 1.21; previous revision: 1.20
/cvsroot/ParaView3/ParaView3/Qt/Components/Resources/UI/pqSelectReaderDialog.ui,
v <-- Qt/Components/Resources/UI/pqSelectReaderDialog.ui
new revision: 1.3; previous revision: 1.2
/cvsroot/ParaView3/ParaView3/Servers/Common/vtkPVXMLParser.cxx,v <-- Servers/C
ommon/vtkPVXMLParser.cxx
new revision: 1.3; previous revision: 1.2
/cvsroot/ParaView3/ParaView3/Servers/Common/vtkPVXMLParser.h,v <-- Servers/Com
mon/vtkPVXMLParser.h
new revision: 1.2; previous revision: 1.1
/cvsroot/ParaView3/ParaView3/Servers/Filters/vtkPPhastaReader.cxx,v <-- Server
s/Filters/vtkPPhastaReader.cxx
new revision: 1.6; previous revision: 1.5
/cvsroot/ParaView3/ParaView3/Servers/Filters/vtkPPhastaReader.h,v <-- Servers/
Filters/vtkPPhastaReader.h
new revision: 1.4; previous revision: 1.3
/cvsroot/ParaView3/ParaView3/Servers/Filters/vtkVRMLSource.cxx,v <-- Servers/F
ilters/vtkVRMLSource.cxx
new revision: 1.12; previous revision: 1.11
/cvsroot/ParaView3/ParaView3/Servers/Filters/vtkVRMLSource.h,v <-- Servers/Fil
ters/vtkVRMLSource.h
new revision: 1.6; previous revision: 1.5
/cvsroot/ParaView3/ParaView3/VTK/Hybrid/vtkFacetReader.cxx,v <-- VTK/Hybrid/vt
kFacetReader.cxx
new revision: 1.6; previous revision: 1.5
/cvsroot/ParaView3/ParaView3/VTK/Hybrid/vtkFacetReader.h,v <-- VTK/Hybrid/vtkF
acetReader.h
new revision: 1.5; previous revision: 1.4
/cvsroot/ParaView3/ParaView3/VTK/IO/vtkBYUReader.cxx,v <-- VTK/IO/vtkBYUReader
.cxx
new revision: 1.52; previous revision: 1.51
/cvsroot/ParaView3/ParaView3/VTK/IO/vtkBYUReader.h,v <-- VTK/IO/vtkBYUReader.h
new revision: 1.47; previous revision: 1.46
/cvsroot/ParaView3/ParaView3/VTK/IO/vtkGenericEnSightReader.cxx,v <-- VTK/IO/v
tkGenericEnSightReader.cxx
new revision: 1.87; previous revision: 1.86
/cvsroot/ParaView3/ParaView3/VTK/IO/vtkGenericEnSightReader.h,v <-- VTK/IO/vtk
GenericEnSightReader.h
new revision: 1.33; previous revision: 1.32
/cvsroot/ParaView3/ParaView3/VTK/IO/vtkNetCDFCOARDSReader.cxx,v <-- VTK/IO/vtk
NetCDFCOARDSReader.cxx
new revision: 1.2; previous revision: 1.1
/cvsroot/ParaView3/ParaView3/VTK/IO/vtkNetCDFCOARDSReader.h,v <-- VTK/IO/vtkNe
tCDFCOARDSReader.h
new revision: 1.2; previous revision: 1.1
/cvsroot/ParaView3/ParaView3/VTK/IO/vtkPLYReader.cxx,v <-- VTK/IO/vtkPLYReader
.cxx
new revision: 1.21; previous revision: 1.20
/cvsroot/ParaView3/ParaView3/VTK/IO/vtkPLYReader.h,v <-- VTK/IO/vtkPLYReader.h
new revision: 1.10; previous revision: 1.9
(0016321)
Alan Scott (manager)
2009-05-06 16:54

 This seems to work for the few files I have tested. I am closing this, with the understanding that if anyone starts reporting errors we will re-open it.

Tested local client.

 Issue History
Date Modified Username Field Change
2008-04-07 16:13 Berk Geveci Note Added: 0011112
2008-04-07 16:13 Berk Geveci Category => 3.6
2009-02-16 14:41 Ken Moreland Assigned To Berk Geveci => Burlen
2009-04-29 13:29 Ken Moreland Assigned To Burlen => Ken Moreland
2009-04-29 13:32 Ken Moreland Note Added: 0016249
2009-04-29 15:06 Ken Moreland Note Added: 0016251
2009-05-01 09:28 Ken Moreland Note Added: 0016272
2009-05-01 09:28 Ken Moreland Status tabled => @80@
2009-05-01 09:28 Ken Moreland Resolution open => fixed
2009-05-06 16:54 Alan Scott Note Added: 0016321
2009-05-06 16:54 Alan Scott Status @80@ => closed
2009-05-13 13:44 Utkarsh Ayachit Target Version => 3.6
2011-06-16 13:10 Zack Galbreath Category => (No Category)

Copyright © 2000 - 2018 MantisBT Team