View Issue Details [ Jump to Notes ] [ Print ]
IDProjectCategoryView StatusDate SubmittedLast Update
0013891ParaView(No Category)public2013-02-18 18:062013-05-23 15:00
ReporterWarren Hunt 
Assigned ToUtkarsh Ayachit 
PrioritynormalSeverityminorReproducibilityhave not tried
StatusclosedResolutionwon't fix 
PlatformOSOS Version
Product Version3.98.1 
Target VersionFixed in Version4.0 
Summary0013891: incorrect param to Threshold() method in paraview.simple
DescriptionThe coprocessing plugin python generator adds UseContinuousCellRange to the Threshold constructor, but the presence of that parameter causes the instantiation to crash with error below:

AttributeError: Attribute UseContinuousCellRange does not exist. This class does not allow addition of new attributes to avoid mistakes due to typos. Use add_attribute() if you really want to add this attribute.


#this works:
Threshold1 = Threshold( guiName="Threshold1", Scalars=['CELLS', 'KILLED'], ThresholdRange=[2.0, 2.0], AllScalars=1)

#this is the auto-generated python but does not work:
#Threshold1 = Threshold( guiName="Threshold1", Scalars=['CELLS', 'KILLED'], ThresholdRange=[2.0, 2.0], AllScalars=1, UseContinuousCellRange=0 )
 
Thanks!
Warren
Warren Hunt
Sandia Labs
TagsNo tags attached.
ProjectSandia-Catalyst
Topic Name
Typecrash
Attached Files

 Relationships
related to 0010965closedUtkarsh Ayachit Trace State does not cull default values 

  Notes
(0030442)
Andy Bauer (developer)
2013-03-14 15:52

 I think this is a version mismatch issue. the UseContinuousCellRange was added to the Threshold filter between 3.98.0 and 3.98.1.
(0030443)
Utkarsh Ayachit (administrator)
2013-03-14 16:01

 The solution for this issue is the same as the one we had discussed for "missing arrays".

For Catalyst/Batch scripts, missing arrays or missing attributes should not raise errors. Only for interactive Python shells should we throw exceptions in such cases.
(0030444)
Andy Bauer (developer)
2013-03-14 16:11

 I'm not a Python expert but it seems that this would be problematic when the non-existing attribute is being specified in the constructor. Also, having process 0 give a warning would also be helpful to let the user know that something may be amiss.
(0030447)
Utkarsh Ayachit (administrator)
2013-03-18 15:08

 After discussing with Berk, here's the conclusion:

The error in this case is happening because Warren is using a newer ParaView version to generate the Python script that the version used to run it. Alas, not much can be done to support this case. He should simply edit/update the scripts in such cases. ParaView Python is not forward-compatible.

BUG 0010965 will attempt to minimize the Python state itself to cull several properties so in this case, for example, the UseContinuousCellRange won't be saved in the state file at all and we will run into such issue at a fewer rate.
(0030849)
Alan Scott (manager)
2013-05-23 15:00

 Version mismatch. Closing.

 Issue History
Date Modified Username Field Change
2013-02-18 18:06 Warren Hunt New Issue
2013-03-14 13:30 Andy Bauer Assigned To => Andy Bauer
2013-03-14 14:22 Andy Bauer Status backlog => todo
2013-03-14 15:52 Andy Bauer Note Added: 0030442
2013-03-14 15:53 Andy Bauer Assigned To Andy Bauer => Utkarsh Ayachit
2013-03-14 16:01 Utkarsh Ayachit Note Added: 0030443
2013-03-14 16:11 Andy Bauer Note Added: 0030444
2013-03-18 15:08 Utkarsh Ayachit Note Added: 0030447
2013-03-18 15:09 Utkarsh Ayachit Status todo => gatekeeper review
2013-03-18 15:09 Utkarsh Ayachit Fixed in Version => git-master
2013-03-18 15:09 Utkarsh Ayachit Resolution open => won't fix
2013-03-18 15:09 Utkarsh Ayachit Status gatekeeper review => customer review
2013-03-18 15:09 Utkarsh Ayachit Relationship added related to 0010965
2013-05-18 20:00 Utkarsh Ayachit Fixed in Version git-master => 4.0
2013-05-23 15:00 Alan Scott Note Added: 0030849
2013-05-23 15:00 Alan Scott Status customer review => closed

Copyright © 2000 - 2018 MantisBT Team