View Issue Details [ Jump to Notes ]

[ Print ]

Project

Category

View Status

Date Submitted

Last Update

0011854

ParaView

Feature

public

2011-02-14 15:16

2011-08-17 20:07

Reporter

Alan Scott

Assigned To

Kyle Lutz

Priority

high

Severity

minor

Reproducibility

always

Status

closed

Resolution

fixed

Platform

OS Version

Product Version

Target Version

3.12

Fixed in Version

3.12

Summary

0011854: Add multiple variable selection to AMR Contour

Description

We need to add multiple variable support for the AMR Contour, and probably AMR Clip filters. I have a user that is looking at an object, such as a car. Every part of the car is a different Material Volume Fraction (engine, transmission, frame, alternator - around 15-20 of these), and he would like to be able to select them in one step, rather than one by one, over and over again.

A example would be Dave's small CTH dataset - we want to select the ball and plate with one filter.

What I want is basically the code in the Material Interface Filter.

Tags

No tags attached.

Project

Sandia

Topic Name

11854-multiple-amr-contour-arrays

Type

incorrect functionality

Attached Files

bug.patch [^] (735 bytes) 2011-07-28 15:44 [Show Content] [Hide Content]

diff --git a/ParaViewCore/VTKExtensions/vtkPVAMRDualClip.cxx b/ParaViewCore/VTKExtensions/vtkPVAMRDualClip.cxx
index f026b51..9a585bb 100644
--- a/ParaViewCore/VTKExtensions/vtkPVAMRDualClip.cxx
+++ b/ParaViewCore/VTKExtensions/vtkPVAMRDualClip.cxx
@@ -62,7 +62,9 @@ int vtkPVAMRDualClip::RequestData(vtkInformation* vtkNotUsed(request),
     outInfo->Get(vtkDataObject::DATA_OBJECT()));
 
 
-  this->SetIsoValue(this->VolumeFractionSurfaceValue *
+  // Don't call SetIsoValue() that changes filter's MTime which is not
+  // acceptable in RequestData().
+  this->IsoValue = (this->VolumeFractionSurfaceValue *
                     PV_AMR_SURFACE_VALUE_UNSIGNED_CHAR);
 
   size_t noOfArrays = this->Implementation->CellArrays.size();

Relationships

Notes
(0027096) Utkarsh Ayachit (administrator) 2011-07-28 15:42	Look at vtkPVAMRDualClip. We need to do similar for the contour.

(0027097) Utkarsh Ayachit (administrator) 2011-07-28 15:44	Look at attached patch. It's a bug in vtkPVAMRDualClip, don't repeat the same in your version :). I am committing a fix for vtkPVAMRDualClip but it may take a while for it to show up on master.

(0027224) Kyle Lutz (viewer) 2011-08-04 11:28	Merge topic '11854-multiple-amr-contour-arrays' into next 78030f5 Add the ability to use multiple input arrays in AMRDualContour

(0027239) Utkarsh Ayachit (administrator) 2011-08-04 18:08	Developer note: Issue slated for gatekeeper review on Aug 7, 2 PM EDT.

(0027258) Robert Maynard (developer) 2011-08-05 10:32	verified that this works.

(0027280) Utkarsh Ayachit (administrator) 2011-08-05 15:55	Merged into master (if applicable)

(0027384) Alan Scott (manager) 2011-08-17 20:07	Oh, that is sooo excellent. Tested local server, development branch, Linux.

Issue History
Date Modified	Username	Field	Change
2011-02-14 15:16	Alan Scott	New Issue
2011-02-14 15:19	Alan Scott	Target Version	=> 3.10.1
2011-02-14 15:19	Alan Scott	Description Updated
2011-02-15 15:55	Utkarsh Ayachit	Assigned To	=> Utkarsh Ayachit
2011-02-15 15:55	Utkarsh Ayachit	Status	backlog => tabled
2011-03-31 11:24	Utkarsh Ayachit	Target Version	3.10.1 => 3.12
2011-06-16 13:10	Zack Galbreath	Category	Feature Request => Feature
2011-07-27 22:00	Alan Scott	Project	=> Sandia
2011-07-27 22:00	Alan Scott	Type	=> incorrect functionality
2011-07-28 15:42	Utkarsh Ayachit	Note Added: 0027096
2011-07-28 15:42	Utkarsh Ayachit	Assigned To	Utkarsh Ayachit => Kyle Lutz
2011-07-28 15:42	Utkarsh Ayachit	Status	backlog => todo
2011-07-28 15:42	Utkarsh Ayachit	Target Version	=> 3.12
2011-07-28 15:44	Utkarsh Ayachit	Note Added: 0027097
2011-07-28 15:44	Utkarsh Ayachit	File Added: bug.patch
2011-08-04 10:50	Kyle Lutz	Status	todo => active development
2011-08-04 11:09	Kyle Lutz	Topic Name	=> 11854-multiple-amr-contour-arrays
2011-08-04 11:28	Kyle Lutz	Note Added: 0027224
2011-08-04 11:28	Kyle Lutz	Status	active development => gatekeeper review
2011-08-04 11:28	Kyle Lutz	Fixed in Version	=> 3.12
2011-08-04 11:28	Kyle Lutz	Resolution	open => fixed
2011-08-04 18:08	Utkarsh Ayachit	Note Added: 0027239
2011-08-05 10:32	Robert Maynard	Note Added: 0027258
2011-08-05 15:55	Utkarsh Ayachit	Status	gatekeeper review => customer review
2011-08-05 15:55	Utkarsh Ayachit	Note Added: 0027280
2011-08-17 20:07	Alan Scott	Note Added: 0027384
2011-08-17 20:07	Alan Scott	Status	customer review => closed