View Issue Details [ Jump to Notes ] [ Print ]
IDProjectCategoryView StatusDate SubmittedLast Update
0012546ParaView(No Category)public2011-08-24 07:392012-02-08 17:27
ReporterPaul Edwards 
Assigned ToUtkarsh Ayachit 
PrioritynormalSeverityminorReproducibilityhave not tried
StatusclosedResolutionfixed 
PlatformOSOS Version
Product VersionDevelopment 
Target Version3.12Fixed in Version3.12 
Summary0012546: RequestData called twice in parallel
DescriptionFilters have their RequestData called twice when using a parallel server. This happens when the filter is first added - subsequent "Apply"s only called it once.

To reproduce:
First run a parallel server, attach a debugger and set a breakpoint in "vtkArrayCalculator::RequestData"
Create a Box source
Use calculator to add an array, e.g. "coordsX"
Click Apply [ RequestData gets called twice ]
Update arrat , e.g. "coordsX + 0"
Click Apply [ RequestData only gets called once ]
TagsNo tags attached.
ProjectKitware
Topic Name12546_fix_multiple_executes
Typeincorrect functionality
Attached Files

 Relationships
related to 0011811closedUtkarsh Ayachit Internal surfaces visible after D3 when opacity < 1 

  Notes
(0027417)
Utkarsh Ayachit (administrator)
2011-08-24 09:02
edited on: 2011-08-24 09:02

 The fix for BUG 0011811 is what's causing this issue.
(0027425)
Utkarsh Ayachit (administrator)
2011-08-24 15:32

 commit cb9bc043970d96dc5f36ca9b7e06c74d0b78eda6
Author: Utkarsh Ayachit <utkarsh.ayachit@kitware.com>
Date: Wed Aug 24 15:31:22 2011 -0400

    Fixed BUG 0012546. RequestData was called twice in parallel.
    
    The issue was introduced by fix for BUG 0011811. We were now requesting
    ghost-cells in representations to use the ghost-cell information when available
    during rendering. However that meant that all the request changed since previous
    update on the pipeline, since the GUI would call UpdatePipeline() when creating
    the representation but without the extra ghost cells.
    
    Fixed this issue by making the GUI also ask for ghost-cells in a similar fashion
    to vtkGeometryRepresentation. This was done by updating
    vtkSISourceProxy::UpdatePipeline to request ghost cells using the logic in
    vtkGeometryRepresentation.
    
    vtkGeometryRepresentation was checking for data-type when deciding whether to
    ask for ghost cells. That meant that vtkSISourceProxy couldn't use it since the
    algorithm had not produced any data yet. Anyways, the datatype check was
    unnecessary since we were using the presence of WHOLE_EXTENT() key to exclude
    any structure data pipelines. So removed that type check. I verified that it
    doesn't lead to any wrong ghost cell requests when contouring, for example.
(0028183)
Utkarsh Ayachit (administrator)
2012-02-08 17:27

 Closing bugs sitting in "customer review" for a long time.

 Issue History
Date Modified Username Field Change
2011-08-24 07:39 Paul Edwards New Issue
2011-08-24 08:58 Utkarsh Ayachit Assigned To => Utkarsh Ayachit
2011-08-24 08:59 Utkarsh Ayachit Target Version => 3.12
2011-08-24 09:02 Utkarsh Ayachit Relationship added related to 0011811
2011-08-24 09:02 Utkarsh Ayachit Note Added: 0027417
2011-08-24 09:02 Utkarsh Ayachit Note Edited: 0027417
2011-08-24 15:32 Utkarsh Ayachit Topic Name => 12546_fix_multiple_executes
2011-08-24 15:32 Utkarsh Ayachit Note Added: 0027425
2011-08-24 15:32 Utkarsh Ayachit Status backlog => gatekeeper review
2011-08-24 15:32 Utkarsh Ayachit Fixed in Version => 3.12
2011-08-24 15:32 Utkarsh Ayachit Resolution open => fixed
2011-08-24 15:36 Utkarsh Ayachit Project TBD => Kitware
2011-08-26 15:57 Utkarsh Ayachit Status gatekeeper review => customer review
2012-02-08 17:27 Utkarsh Ayachit Status customer review => closed
2012-02-08 17:27 Utkarsh Ayachit Note Added: 0028183

Copyright © 2000 - 2018 MantisBT Team