Ticket #3271 (closed: fixed)
vtkThresholdingHexahedronFactory: parallelize/speed up
Reported by: | Janik Zikovsky | Owned by: | Janik Zikovsky |
---|---|---|---|
Priority: | major | Milestone: | Iteration 30 |
Component: | Mantid | Keywords: | |
Cc: | Blocked By: | ||
Blocking: | Tester: | Stuart Campbell |
Description
Use multiple cores (if possible) for optimal speed in creating a visualization data set.
Change History
comment:2 Changed 9 years ago by Janik Zikovsky
(In [12935]) Corrected Q in MakeDiffractionEventWorkspace (to not have 2*pi and to be q = ki-kf, it was kf-ki). Added a Paraview plugin, PeaksReader, to load .peaks file to visualize them. Needed to add a link to the Crystal library. Refs #3232. Added a vtkPeakMarkerFactory to render single-crystal peaks (as points). Not totally finished. Refs #3232. Made vtkDataSetFactory::initialise() take a Workspace_sptr instead of a IMDWorkspace_sptr. This allowed the visualization of a PeaksWorkspace. Still needs tests. Made the MultiDimensionalDbPresenter::getMesh() method return a vtkDataSet* (instead of vtkUnstructuredGrid*) to be more general. Attempted to speed up/parallelize vtkThresholdingHexahedronFactory without much improvement so far. Refs #3271. Made a MockObjects.hh file to reuse mock objects code in VatesAPITest. Fixes #3291. Sped up setUp of EventListTestPerformance; also InstrumentRayTracerTestPerformance. Refs #3217.
comment:17 Changed 9 years ago by Owen Arnold
In [13518]:
comment:18 Changed 9 years ago by Owen Arnold
In [13519]:
comment:19 Changed 9 years ago by Owen Arnold
In [13520]:
comment:20 Changed 9 years ago by Janik Zikovsky
- Summary changed from vtkThresholdingHexahedronFactory: parallelize to vtkThresholdingHexahedronFactory: parallelize/speed up
comment:21 Changed 9 years ago by Janik Zikovsky
In [13610]:
comment:22 Changed 9 years ago by Janik Zikovsky
In [13714]:
comment:23 Changed 9 years ago by Janik Zikovsky
- Status changed from accepted to verify
- Resolution set to fixed
I think most that can be done within the factory is now done. The larger slow downs are in other parts of paraview.
comment:24 Changed 9 years ago by Stuart Campbell
- Status changed from verify to verifying
- Tester set to Stuart Campbell
comment:25 Changed 9 years ago by Stuart Campbell
- Status changed from verifying to closed
Well trying with CNCS_7860 data set and setting some ridiculous number of bins... it seems quite quick (200x200x200 = 0.028s)
comment:26 Changed 5 years ago by Stuart Campbell
This ticket has been transferred to github issue 4118