[Bf-blender-cvs] [30bebec] pie-menus: Intensity and color are now adjusted upon import to get correct values

gaiaclary noreply at git.blender.org
Sat Aug 9 20:44:03 CEST 2014


Commit: 30bebec4778966df81ad8b32ed2c278e70d9b418
Author: gaiaclary
Date:   Sat Aug 9 19:42:17 2014 +0100
Branches: pie-menus
https://developer.blender.org/rB30bebec4778966df81ad8b32ed2c278e70d9b418

Intensity and color are now adjusted upon import to get correct values

===================================================================

M	source/blender/collada/DocumentImporter.cpp

===================================================================

diff --git a/source/blender/collada/DocumentImporter.cpp b/source/blender/collada/DocumentImporter.cpp
index bbbbbf2..97d300e 100644
--- a/source/blender/collada/DocumentImporter.cpp
+++ b/source/blender/collada/DocumentImporter.cpp
@@ -805,10 +805,14 @@ void DocumentImporter::write_profile_COMMON(COLLADAFW::EffectCommon *ef, Materia
 	// DIFFUSE
 	// color
 	if (ef->getDiffuse().isColor()) {
+		/* too high intensity can create artefacts (fireflies)
+		   So here we take care that intensity is set to 0.8 wherever possible
+		*/
 		col = ef->getDiffuse().getColor();
-		ma->r = col.getRed();
-		ma->g = col.getGreen();
-		ma->b = col.getBlue();
+		ma->ref = max_ffff(col.getRed(), col.getGreen(), col.getBlue(), 0.8);
+		ma->r = col.getRed()   / ma->ref;
+		ma->g = col.getGreen() / ma->ref;
+		ma->b = col.getBlue()  / ma->ref;
 	}
 	// texture
 	else if (ef->getDiffuse().isTexture()) {




More information about the Bf-blender-cvs mailing list