Skip to content

Commit b9fb001

Browse files
committed
Tiling: fix edge case
1 parent 330c9ac commit b9fb001

File tree

1 file changed

+17
-9
lines changed

1 file changed

+17
-9
lines changed

ggml_extend.hpp

Lines changed: 17 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -361,7 +361,9 @@ __STATIC_INLINE__ void ggml_merge_tensor_2d(struct ggml_tensor* input,
361361
int x,
362362
int y,
363363
int overlap_x,
364-
int overlap_y) {
364+
int overlap_y,
365+
int x_skip = 0,
366+
int y_skip = 0) {
365367
int64_t width = input->ne[0];
366368
int64_t height = input->ne[1];
367369
int64_t channels = input->ne[2];
@@ -370,16 +372,16 @@ __STATIC_INLINE__ void ggml_merge_tensor_2d(struct ggml_tensor* input,
370372
int64_t img_height = output->ne[1];
371373

372374
GGML_ASSERT(input->type == GGML_TYPE_F32 && output->type == GGML_TYPE_F32);
373-
for (int iy = 0; iy < height; iy++) {
374-
for (int ix = 0; ix < width; ix++) {
375+
for (int iy = y_skip; iy < height; iy++) {
376+
for (int ix = x_skip; ix < width; ix++) {
375377
for (int k = 0; k < channels; k++) {
376378
float new_value = ggml_tensor_get_f32(input, ix, iy, k);
377379
if (overlap_x > 0 && overlap_y > 0) { // blend colors in overlapped area
378380
float old_value = ggml_tensor_get_f32(output, x + ix, y + iy, k);
379381

380-
const float x_f_0 = (overlap_x > 0 && x > 0) ? ix / float(overlap_x) : 1;
382+
const float x_f_0 = (overlap_x > 0 && x > 0) ? (ix - x_skip) / float(overlap_x) : 1;
381383
const float x_f_1 = (overlap_x > 0 && x < (img_width - width)) ? (width - ix) / float(overlap_x) : 1;
382-
const float y_f_0 = (overlap_y > 0 && y > 0) ? iy / float(overlap_y) : 1;
384+
const float y_f_0 = (overlap_y > 0 && y > 0) ? (iy - y_skip) / float(overlap_y) : 1;
383385
const float y_f_1 = (overlap_y > 0 && y < (img_height - height)) ? (height - iy) / float(overlap_y) : 1;
384386

385387
const float x_f = std::min(std::min(x_f_0, x_f_1), 1.f);
@@ -508,10 +510,10 @@ __STATIC_INLINE__ void sd_tiling(ggml_tensor* input, ggml_tensor* output, const
508510
input_tile_size = tile_size * scale;
509511
output_tile_size = tile_size;
510512
}
511-
int num_tiles_x = (float)(input_width - input_tile_size * tile_overlap_factor) / (float)(input_tile_size * (1 - tile_overlap_factor));
513+
int num_tiles_x = (input_width - (int)(input_tile_size * tile_overlap_factor)) / (int)(input_tile_size * (1 - tile_overlap_factor));
512514
float tile_overlap_factor_x = (float)(input_tile_size * num_tiles_x - input_width) / (float)(input_tile_size * (num_tiles_x - 1));
513515

514-
int num_tiles_y = (float)(input_height - input_tile_size * tile_overlap_factor) / (float)(input_tile_size * (1 - tile_overlap_factor));
516+
int num_tiles_y = (input_height - (int)(input_tile_size * tile_overlap_factor)) / (int)(input_tile_size * (1 - tile_overlap_factor));
515517
float tile_overlap_factor_y = (float)(input_tile_size * num_tiles_y - input_height) / (float)(input_tile_size * (num_tiles_y - 1));
516518

517519
LOG_DEBUG("optimal overlap : %f, %f (targeting %f)", tile_overlap_factor_x, tile_overlap_factor_y, tile_overlap_factor);
@@ -551,22 +553,28 @@ __STATIC_INLINE__ void sd_tiling(ggml_tensor* input, ggml_tensor* output, const
551553
bool last_y = false, last_x = false;
552554
float last_time = 0.0f;
553555
for (int y = 0; y < input_height && !last_y; y += non_tile_overlap_y) {
556+
int dy = 0;
554557
if (y + input_tile_size >= input_height) {
558+
int _y = y;
555559
y = input_height - input_tile_size;
560+
dy = _y - y;
556561
last_y = true;
557562
}
558563
for (int x = 0; x < input_width && !last_x; x += non_tile_overlap_x) {
564+
int dx = 0;
559565
if (x + input_tile_size >= input_width) {
566+
int _x = x;
560567
x = input_width - input_tile_size;
568+
dx = _x - x;
561569
last_x = true;
562570
}
563571
int64_t t1 = ggml_time_ms();
564572
ggml_split_tensor_2d(input, input_tile, x, y);
565573
on_processing(input_tile, output_tile, false);
566574
if (scaled_out) {
567-
ggml_merge_tensor_2d(output_tile, output, x * scale, y * scale, tile_overlap_x * scale, tile_overlap_y * scale);
575+
ggml_merge_tensor_2d(output_tile, output, x * scale, y * scale, tile_overlap_x * scale, tile_overlap_y * scale, dx * scale, dy * scale);
568576
} else {
569-
ggml_merge_tensor_2d(output_tile, output, x / scale, y / scale, tile_overlap_x / scale, tile_overlap_y / scale);
577+
ggml_merge_tensor_2d(output_tile, output, x / scale, y / scale, tile_overlap_x / scale, tile_overlap_y / scale, dx / scale, dy / scale);
570578
}
571579
int64_t t2 = ggml_time_ms();
572580
last_time = (t2 - t1) / 1000.0f;

0 commit comments

Comments
 (0)