View Issue Details [ Jump to Notes ] | [ Print ] | ||||||||
ID | Project | Category | View Status | Date Submitted | Last Update | ||||
0014647 | ParaView | (No Category) | public | 2014-03-21 15:37 | 2014-08-29 10:59 | ||||
Reporter | Clinton Stimpson | ||||||||
Assigned To | Ben Boeckel (Kitware) | ||||||||
Priority | normal | Severity | minor | Reproducibility | have not tried | ||||
Status | closed | Resolution | fixed | ||||||
Platform | OS | OS Version | |||||||
Product Version | |||||||||
Target Version | 4.2 | Fixed in Version | 4.2 | ||||||
Summary | 0014647: multiblock performance regression | ||||||||
Description | After updating to VTK 5.10.1, we eventually found our rendering performance had decreased about 3 times for some models. I narrowed it down to vtkCompositePolyDataMapper2::GetIsOpaque(). We have few mappers with hundreds or thousands of datasets. What used to take about .6 seconds to render a scene, now takes about 2 seconds. I can derive from the class and reimplement GetIsOpaque to return true, and my speed goes back to .6 seconds per render. I was using vtkCompositePolyDataMapper2 because a previous upgrade made many vtkActor objects slow. So I rearranged my code to use few actors. But now another upgrade has slowed that down as well. I also manually defer garbage collection when I can to get even more speed ups. Bottom line, VTK seems to get heaver and slower with each release. | ||||||||
Tags | No tags attached. | ||||||||
Project | Sandia | ||||||||
Topic Name | |||||||||
Type | performance | ||||||||
Attached Files | |||||||||
Relationships | |
Relationships |
Notes | |
(0032528) Utkarsh Ayachit (administrator) 2014-03-21 16:22 |
Good catch. Seems like we really need to a mechanism to mapper to avoid having to do data dependent checks on each render and only do those when data (or mapper) changes. |
(0032530) Utkarsh Ayachit (administrator) 2014-03-21 16:34 |
Moving to ParaView bug tracker since this greatly affects ParaView. |
(0033172) Utkarsh Ayachit (administrator) 2014-07-31 13:06 |
I believe, this is now fixed. Ben, can you confirm that the fix is in git/master. |
(0033173) Ben Boeckel (Kitware) (developer) 2014-07-31 13:40 |
Looks to be in to me: % git pull Current branch master is up to date. % git submodule update Submodule path 'VTK': checked out 'dace397909107915be46a67ff1003d1543597fd9' % grep LastOpaque ./VTK/Rendering/Core/vtkCompositePolyDataMapper2.cxx this->LastOpaqueCheckTime = 0; if (lastMTime <= this->LastOpaqueCheckTime) return this->LastOpaqueCheckValue; this->LastOpaqueCheckTime = lastMTime; this->LastOpaqueCheckValue = false; this->LastOpaqueCheckValue = false; this->LastOpaqueCheckValue = this->Superclass::GetIsOpaque(); return this->LastOpaqueCheckValue; |
(0033265) Alan Scott (manager) 2014-08-19 21:55 |
Clinton, I am closing this bug untested. If you think it was closed in error, please either reopen it. |
Notes |
Issue History | |||
Date Modified | Username | Field | Change |
2014-03-21 15:37 | Clinton Stimpson | New Issue | |
2014-03-21 16:21 | Utkarsh Ayachit | Assigned To | => Utkarsh Ayachit |
2014-03-21 16:21 | Utkarsh Ayachit | Status | backlog => tabled |
2014-03-21 16:22 | Utkarsh Ayachit | Note Added: 0032528 | |
2014-03-21 16:23 | Utkarsh Ayachit | Assigned To | Utkarsh Ayachit => |
2014-03-21 16:33 | Utkarsh Ayachit | Assigned To | => Ben Boeckel (Kitware) |
2014-03-21 16:34 | Utkarsh Ayachit | Project | VTK => ParaView |
2014-03-21 16:34 | Utkarsh Ayachit | Note Added: 0032530 | |
2014-03-21 16:35 | Utkarsh Ayachit | Status | tabled => todo |
2014-03-21 16:35 | Utkarsh Ayachit | Project | TBD => Sandia |
2014-05-12 11:54 | Utkarsh Ayachit | Target Version | => 4.2 |
2014-07-31 13:06 | Utkarsh Ayachit | Note Added: 0033172 | |
2014-07-31 13:40 | Ben Boeckel (Kitware) | Note Added: 0033173 | |
2014-07-31 13:41 | Utkarsh Ayachit | Status | todo => gatekeeper review |
2014-07-31 13:41 | Utkarsh Ayachit | Resolution | open => fixed |
2014-07-31 13:41 | Utkarsh Ayachit | Status | gatekeeper review => customer review |
2014-07-31 13:41 | Utkarsh Ayachit | Fixed in Version | => git-master |
2014-08-19 21:55 | Alan Scott | Note Added: 0033265 | |
2014-08-19 21:55 | Alan Scott | Status | customer review => closed |
2014-08-29 10:59 | Utkarsh Ayachit | Fixed in Version | git-master => 4.2 |
Issue History |
Copyright © 2000 - 2018 MantisBT Team |