Adding Gamma correction to all the Color coming for the attribute stream if used as color

libraries/entities-renderer/src/paintStroke.slv

@@ -13,8 +13,8 @@
 //
 
 <@include gpu/Inputs.slh@>
+<@include gpu/Color.slh@>
 <@include gpu/Transform.slh@>
-
 <$declareStandardTransform()$>
 
 // the interpolated normal
@@ -30,7 +30,7 @@ void main(void) {
     varTexcoord = inTexCoord0.st;
     
     // pass along the diffuse color
-    varColor = inColor;
+    varColor = colorToLinearRGBA(inColor);
 
     // standard transform
     TransformCamera cam = getTransformCamera();

libraries/gpu/src/gpu/Color.slh

@@ -0,0 +1,52 @@
+<!
+//  Color.slh
+//  libraries/gpu/src
+//
+//  Created by Sam Gateau on 2015/12/18.
+//  Copyright 2015 High Fidelity, Inc.
+//
+//  Distributed under the Apache License, Version 2.0.
+//  See the accompanying file LICENSE or http://www.apache.org/licenses/LICENSE-2.0.html
+!>
+<@if not GPU_COLOR_SLH@>
+<@def GPU_COLOR_SLH@>
+<!
+float colorComponentToLinear(float cs) {
+    // sRGB to linear conversion
+    //      {  cs / 12.92,                 cs <= 0.04045
+    // cl = {
+    //      {  ((cs + 0.055)/1.055)^2.4,   cs >  0.04045
+    // constants:
+    // T = 0.04045
+    // A = 1 / 1.055 = 0.94786729857
+    // B = 0.055 * A = 0.05213270142
+    // C = 1 / 12.92 = 0.0773993808
+    // G = 2.4
+    const float T = 0.04045;
+    const float A = 0.947867;
+    const float B = 0.052132;
+    const float C = 0.077399;
+    const float G = 2.4;
+
+    if (cs > T) {
+        return pow((cs * A + B), G);
+    } else {
+        return cs * C;
+    }
+}
+
+vec3 colorToLinear(vec3 srgb) {
+    return vec3(colorComponentToLinear(srgb.x), colorComponentToLinear(srgb.y), colorComponentToLinear(srgb.z));
+}
+!>
+
+vec3 colorToLinearRGB(vec3 srgb) {
+    const float GAMMA_22 = 2.2;
+    return pow(srgb, vec3(GAMMA_22));
+}
+
+vec4 colorToLinearRGBA(vec4 srgba) {
+    return vec4(colorToLinearRGB(srgba.xyz), srgba.w);
+}
+
+<@endif@>

libraries/render-utils/src/animdebugdraw.slv

@@ -9,16 +9,15 @@
 //
 
 <@include gpu/Inputs.slh@>
-
+<@include gpu/Color.slh@>
 <@include gpu/Transform.slh@>
-
 <$declareStandardTransform()$>
 
 out vec4 _color;
 
 void main(void) {
     // pass along the diffuse color
-    _color = inColor.rgba;
+    _color = colorToLinearRGBA(inColor.rgba);
 
     TransformCamera cam = getTransformCamera();
     TransformObject obj = getTransformObject();

libraries/render-utils/src/model.slv

@@ -12,9 +12,8 @@
 //
 
 <@include gpu/Inputs.slh@>
-
+<@include gpu/Color.slh@>
 <@include gpu/Transform.slh@>
-
 <$declareStandardTransform()$>
 
 const int MAX_TEXCOORDS = 2;
@@ -27,9 +26,8 @@ out vec3 _color;
 out vec2 _texCoord0;
 
 void main(void) {
-    
-    // pass along the diffuse color
-    _color = inColor.xyz;
+    // pass along the diffuse color in linear space
+    _color = colorToLinearRGB(inColor.xyz);
 
     // and the texture coordinates
     _texCoord0 = (texcoordMatrices[0] * vec4(inTexCoord0.st, 0.0, 1.0)).st;

libraries/render-utils/src/model_lightmap.slv

@@ -13,9 +13,8 @@
 //
 
 <@include gpu/Inputs.slh@>
-
+<@include gpu/Color.slh@>
 <@include gpu/Transform.slh@>
-
 <$declareStandardTransform()$>
 
 const int MAX_TEXCOORDS = 2;
@@ -29,7 +28,8 @@ out vec3 _normal;
 out vec3 _color;
 
 void main(void) {
-    _color = inColor.xyz;
+    // pass along the diffuse color in linear space
+    _color = colorToLinearRGB(inColor.xyz);
 
     // and the texture coordinates
     _texCoord0 = (texcoordMatrices[0] * vec4(inTexCoord0.st, 0.0, 1.0)).st;

libraries/render-utils/src/model_lightmap_normal_map.slv

@@ -13,9 +13,8 @@
 //
 
 <@include gpu/Inputs.slh@>
-
+<@include gpu/Color.slh@>
 <@include gpu/Transform.slh@>
-
 <$declareStandardTransform()$>
 
 const int MAX_TEXCOORDS = 2;
@@ -30,7 +29,8 @@ out vec3 _tangent;
 out vec3 _color;
 
 void main(void) {
-    _color = inColor.xyz;
+    // pass along the diffuse color in linear space
+    _color = colorToLinearRGB(inColor.xyz);
 
     // and the texture coordinates
     _texCoord0 = (texcoordMatrices[0] * vec4(inTexCoord0.st, 0.0, 1.0)).st;

libraries/render-utils/src/model_normal_map.slv

@@ -13,9 +13,8 @@
 //
 
 <@include gpu/Inputs.slh@>
-
+<@include gpu/Color.slh@>
 <@include gpu/Transform.slh@>
-
 <$declareStandardTransform()$>
 
 const int MAX_TEXCOORDS = 2;
@@ -30,7 +29,7 @@ out vec3 _color;
 
 void main(void) {
     // pass along the diffuse color
-    _color = inColor.rgb;
+    _color = colorToLinearRGB(inColor.xyz);
 
     // and the texture coordinates
     _texCoord0 = (texcoordMatrices[0] * vec4(inTexCoord0.xy, 0.0, 1.0)).st;

libraries/render-utils/src/overlay3D.slv

@@ -11,9 +11,8 @@
 //
 
 <@include gpu/Inputs.slh@>
-
+<@include gpu/Color.slh@>
 <@include gpu/Transform.slh@>
-
 <$declareStandardTransform()$>
 
 out vec2 varTexcoord;
@@ -30,7 +29,7 @@ void main(void) {
     varTexcoord = inTexCoord0.xy;
 
     // pass along the color
-    varColor = inColor;
+    varColor = colorToLinearRGBA(inColor);
 
     // standard transform
     TransformCamera cam = getTransformCamera();

libraries/render-utils/src/simple.slv

@@ -13,9 +13,8 @@
 //
 
 <@include gpu/Inputs.slh@>
-
+<@include gpu/Color.slh@>
 <@include gpu/Transform.slh@>
-
 <$declareStandardTransform()$>
 
 uniform bool Instanced = false;
@@ -28,7 +27,7 @@ out vec2 _texCoord0;
 out vec4 _position;
 
 void main(void) {
-    _color = inColor.rgb;
+    _color = colorToLinearRGB(inColor.rgb);
     _texCoord0 = inTexCoord0.st;
     _position = inPosition;
     _modelNormal = inNormal.xyz;

libraries/render-utils/src/skin_model.slv

@@ -13,9 +13,8 @@
 //
 
 <@include gpu/Inputs.slh@>
-
+<@include gpu/Color.slh@>
 <@include gpu/Transform.slh@>
-
 <$declareStandardTransform()$>
 
 <@include Skinning.slh@>
@@ -34,7 +33,7 @@ void main(void) {
     skinPositionNormal(inSkinClusterIndex, inSkinClusterWeight, inPosition, inNormal.xyz, position, interpolatedNormal);
 
     // pass along the diffuse color
-    _color = inColor.rgb;
+    _color = colorToLinearRGB(inColor.rgb);
 
     // and the texture coordinates
     _texCoord0 = (texcoordMatrices[0] * vec4(inTexCoord0.st, 0.0, 1.0)).st;

libraries/render-utils/src/skin_model_normal_map.slv

@@ -13,9 +13,8 @@
 //
 
 <@include gpu/Inputs.slh@>
-
+<@include gpu/Color.slh@>
 <@include gpu/Transform.slh@>
-
 <$declareStandardTransform()$>
 
 <@include Skinning.slh@>
@@ -36,7 +35,7 @@ void main(void) {
     skinPositionNormalTangent(inSkinClusterIndex, inSkinClusterWeight, inPosition, inNormal.xyz, inTangent.xyz, position, interpolatedNormal.xyz, interpolatedTangent.xyz);
 
     // pass along the diffuse color
-    _color = inColor.rgb;
+    _color = colorToLinearRGB(inColor.rgb);
 
     // and the texture coordinates
     _texCoord0 = (texcoordMatrices[0] * vec4(inTexCoord0.st, 0.0, 1.0)).st;

libraries/render-utils/src/standardTransformPNTC.slv

@@ -13,9 +13,8 @@
 //
 
 <@include gpu/Inputs.slh@>
-
+<@include gpu/Color.slh@>
 <@include gpu/Transform.slh@>
-
 <$declareStandardTransform()$>
 
 out vec3 varPosition;
@@ -25,7 +24,7 @@ out vec4 varColor;
 
 void main(void) {
     varTexCoord0 = inTexCoord0.st;
-    varColor = inColor;
+    varColor = colorToLinearRGBA(inColor);
     
     // standard transform
     TransformCamera cam = getTransformCamera();

libraries/render-utils/src/stars.slv

@@ -13,9 +13,8 @@
 //
 
 <@include gpu/Inputs.slh@>
-
+<@include gpu/Color.slh@>
 <@include gpu/Transform.slh@>
-
 <$declareStandardTransform()$>
 
 // TODO we need to get the viewport resolution and FOV passed to us so we can modify the point size
@@ -26,7 +25,7 @@ out vec4 varColor;
 out float varSize;
 
 void main(void) {
-    varColor = inColor.rgba;
+    varColor = colorToLinearRGBA(inColor);
     
     // standard transform
     TransformCamera cam = getTransformCamera();