MantisBT - ParaView
View Issue Details
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 
ProjectSandia-Catalyst
Topic Name
Typecrash
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
Steps To Reproduce
Additional Information
TagsNo tags attached.
Relationships
related to 0010965closed Utkarsh Ayachit Trace State does not cull default values 
Attached Files
Issue History
Date ModifiedUsernameFieldChange
2013-02-18 18:06Warren HuntNew Issue
2013-03-14 13:30Andy BauerAssigned To => Andy Bauer
2013-03-14 14:22Andy BauerStatusbacklog => todo
2013-03-14 15:52Andy BauerNote Added: 0030442
2013-03-14 15:53Andy BauerAssigned ToAndy Bauer => Utkarsh Ayachit
2013-03-14 16:01Utkarsh AyachitNote Added: 0030443
2013-03-14 16:11Andy BauerNote Added: 0030444
2013-03-18 15:08Utkarsh AyachitNote Added: 0030447
2013-03-18 15:09Utkarsh AyachitStatustodo => gatekeeper review
2013-03-18 15:09Utkarsh AyachitFixed in Version => git-master
2013-03-18 15:09Utkarsh AyachitResolutionopen => won't fix
2013-03-18 15:09Utkarsh AyachitStatusgatekeeper review => customer review
2013-03-18 15:09Utkarsh AyachitRelationship addedrelated to 0010965
2013-05-18 20:00Utkarsh AyachitFixed in Versiongit-master => 4.0
2013-05-23 15:00Alan ScottNote Added: 0030849
2013-05-23 15:00Alan ScottStatuscustomer review => closed

Notes
(0030442)
Andy Bauer   
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   
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   
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   
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   
2013-05-23 15:00   
Version mismatch. Closing.