GitLab

BUG=aomedia:926

Change-Id: I3ec5ff501e2a6791ed13dcb0a7f4755b86e3910b

This reverts commit 1542157b.

The associated bug (#881) was fixed a couple of weeks ago.

Change-Id: If05627fd32ff9be1c4f425159798cfe46b6b2908

Further reduce the context model size needed for base levels down
to 25 per transform size.

Change-Id: I9df4870d2b027cdb1356de0fc4d5bcc22155319e

Reduce the context model size required to code the base level
syntax values from 42 to 32 per transform size.

Change-Id: I08485fdb563bc5c3323b0e147d69b47c286b9bb5

This loop is wrongly vectorized by the MSVC2017 compiler, this change
is a work-around for the compiler bug.

Change-Id: Ie4c8403965c3e4cd6d70eb3dbc92148f5272f0ab
(cherry picked from commit 5b33d7184bf319d9c10e34ef0fdcdd244d2fdb56)

Int was used but should have been int16_t. This does not alter the
encoder/decoder behavior

Change-Id: I6bfd8a3942e9f9b86cccf10c144fec147e996944

This changes serves to illustrate that DC_PRED does not
need 32 bits. It only needs the specified pixel bit depth.
Which fits inside a signed 16 bit integer.

Change-Id: Ice062c8c4a794cb4f32a3963278f4a99ef326dda

With the previous addition of cfl_build_prediction_lbd and
cfl_build_prediction_hbd there's no need for cfl_build_prediction.

Change-Id: I4246a53378f1e85be40c14433a3084d7120ad060

Change-Id: I97f7d390e580b6e1b2b471a6f1a95082323d39aa

This experiment has been abandonned.

Change-Id: Ifde0a76266af9f8c202c01805410b72d64594b0b

This experiment has been adopted, we can simplify the code
by dropping the associated preprocessor conditionals.

Change-Id: I54fee8a18afaec5333ae852eb6bd9ac0e2550db1

Avoid switch warning regarding fallthrough, add consts.

Change-Id: Ie3a03e60420d962941abc4e144648f29401916b9

When the frame header requires to cut off the motion vector
referencing from prior frames, shut off the use of motion field.

BUG=aomedia:984

Change-Id: Ib47129ea538248f4e67d115be2da3b5dd861693e

Disabling such cases will reduce search time without hurting
coding performance (-0.001%).

Change-Id: Iaa4385053fcf5bd59fb1f94d5583eb19cf792242

This experiment has been adopted, we can simplify the code
by dropping the associated preprocessor conditionals.

Change-Id: I2dce80e1e1b2116708b6ba9feeacaacc12af8fc4

Adopted by AOM WG on 08/17.

Change-Id: I3e5b369e9d8220c7b3f9e6f9d300eb640e2bf3a8

Adopted 08/24.

Change-Id: Ic6ccc766c821f7bba2779c069f00ccc131ff9c10

This patch factors out a function that computes the rd cost for
a given transform type given the transform partition already
computed. This is then used to develop a speed feature where the
transform size search disables trellis optimization but once the
transform sizes are decided, a final search is conducted with
optimization turned back on.
This patch does not change anything in speed 0 yet.

Change-Id: I30acfc5e2dd353d711e5f4260d5b344847b03ade

This experiment was provisionally adopted at the F2F 2017-08-22.

Enabling CFL on Subset1
   PSNR |  PSNR Cb |  PSNR Cr | PSNR HVS |    SSIM | MS SSIM | CIEDE 2000
-0.5330 | -12.8717 | -10.7454 |  -0.3119 | -0.3409 | -0.3415 |    -4.8720

https://arewecompressedyet.com/?job=no-cfL%402017-10-30T17%3A14%3A33.925Z&job=cfL%402017-10-30T17%3A15%3A02.467Z

Change-Id: I2b35f352366c585887aa4fb77ae5acb159d6819f

Change-Id: I35a5334972b73bdf31d6476d376af982c964a54b

Change-Id: I9f752dedfba29de9a4cfdd285c4b6dc32bd1630d

When 4:2:2 subsampling is used, both horizontal pixels are added and
shifted left by 1 (instead of a right shift of 2 followed by a left
shift of 3).

Change-Id: Ib84c51cafabe0bd0de02dfe7868278e44d76f6db

Reset the frame index tracking to 0 when key frame is inserted in
the frame sequence.

Change-Id: Ib80313004fc000e7127a9797af5afe9029c0cb45

This reverts commit c002bdf6.

Now the issue has been tracked down to what is described in:
https://aomedia-review.googlesource.com/#/c/aom/+/29761/

Change-Id: I49fe8395781e1fea8095dbdc6c00d36b90dc7575

Change-Id: I39cfbb135add0553cadf64481b13786831fbdddd

Change-Id: I9b1351263286041d718a1b54735c17aeb5f8a0a7

I'd got the scaling backwards. This gets it right and adds a comment
explaining the calculation.

Change-Id: Ife2913700cc73996c09b702b394832799c449a8c

The frame marker system supports one to map the reference frame
index into the natural order. It allows direct checking on the
efficacy of the reference frames given their relative locations
with respect to the current coding frame.

This commit uses such property to filter out reference frames
less likely to contribute coding gains from the rate-distortion
optimization process. For example, it takes out the check on
last2 / 3 frames, when their actual location is further away
from the golden frame.

The AWCY results show 0.6% performance regression. The encoding
speed gets doubled.

To use the speed up, one needs to turn on frame-marker experiment
before we turn it on by default, and enable selective_ref_frame
entry in the speed feature.

Change-Id: Ifb03ed90acd980bbc7ff1c2e17982e21e68d2588

This experiment has been adopted, we can simplify the code
by dropping the associated preprocessor conditionals.

Change-Id: I9c9d6ef5317798cbf237307a9754fe7e03bdda47

Sub8x8 Validation code is changed to be more robust. The scope of the
validation is narrowed to validating that all of the required content in
the storage buffer was stored between CfL predictions. The early
termination used in the current mode decision code does not allow to
validate more than that.

This change does not change encoder output

BUG=aomedia:925

Change-Id: I7f1ed84da5037dcfaaf5da9cf33b4b8d664d2352

Backwards adaptation doesn't really make sense in this context,
because ext-tile means the decoder might not decode every tile in the
frame, in which case it can't work out how to adapt probabilities.

It turns out that the tests in the tree were passing when
SIMPLE_BWD_ADAPT was disabled because (presumably) all of them
happened to decode every tile, so backward adaptation worked as you'd
expect. With SIMPLE_BWD_ADAPT, there's some code in get_tile_buffers
to figure out which is the largest tile. This wasn't mirrored in
get_ls_tile_buffers, which caused the encoder and decoder to get out
of sync because they ended up with different CDFs. You also can't
easily fix things in get_ls_tile_buffers because that code is careful
not to calculate the size of tiles it doesn't need.

BUG=aomedia:917

Change-Id: Ia926692f86ca1466252108e09da3de590d45f048

Change-Id: I0cc53a03f07a11a6f7ea0570ff4ee8cf7c18c5aa

Remove the special case handling for the topmost/bottommost
rows in each processing unit. This causes slightly different
effects depending on whether striped-loop-restoration is enabled.

With striped-loop-restoration:
  Now that we explicitly fill out 3 rows of above/below pixels
  for each stripe, we don't need to use stepdown_wiener_kernel.
  Instead, the duplication of the topmost/bottommost pixels
  accomplishes the same task, while making the code much cleaner.

  This patch should not cause a change in output, except in a
  couple of cases which were already questionable. In particular,
  it fixes bug #953, where the Wiener filter could not handle
  small processing units (<4 rows high)

Without striped-loop-restoration:
  The Wiener filter returns to using a full 3 pixels above/below
  the processing unit. In order to make sure there are enough
  pixels, we need to expand WIENER_BORDER_VERT to 3 pixels.

  This will result in a slight change in output, but should be
  fairly minor.

BUG=aomedia:953

Change-Id: I9530ef55909246f7ba488b7ecfd92d59e776b2f9

Change-Id: I69d1c4b63960b46affb3f907fe8a1385f0de8948

Move #if CONFIG_EXT_TX in av1_fht32x32_test.c to be consistent as in
dct.c

BUG=aomedia:964

Change-Id: I0e7a777588f76a9088ce05ec8598f6ffa7c19e48

Replaces the lifting based orthonormal 4-point Type-IV DST with an
 orthonormal 4-point Type-VII DST that has no iterative multiplies.

Change-Id: I0a1f1a8d8cecce1c8002b7891baea601bc088690

Account for 1-D/2-D transform kernels for the eob modeling. To
maintain a smaller context cardinality, set the two 1-D transform
kernels in the same category. The difference in directions should
be largely covered by the scan order.

This and the previous CLs on nz_map context modeling together
improve the compression performance of level-map coefficient coding
system by 0.4% for lowres.

Change-Id: I8c4f03ca01ce3d248950d04bd1266f445b4227a0

Account for the rectangular transform block sizes in the non-zero
map context model.

Change-Id: I16cf21a4120c10c213df10950aeb4ef0ea40c477

Support compound_segment and wedge.
JNT_COMP is a competitor to compound_segment and wedge. They are
mutually exclusive.

Change-Id: Id93c36de44a69008182424f40325e42e699bedcd

For ext-intra direcation intra modes:

1. Use neighbor block modes to modify edge filtering strength
   Coding gain (lowres/midres/hdres):
     (8 keyframes)
     PSNR: -0.19 -0.22 -0.10
     SSIM: -0.29 -0.27 -0.13

2. Use context-based cdf to code angle_delta syntax
     (8 keyframes)
     PSNR: -0.20 -0.24 -0.27
     SSIM: -0.29 -0.33 -0.37

3. Filter corner sample:
     (8 keyframes)
     PSNR: -0.01 -0.02 -0.05
     SSIM: -0.03 -0.04 -0.05

Combined Bd-rate improvement for 8 keyframes
     PSNR: -0.40 -0.47 -0.40
     SSIM: -0.57 -0.60 -0.51

Change-Id: Id47ac17b6bf91cd810b70cacfc5b457341f417f3