MantisBT - ParaView
View Issue Details
0010841ParaViewBugpublic2010-06-16 17:062010-07-21 19:10
Alan Scott 
Utkarsh Ayachit 
highminoralways
closedfixed 
 
3.103.10 
0010841: Lighting incorrect for point data with CTH AMR
ParaView trunk, Windows, local server.

* Open Dave's small dataset spcta_a. All vars on, Apply.
* CellDataToPointData filter.
* IsoVolume OR clip by scalar. Scalars - Material Volume Fraction. Value - 128. Apply.

The inside of the object is bright, the inside is dark. Normals are reversed?

Must be fixed for 3.8.1.
No tags attached.
related to 0009849closed Zhanping Liu Contour creates normals from AMR data, whether desired or not. 
related to 0009785closed Alan Scott Extract CTH Parts has bugs 
Issue History
2010-06-16 17:06Alan ScottNew Issue
2010-06-16 17:07Alan ScottRelationship addedrelated to 0009785
2010-06-18 11:21Utkarsh AyachitRelationship addedrelated to 0009849
2010-06-18 11:23Utkarsh AyachitNote Added: 0021091
2010-06-18 11:23Utkarsh AyachitStatusbacklog => @80@
2010-06-18 11:23Utkarsh AyachitFixed in Version => 3.8.1
2010-06-18 11:23Utkarsh AyachitResolutionopen => fixed
2010-06-18 11:23Utkarsh AyachitAssigned To => Utkarsh Ayachit
2010-06-18 11:25Utkarsh AyachitNote Added: 0021092
2010-06-18 11:25Utkarsh AyachitFixed in Version3.8.1 => 3.10
2010-06-18 11:25Utkarsh AyachitTarget Version3.8.1 => 3.10
2010-07-21 19:10Alan ScottNote Added: 0021447
2010-07-21 19:10Alan ScottStatus@80@ => closed

Notes
(0021091)
Utkarsh Ayachit   
2010-06-18 11:23   
commit d6a140e77c44f9743a54e3a7fb07dc15502e6bce
Author: Utkarsh Ayachit <utkarsh.ayachit@kitware.com>
Date: Fri Jun 18 11:21:20 2010 -0400

    Fixed BUG 0010841.
    
    Reverting part of commit 1be529023b4adac366c8878c59aee0ac67247046. We were
    disabling generation of cell normals in the geometry filter that can mess with
    ligthing when the data doesn't have any normals. Undoing the fix for BUG 0009849.
(0021092)
Utkarsh Ayachit   
2010-06-18 11:25   
The bug was introduced by commit: 1be529023b4adac366c8878c59aee0ac67247046 which is not there on the release branch hence, we don't need to merge this patch into the release branch.
(0021447)
Alan Scott   
2010-07-21 19:10   
Tested local server, windows, paraview trunk.