Commit 7546b304 authored by Rupert Swarbrick's avatar Rupert Swarbrick Committed by Yunqing Wang
Browse files

Reset restoration info at start of large_scale_tiles too

Since commit 6c545216, loop restoration coefficients are written inside
tiles data. Since the Wiener and Sgrproj coefficients are
delta-encoded (and tiles must be decoded independently) this means we
must call set_default_wiener and set_default_sgrproj at the start of
encoding and decoding each tile. That patch missed doing so when
encoding large scale tiles (part of the EXT_TILE experiment). This
adds the missing lines.

Note: This caused AV1/AVxEncoderThreadLSTest.EncoderResultTest/3 to
      fail with --enable-ext-tile, which is tracked in bug
      917. Unfortunately, there's something else going wrong that
      means the test is still failing: I'll address that in a separate


Change-Id: I5bd8e9d7f7a2abd60779abf170185a1c5b090474
parent efb755cd
......@@ -3422,6 +3422,13 @@ static uint32_t write_tiles(AV1_COMP *const cpi, uint8_t *const dst,
mode_bc.size = 1 << cpi->common.ans_window_size_log2;
for (int p = 0; p < MAX_MB_PLANE; ++p) {
set_default_wiener(cpi->td.mb.e_mbd.wiener_info + p);
set_default_sgrproj(cpi->td.mb.e_mbd.sgrproj_info + p);
aom_start_encode(&mode_bc, buf->data + data_offset);
write_modes(cpi, &tile_info, &mode_bc, &tok, tok_end);
assert(tok == tok_end);
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment