View Issue Details Jump to Notes ] Print ]
IDProjectCategoryView StatusDate SubmittedLast Update
0013933ParaView(No Category)public2013-03-09 21:442015-01-11 08:27
ReporterUtkarsh Ayachit 
Assigned ToUtkarsh Ayachit 
PrioritynormalSeverityminorReproducibilityhave not tried
StatusclosedResolutionfixed 
PlatformOSOS Version
Product Version 
Target VersionFixed in Version4.0 
Summary0013933: InSitu Python scripts should not check for valid array names
DescriptionInSitu Python scripts should not check for valid array names. There are several occasions when we raise errors if the user tried to use a non-present array. This should not happen in insitu mode.
TagsNo tags attached.
ProjectSandia-Catalyst
Topic Name13933_fix_python_missing_arrays
Typeincorrect functionality
Attached Files

 Relationships

  Notes
(0030448)
Utkarsh Ayachit (administrator)
2013-03-18 16:40

commit 461501c1d0f3af7bd8d4193bfaf5b8336755449a
Author: Utkarsh Ayachit <utkarsh.ayachit@kitware.com>
Date: Mon Mar 18 16:35:16 2013 -0400

    BUG 0013933. Allow non-present arrays in Python scripts.
    
    "ColorArrayName" property no longer checks the validity of the array specified
    so long as the array is specified as a tuple (<attribute>, <name>). It will
    check (and fail if array not found) when the user simply specifies the array
    name as a string.
    
    Updated Python trace to ensure that for "ColorArrayName" we indeed trace out a
    tuple rather than just the arrayname.
    
    Also cleaned up vtkPVOptions to expose publicly accessed enum. Updated
    servermanager to use the enum rather than the hex values.
    
    Change-Id: I219e83ba58b825e40bb4c5a32b69207ff57d9d3c
(0030449)
Utkarsh Ayachit (administrator)
2013-03-18 16:48

ColorArrayName can be set in two ways e.g.,

# ---(a)----
foo.ColorArrayName = "RTData"

# ---(b)----
foo.ColorArrayName = ("POINT_DATA", "RTData")

If (a) is used, ParaView Python will need to check the available arrays to determine the attribute type. Hence we raise test for arrays and raise exceptions if the array is not found in the local data information as earlier.

If (b) is used, ParaView Python will no longer check the available arrays and simply update the property since all information is provided to the call.

CoProcessing scripts should use (b) form to overcome errors due to locally missing arrays. However form (a) also avoid raising of errors in SymmetricMode (which is ON for CoProcessing).

Also updated Python tracing/state saving mechanism to save the (<attribute>,<name>) tuple for ColorArrayName rather than just the array name.
(0030450)
Utkarsh Ayachit (administrator)
2013-03-18 16:50

commit 617617f38b1773f5912ceb9c4f338767209b8166
Author: Utkarsh Ayachit <utkarsh.ayachit@kitware.com>
Date: Mon Mar 18 16:48:35 2013 -0400

    BUG 0013933. Ensure ColorArrayName doesn't raise exceptions.
    
    Ensure that older scripts that simply set arraynames for ColorArrayName property
    don't raise exceptions in symmetric-mpi mode i.e. coprocessing mode.
    
    Change-Id: I4b15275ac54ced7b5facb969f20f83fb300b346f
(0030458)
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
(0032064)
Alan Scott (manager)
2013-12-24 19:39

Old customer review bug. Closing untested.

 Issue History
Date Modified Username Field Change
2013-03-09 21:44 Utkarsh Ayachit New Issue
2013-03-09 21:44 Utkarsh Ayachit Assigned To => Utkarsh Ayachit
2013-03-09 21:45 Utkarsh Ayachit Status backlog => todo
2013-03-18 16:40 Utkarsh Ayachit Topic Name => 13933_fix_python_missing_arrays
2013-03-18 16:40 Utkarsh Ayachit Note Added: 0030448
2013-03-18 16:40 Utkarsh Ayachit Status todo => gatekeeper review
2013-03-18 16:40 Utkarsh Ayachit Fixed in Version => git-next
2013-03-18 16:40 Utkarsh Ayachit Resolution open => fixed
2013-03-18 16:48 Utkarsh Ayachit Note Added: 0030449
2013-03-18 16:50 Utkarsh Ayachit Note Added: 0030450
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: 0030458
2013-05-18 20:00 Utkarsh Ayachit Fixed in Version git-master => 4.0
2013-12-24 19:39 Alan Scott Note Added: 0032064
2013-12-24 19:39 Alan Scott Status customer review => closed
2015-01-11 08:27 Utkarsh Ayachit Source_changeset_attached => ParaView master 3c938fa9
2015-01-11 08:27 Utkarsh Ayachit Source_changeset_attached => ParaView master 617617f3
2015-01-11 08:27 Utkarsh Ayachit Source_changeset_attached => ParaView master 461501c1


Copyright © 2000 - 2018 MantisBT Team