View Issue Details [ Jump to Notes ] [ Print ]
IDProjectCategoryView StatusDate SubmittedLast Update
0013956ParaView(No Category)public2013-03-21 16:002015-01-11 08:29
ReporterUtkarsh Ayachit 
Assigned ToUtkarsh Ayachit 
PrioritynormalSeverityminorReproducibilityhave not tried
StatusclosedResolutionfixed 
PlatformOSOS Version
Product Version 
Target VersionFixed in Version4.0 
Summary0013956: Duplicate execution of certain filters in parallel
DescriptionStep to reproduce:

- client-server with 2 server procs.
- Open any CTH data file, Apply
- Create "Programmable Filter", change output type to MultiBlock, set the code to "print 'hello'"
- Hit apply

You'll see the "hello" being printed on the server side twice (once for each process). All's good so far.

Change the text to "hello again" and the hit apply.

This time we get 4 "hello again" printed (twice for each process)!!!
TagsNo tags attached.
ProjectSandia
Topic Name13965_paraview_ghost_cell_issue
Typeincorrect functionality
Attached Files

 Relationships

  Notes
(0030453)
Utkarsh Ayachit (administrator)
2013-03-21 16:03

 The problem is that the SelectionRepresentation doesn't ask for ghost cells while the GeometryRepresentation does ask for ghost cells (BUG 0011811). Since SelectionRepresentation is added before the GeometryRepresentation, it updates the pipeline gets updated with 0 and then 1 ghost-cell request, hence the double execution.

The GeometryRepresentation, which is the primary data-representation should always be updated before the Selection Representation. So to fix it, we changed the order in which the representations get added.
(0030454)
Utkarsh Ayachit (administrator)
2013-03-21 16:05

 commit 2f15eaca892c24406ec97aa9908dc2095cc35e07
Author: Utkarsh Ayachit <utkarsh.ayachit@kitware.com>
Date: Thu Mar 21 16:03:58 2013 -0400

    BUG 0013956. Fix duplicate execution of filters.
    
    The problem is that the SelectionRepresentation doesn't ask for ghost cells
    while the GeometryRepresentation does ask for ghost cells (BUG 0011811). Since
    SelectionRepresentation is added before the GeometryRepresentation, it updates
    the pipeline gets updated with 0 and then 1 ghost-cell request, hence the double
    execution.
    
    The GeometryRepresentation, which is the primary data-representation should
    always be updated before the Selection Representation. So to fix it, we changed
    the order in which the representations get added.
    
    Change-Id: Iaadf8e952a99fddc105d6e41d09f71e4a2240d7b
(0030457)
Utkarsh Ayachit (administrator)
2013-03-22 10:45

 SUMMARY
---------------------------------------------
Topics merged into master:
        0013949-web-enable-dependent-option
        13933_fix_python_missing_arrays
        13965_paraview_ghost_cell_issue
        catalyst_essentials_edition
        composite-data-display-attributes
(VTK) crosscompilation_fixes
        fix_fortran_mac_issues
        fix_gmv_test
        remove-legacy-large-int-float
        web_data_prober_rickshaw
(0030850)
Alan Scott (manager)
2013-05-23 15:08

 You have to be careful testing this one - sometimes clusters buffer their output.

Tested Linux, master, 8 remote servers.

 Issue History
Date Modified Username Field Change
2013-03-21 16:00 Utkarsh Ayachit New Issue
2013-03-21 16:03 Utkarsh Ayachit Note Added: 0030453
2013-03-21 16:05 Utkarsh Ayachit Topic Name => 13965_paraview_ghost_cell_issue
2013-03-21 16:05 Utkarsh Ayachit Note Added: 0030454
2013-03-21 16:05 Utkarsh Ayachit Status backlog => gatekeeper review
2013-03-21 16:05 Utkarsh Ayachit Fixed in Version => git-next
2013-03-21 16:05 Utkarsh Ayachit Resolution open => fixed
2013-03-21 16:05 Utkarsh Ayachit Assigned To => Utkarsh Ayachit
2013-03-22 10:45 Utkarsh Ayachit Fixed in Version git-next => git-master
2013-03-22 10:45 Utkarsh Ayachit Status gatekeeper review => customer review
2013-03-22 10:45 Utkarsh Ayachit Note Added: 0030457
2013-05-18 20:00 Utkarsh Ayachit Fixed in Version git-master => 4.0
2013-05-23 15:08 Alan Scott Note Added: 0030850
2013-05-23 15:08 Alan Scott Status customer review => closed
2015-01-11 08:27 Utkarsh Ayachit Source_changeset_attached => ParaView master 25fcd5b3
2015-01-11 08:27 Utkarsh Ayachit Source_changeset_attached => ParaView master 2f15eaca
2015-01-11 08:28 Kyle Lutz Source_changeset_attached => ParaView master 6785dbc6
2015-01-11 08:29 Utkarsh Ayachit Source_changeset_attached => ParaView master 5fd8a9be

Copyright © 2000 - 2018 MantisBT Team