Fix new compositor color to value/bw conversion working different than it did
before. It doesn't make much sense to always multiply with alpha, especially if it's a premul image that already has it multiplied in. Patch by Irie Shinsuke.
This commit is contained in:
@@ -38,7 +38,7 @@ void ConvertColorToBWOperation::executePixel(float *outputValue, float x, float
|
||||
{
|
||||
float inputColor[4];
|
||||
inputOperation->read(&inputColor[0], x, y, sampler, inputBuffers);
|
||||
outputValue[0] = (inputColor[0]*0.35f + inputColor[1]*0.45f + inputColor[2]*0.2f)*inputColor[3];
|
||||
outputValue[0] = inputColor[0]*0.35f + inputColor[1]*0.45f + inputColor[2]*0.2f;
|
||||
}
|
||||
|
||||
void ConvertColorToBWOperation::deinitExecution()
|
||||
|
||||
@@ -38,7 +38,7 @@ void ConvertColourToValueProg::executePixel(float *outputValue, float x, float y
|
||||
{
|
||||
float inputColor[4];
|
||||
inputOperation->read(&inputColor[0], x, y, sampler, inputBuffers);
|
||||
outputValue[0] = ((inputColor[0] + inputColor[1] + inputColor[2])/3.0f)*inputColor[3];
|
||||
outputValue[0] = (inputColor[0] + inputColor[1] + inputColor[2])/3.0f;
|
||||
}
|
||||
|
||||
void ConvertColourToValueProg::deinitExecution()
|
||||
|
||||
Reference in New Issue
Block a user