griefith/test
1
0
Fork 0
Code Issues Pull Requests Actions 1 Packages Projects Releases Wiki Activity

== Inpaint Node ==

Browse Source 
Fixed several small (stupid) issues with the inpaint node:

* on pixel order building, some ranges got wrong and origin was considered
  several times.

* the convolution kernel didn't consider all pixels (+1,0),(-1,0),(0,1) and (0-1)
  were omited, leading to suboptimal results (sometimes even black areas)

* alpha channel is now only affected an areas considered by inpaint.
  That's only important, if you choose low iteration counts.
This commit is contained in:
Peter Schlaile
2012-08-12 15:59:40 +00:00
parent 8d59fa4e40
commit bd71fb6d1d
1 changed files with 5 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

10
source/blender/compositor/operations/COM_InpaintOperation.cpp
View File

@@ -142,7 +142,7 @@ void InpaintSimpleOperation::calc_manhatten_distance()
}
for (int j = height - 1; j >= 0; j--) {
for (int i = width; i >= 0; i--) {
for (int i = width - 1; i >= 0; i--) {
int r = m[j * width + i];
if (i + 1 < width)
@@ -184,8 +184,7 @@ void InpaintSimpleOperation::pix_step(int x, int y)
for (int dx = -1; dx <= 1; dx++) {
for (int dy = -1; dy <= 1; dy++) {
if (dx != 0 && dy != 0) {
if (dx != 0 || dy != 0) {
int x_ofs = x + dx;
int y_ofs = y + dy;
@@ -210,6 +209,8 @@ void InpaintSimpleOperation::pix_step(int x, int y)
}
mul_v3_v3fl(this->get_pixel(x, y), pix, 1.0f / n);
this->get_pixel(x, y)[3] = 1.0f;
}
void *InpaintSimpleOperation::initializeTileData(rcti *rect)
@@ -243,8 +244,7 @@ void *InpaintSimpleOperation::initializeTileData(rcti *rect)
void InpaintSimpleOperation::executePixel(float output[4], int x, int y, void *data)
{
this->clamp_xy(x, y);
copy_v3_v3(output, this->get_pixel(x, y));
output[3] = 1.0f;
copy_v4_v4(output, this->get_pixel(x, y));
}
void InpaintSimpleOperation::deinitExecution()