View Issue Details [ Jump to Notes ] [ Print ]
IDProjectCategoryView StatusDate SubmittedLast Update
0015216ParaView(No Category)public2014-12-18 17:172015-08-11 14:40
ReporterAlan Scott 
Assigned ToUtkarsh Ayachit 
PrioritynormalSeverityminorReproducibilityhave not tried
StatusclosedResolutionno change required 
PlatformOSOS Version
Product Version4.2 
Target Version4.4Fixed in Version 
Summary0015216: CGNS reader is dreadfully slow
DescriptionThe CGNS reader is painfully, reduculiously, dreadfully slow. It takes about 30 seconds to read the header for a toy dataset, and with this dataset the following line gets written out a few hundred times.

*** Warning:The file being read is more recent that the CGNS library used ***



Figure out what is going on, and fix it. Further, turn off the warning for production builds.

File available upon request. File name is aero3d.cgns.
TagsNo tags attached.
ProjectSandia
Topic Name
Typeperformance
Attached Files

 Relationships

  Notes
(0034304)
Alan Scott (manager)
2015-03-13 18:13

 I have a user that tried to use the CGNS, and found it to be unusable...
(0034312)
Mickael PHILIT (reporter)
2015-03-16 15:17
edited on: 2015-03-16 15:17

 It would be interesting to try with the CGNS plugin included in the master branch.
(0034316)
Alan Scott (manager)
2015-03-18 15:57

 I just looked, and couldn't find a CGNS plugin. I looked in 4.3.1 Kitware windows, my own personal Linux 4.3.1 build, my personal Linux build of master, and looked in the Build/ Plugins directory. How do I get the CGNS plugin? Is there a magic CMake switch?
(0034317)
Mickael PHILIT (reporter)
2015-03-18 18:14

 The cmake switch is off by default. You may want to use the latest/current development master to get a working reader. (Until recently, it was available on the superbuild dashboard)
(0034695)
Utkarsh Ayachit (administrator)
2015-07-08 09:55

 Folks, the CGNS plugin was merged into ParaView proper a little while ago.
See:
https://gitlab.kitware.com/paraview/paraview/commit/f011a67942e7ff5a04e1ecbc98427b749fdb5002 [^]

So long as you build with CGNS support enabled, ParaView will indeed use the newer CGNS reader that Mickael contributed.
(0034696)
Utkarsh Ayachit (administrator)
2015-07-08 10:05

 Oh, btw, there's a new flag PARAVIEW_ENABLE_CGNS that you need to turn on to enable CGNS support in ParaView.
(0034899)
Utkarsh Ayachit (administrator)
2015-08-05 21:41

 Please reopen if the new CGNS implementation is still slow.
(0034957)
Alan Scott (manager)
2015-08-11 14:40

 Much, much better.

Tested Kitware's nightly build from 8/10/2015. Windows. Local server.

 Issue History
Date Modified Username Field Change
2014-12-18 17:17 Alan Scott New Issue
2015-03-13 18:13 Alan Scott Note Added: 0034304
2015-03-13 18:13 Alan Scott Target Version => 4.4
2015-03-16 15:17 Mickael PHILIT Note Added: 0034312
2015-03-16 15:17 Mickael PHILIT Note Edited: 0034312
2015-03-18 15:57 Alan Scott Note Added: 0034316
2015-03-18 18:14 Mickael PHILIT Note Added: 0034317
2015-07-08 09:55 Utkarsh Ayachit Note Added: 0034695
2015-07-08 10:05 Utkarsh Ayachit Note Added: 0034696
2015-07-08 16:36 Utkarsh Ayachit Status backlog => todo
2015-08-05 21:41 Utkarsh Ayachit Note Added: 0034899
2015-08-05 21:41 Utkarsh Ayachit Status todo => customer review
2015-08-05 21:41 Utkarsh Ayachit Resolution open => fixed
2015-08-05 21:41 Utkarsh Ayachit Assigned To => Utkarsh Ayachit
2015-08-05 21:41 Utkarsh Ayachit Resolution fixed => no change required
2015-08-11 14:40 Alan Scott Note Added: 0034957
2015-08-11 14:40 Alan Scott Status customer review => closed

Copyright © 2000 - 2018 MantisBT Team