MantisBT - ParaView
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0005331ParaView(No Category)public2007-07-11 17:222007-09-17 19:40
ReporterJon Goldman 
Assigned ToDave DeMarle 
PriorityurgentSeveritymajorReproducibilityalways
StatusclosedResolutionfixed 
PlatformOSOS Version
Product Version 
Target VersionFixed in Version 
Project
Topic Name
Type
Summary0005331: Extract CTH Parts filter very slow - Too many ERROR messages from vtkExtractCTHPart.cxx, line 750
DescriptionOpen a CTH file (I tried 3D AMR).

Apply Extract CTH Parts filter and get tons of messages, which significantly slows the filter operation. This is especially bad with large data:

ERROR: In /usr/local/viz/paraview/src/ParaView/3.0.2/ParaView3/VTK/Parallel/vtkExtractCTHPart.cxx, line 750
vtkExtractCTHPart (0xc3d7d68): Volume fraction arrays are different type. They should all be float, double, or unsigned char
Steps To Reproduce
Additional Information
TagsNo tags attached.
Relationships
related to 0012735closed Utkarsh Ayachit ExtractCTHParts panel is changing property values. 
Attached Files
Issue History
Date ModifiedUsernameFieldChange
2007-08-13 14:11Berk GeveciNote Deleted: 0008134
2007-08-23 17:10Berk GeveciCategory => 3.2
2007-09-03 10:32Berk GeveciAssigned ToBerk Geveci => Dave DeMarle
2007-09-14 15:36Dave DeMarleNote Added: 0008992
2007-09-14 15:36Dave DeMarleStatustabled => @80@
2007-09-14 15:36Dave DeMarleResolutionopen => fixed
2007-09-17 19:40Alan ScottStatus@80@ => closed
2011-06-16 13:10Zack GalbreathCategory => (No Category)
2011-11-17 10:36Utkarsh AyachitRelationship addedrelated to 0012735

Notes
(0008057)
Jon Goldman   
2007-07-12 17:13   
Dave Karelitz had this to say, which might be important for whoever works on this issue:

"That actually seems like a bug in the reader. The reader's supposed to convert all the volume fraction arrays to unsigned char's. If it's only doing that for some, then that would introduce a tremendous amount of extra memory overhead."
(0008138)
user521   
2007-07-27 12:23   
I think that the problem is that the SpyPlot reader makes sure that all of the arrays with names that start with "Material volume fraction" are either float or unsigned char, but the CTH Extract parts filter allows you to use any of the cell arrays. I will change the filter so that it only allows the user to choose from the arrays that start with "Material volume fraction" unless anyone objects. A more involved solution is to change the reader to indpendently decide what cell arrays are read and what ones are marked as volume fraction arrays, and then have the filter choose from any of the marked ones.
(0008992)
Dave DeMarle   
2007-09-14 15:36   
I committed this change to the Server API and the GUI to enforce the requirement that all material arrays be of the same type.

/cvsroot/ParaView3/ParaView3/Qt/Components/CMakeLists.txt,v <-- CMakeLists.txtnew revision: 1.121; previous revision: 1.120
/cvsroot/ParaView3/ParaView3/Qt/Components/pqExtractCTHPartsPanel.h,v <-- pqExtractCTHPartsPanel.h
initial revision: 1.1
/cvsroot/ParaView3/ParaView3/Qt/Components/pqExtractCTHPartsPanel.cxx,v <-- pqExtractCTHPartsPanel.cxx
initial revision: 1.1
/cvsroot/ParaView3/ParaView3/Qt/Components/pqObjectInspectorWidget.cxx,v <-- pqObjectInspectorWidget.cxx
new revision: 1.54; previous revision: 1.53
/cvsroot/ParaView3/ParaView3/Servers/ServerManager/vtkSMArrayListDomain.h,v <-- vtkSMArrayListDomain.h
new revision: 1.11; previous revision: 1.10
/cvsroot/ParaView3/ParaView3/Servers/ServerManager/vtkSMArrayListDomain.cxx,v <-- vtkSMArrayListDomain.cxx
new revision: 1.12; previous revision: 1.11
/cvsroot/ParaView3/ParaView3/Servers/ServerManager/Resources/filters.xml,v <-- filters.xml
new revision: 1.262; previous revision: 1.261
/cvsroot/ParaView3/ParaView3/Servers/ServerManager/Testing/SMStates/SPCTH2.pvsm,v <-- SPCTH2.pvsm
new revision: 1.16; previous revision: 1.15
/cvsroot/ParaView3/ParaView3/VTK/Parallel/vtkExtractCTHPart.h,v <-- vtkExtractCTHPart.h
new revision: 1.9; previous revision: 1.8
/cvsroot/ParaView3/ParaView3/VTK/Parallel/vtkExtractCTHPart.cxx,v <-- vtkExtractCTHPart.cxx
new revision: 1.25; previous revision: 1.24