# HG changeset patch
# User Chris Cannam
# Date 1470047513 -3600
# Node ID b71a0491d2872da68ff7b1f4358b6ac2f91e67b7
# Parent  343887ac6766bca3f93566529bd0f59794ef4044
Some notes on gain

diff -r 343887ac6766 -r b71a0491d287 layer/Colour3DPlotRenderer.cpp
--- a/layer/Colour3DPlotRenderer.cpp	Fri Jul 22 14:42:36 2016 +0100
+++ b/layer/Colour3DPlotRenderer.cpp	Mon Aug 01 11:31:53 2016 +0100
@@ -351,9 +351,8 @@
 
             // order:
             // get column -> scale -> record extents ->
-            // normalise -> peak pick -> apply display gain ->
-            // distribute/interpolate
-
+            // normalise -> peak pick -> apply display gain
+            
             ColumnOp::Column fullColumn = model->getColumn(sx);
                 
             ColumnOp::Column column =
@@ -374,6 +373,9 @@
                 preparedColumn = ColumnOp::peakPick(preparedColumn);
             }
 
+            // Display gain belongs to the colour scale and is
+            // applied by the colour scale object when mapping it
+
             psx = sx;
         }
 
@@ -774,8 +776,8 @@
 
                 // order:
                 // get column -> scale -> record extents ->
-                // normalise -> peak pick -> apply display gain ->
-                // distribute/interpolate
+                // normalise -> peak pick -> distribute/interpolate ->
+                // apply display gain
 
                 ColumnOp::Column fullColumn = sourceModel->getColumn(sx);
 
@@ -801,11 +803,14 @@
                 }
 
                 preparedColumn =
-                    ColumnOp::distribute(column, //!!! gain? ColumnOp::applyGain(column, m_gain),
+                    ColumnOp::distribute(column,
                                          h,
                                          binfory,
                                          minbin,
                                          m_params.interpolate);
+
+                // Display gain belongs to the colour scale and is
+                // applied by the colour scale object when mapping it
                 
                 psx = sx;
             }
@@ -937,12 +942,10 @@
 //                }
 
 //!!!                if (m_colourScale != ColourScaleType::Phase) {
-                    column = ColumnOp::normalize(column, m_params.normalization);
+                preparedColumn = ColumnOp::normalize
+                    (column, m_params.normalization);
 //!!!                }
 
-                    preparedColumn = column;
-//!!! gain?                preparedColumn = ColumnOp::applyGain(column, m_params.gain);
-                
                 psx = sx;
             }