MantisBT - ParaView
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0012780ParaView(No Category)public2011-12-08 16:322012-07-11 16:42
ReporterGreg Weirs 
Assigned ToUtkarsh Ayachit 
PrioritynormalSeverityminorReproducibilityhave not tried
StatusclosedResolutionfixed 
PlatformOSOS Version
Product Version3.12 
Target VersionFixed in Version3.14 
ProjectSandia
Topic Name12780_prism_improvements
Typeincorrect functionality
Summary0012780: Prism: incorrect treatment of log scaling in 500/600 series tables
DescriptionPrism SESAME reader to open copper_spitzer.mat.1.600

1.) When the 601 table is opened the "Use Log Scaling" boxes are not
checked, but when the 602 or 603 are opened they are. Behavior for the
602 and 603 tables is correct.
2.) Values displayed in the threshold text boxes and the information
tab appears to be the log of the quantity, rather than tha value of
the quantitiy itself.
3.) When unit conversions are applied, they are applied to the log of
the value, and not the value itself.

Recall that for 500 and 600 series tables, the sesame file holds the
log10 of the quantity of interest. I think the steps in reading and
transforming the data should be:
a.) read the data
b.) convert all data from log10 to unlogged values
c.) apply unit conversions
d.) if user selects to use log scaling, apply it
e.) apply auto scaling

The difference compared to the 300 and 400 series tables is that for
500/600 tables, step b is required. If step b is done for 500/600
tables, from that point forward everthing should be the same as for
the 300 tables, and I think that corrects issues 2 and 3. Issue 1 has
to do with whether the "Use Log Scaling" boxes are checked by default
or not, and this corresponds to step d, not step b.
Steps To Reproduce
Additional Information
TagsNo tags attached.
Relationships
Attached Files
Issue History
Date ModifiedUsernameFieldChange
2011-12-08 16:32Greg WeirsNew Issue
2012-02-06 14:56Alan ScottStatusbacklog => todo
2012-02-06 15:09Alan ScottProjectTBD => Sandia
2012-02-06 15:09Alan ScottDescription Updatedbug_revision_view_page.php?rev_id=420#r420
2012-02-06 16:05Robert MaynardAssigned To => Robert Maynard
2012-02-06 16:12Robert MaynardStatustodo => active development
2012-02-07 12:29Utkarsh AyachitTopic Name => 12780_prism_improvements
2012-02-07 12:29Utkarsh AyachitNote Added: 0028073
2012-02-07 12:29Utkarsh AyachitStatusactive development => gatekeeper review
2012-02-07 12:29Utkarsh AyachitFixed in Version => git-next
2012-02-07 12:29Utkarsh AyachitResolutionopen => fixed
2012-02-07 12:29Utkarsh AyachitAssigned ToRobert Maynard => Utkarsh Ayachit
2012-02-08 10:41Utkarsh AyachitFixed in Versiongit-next => git-master
2012-02-08 10:42Utkarsh AyachitStatusgatekeeper review => customer review
2012-02-08 10:42Utkarsh AyachitNote Added: 0028097
2012-02-08 17:21Utkarsh AyachitFixed in Versiongit-master => 3.14
2012-07-11 16:42Greg WeirsStatuscustomer review => closed

Notes
(0028073)
Utkarsh Ayachit   
2012-02-07 12:29   
commit f1252ec3ef4b97dac1f925fc5bf9ccc04b5308d2
Author: Utkarsh Ayachit <utkarsh.ayachit@kitware.com>
Date: Tue Feb 7 09:17:48 2012 -0500

    Update SESAMEConversions file.


commit c3a1be58337e2ec0be581a2757da8d4e83f80171
Author: Utkarsh Ayachit <utkarsh.ayachit@kitware.com>
Date: Tue Feb 7 11:24:10 2012 -0500

    When reading in SESAME file for 500/600 tables, unlog values.
    
    * Changed vtkPrismSESAMEReader to read in unlogged values when appropriate.
    * Changed vtkPrismFilter to use log when user requested irrespective of table id.


commit 9841718b30815e9f3d1d319ea602f591153f3893
Author: Utkarsh Ayachit <utkarsh.ayachit@kitware.com>
Date: Tue Feb 7 12:27:03 2012 -0500

    BUG: 12780. Ensure that the log-check boxes are set correctly.
    
    The log check box status was not set correctly when the panel was initialized.
    Fixed that. Also considated initialization code into one method and fixed some
    indentation.
(0028097)
Utkarsh Ayachit   
2012-02-08 10:42   
merged into master, if applicable.