GitLab

Also replaced "ToDo" with "TODO" in comments.

BUG=aomedia:86

Change-Id: I5cb044ce5bfdba5bde17d5546b57619ced8dbf77

Change-Id: I7b0a00627a9807c42345d3d28c3499c208c5e792

Uses a |difference| ^ 0.6 metric instead of clamped abs.
Improves lowres to -0.552%

Change-Id: Ic5159c4e20e5c4de8e7b1501bd68e014c9d1cf4b

To avoid unsigned int overflow

Change-Id: Ie070fea50b4b6c4a0e80c61371a2366dd79ed570

These transforms need to be updated to match c versions after they
are finalized for AV1.

Change-Id: Iddc3369b58ac3ce8d9c0c08724390c477ae89065

Adds a feature to recode if global motion is used for a reference but
has very few blocks in the frame actually using it.

lowres improves to -0.512% on average.

Change-Id: I61a36770e1b7103b9a27706909443c3f14ee4e42

Change-Id: I809eb52e8a632189c49b8ea0a2b5de760cc2a34c

av1_txfm.h: left shift of a negative number
av1/encoder/quantize.c: unsigned int overflow
aom_dsp/entenc.c: unsigned int overflow

Change-Id: I6143e68f7d6e2621f97900808c8ef7ee0ad0c814

av1/common/x86/av1_fwd_txfm1d_sse4.c
av1/encoder/rdopt.c

Change-Id: Ia386f78da64029b5f68b68a41485cc068dbf4098

Change-Id: I48c252cf1c8ad9fb6e94e3cadef6061688ab7ad4

The global motion detection only works on 8-bit buffers,
so any frames using 16-bit buffers are now down-converted
to 8 bits when necessary.

Change-Id: I4f88f4ccd449e73a2292cda70fe573dc49fcb8a0

There is a rare edge case in ransac(), causing the tests
AV1/DatarateTestLarge.ChangingDropFrameThresh/{4,5,6}
to fail when global motion is enabled. The sequence of events is:

* Since GLOBAL_MOTION_TYPES = 3, we try to generate a ROTZOOM model
* A model is generated at ransac.c:215, and its first 4 components
  are copied to best_params. The last four components of best_params
  are left as {0, 1, 0, 0}.
* We then finish the trial_count loop, and call find_transformation
  one final time to generate a refined model. But this fails, and
  best_params is not overwritten.
* get_wmtype decides that this is an AFFINE model, since
  wmmat[4] != -wmmat[3] and wmmat[5] != wmmat[2].
* We try to encode this global motion model. But the entropy coder
  cannot encode an AFFINE model, and crashes at daalaboolwriter.h:74
  (with OD_ASSERT(symb != -1))

The fix is to copy 8 entries to best_params regardless of the
model type we are generating, in line with the changes in
https://aomedia-review.googlesource.com/#/c/5589/

Change-Id: I6dbdfb997924f8ddf7ea3a1d557463264ea63cbe

Change-Id: I8f48db56e3f55b1d2fcc8a06aaa51e5c383e651f

In several places bits are overwritten in the bitstream. These
functions avoid zeroing bytes during writing so that this can
happen correctly when the number of bits is not 8*N.

Re-addresses the attempted fix in
133c57c3

which broke threaded encoding tests, which relied on re-using
byte buffers.

Change-Id: I682c5e3a7869eac7ad475584db8bf170d47a56c9

BUG=aomedia:79

Merge of nextgenv2 caused av1_optimize_b not to take account
of different inverse quantization when quant matrices are on.

Change-Id: I1b8da2e110ce201183be777663222e3d73c9f17b

It is possible down the road new compound modes are added in addition to
COMPOUND_AVERAGE and COMPOUND_WEDGE.

Change-Id: I85985d4c104effd34df13845344aac2a08d28019

Removes spurious ';'

Change-Id: I63c9fe1ef116d1d3c6a61a55072445a2b913b616

Increases amortization blocks. Improves results a little.
Also adds aom_clear_system_state() calls correctly to avoid
some occasional crash issues.

Change-Id: Ib776ee5ec30c70c1cd4a1fae7e7f23c2acc58ef6

Change-Id: I568365060848dc9e2171becaef4928ff430dc622

Adding the variables to the cache allows the user running cmake
to control configuration variable values using the -D parameter
passed to cmake.

Variables set by the user are used directly unless, for example,
a failing compiler flag or feature test overrides the user's
specified value. Variables not set by the user pick up the cached
default value.

BUG=https://bugs.chromium.org/p/aomedia/issues/detail?id=76

Change-Id: Ie8a80968345998c1869011d8a2c5eff01af678a3

- Avoid excessive copying
- Don't bother searching if no update can possibly offer savings
- Simplify the interface
- Remove the confusing av1_cost_upd256 macro

(cherry picked from libvpx/master commit
19e0b406)

Change-Id: Id9d9676a361fd1203b27e930cd29c23b2813ce59

Change-Id: If4a480633032d8738a84fa8173c6ebd90564f0a4

1. block_rd_txfm() : av1_optimize_b() should not be called
if pvq is enabled.
Setting t_above and t_left is missing when pvq is enabled,
so added.

2. dist_block() : The nextgen2v has new feature of computing
distortion in pixel domain for speed level = 0, 1, where pvq
works incorrectly since it requres the blank destination
buffer (w/o adding predicted pixels) passed to
inverse trasnform function.

Change-Id: Ia0ee426e796781ee56b4503ea425d447cf88ed8c

Probably these are missed while negenv2 is merged.
This patch does not fix the regression yet.

Change-Id: Ice51141adc504d16d1bb2cd6ab6c062889b5cd23

Change-Id: Ic6beb300229deb5270a2b5eb00a24369c3aac080

BUG=aomedia:80

Change-Id: I52df0b9a2284b35bef57a129870d645166ee871d

This reverts commit 133c57c3,
which appears to cause test failures in
AV1/AVxEncoderThreadTest.EncoderResultTest/*

Change-Id: I200b7a135ed65dc2c3f23b23b8c3dbf0872715fa

This commit reinstates portion of a reverted commit to fix warnings
and errors with MSVC2013 build.

Change-Id: Ibb5fd665db6d8c897a657e5994547a1f82e3f188

BUG=aomedia:81

Change-Id: Ia9c853b38463aa88e1d6c1df49a5930f78add77d

To be consistent with rest of comand line parameters.

Change-Id: Id7588ddf5f8a2b468311aa1b86cf1664a3e3c660

Change-Id: Ib56273c23b7c7cb7c44d5ed95078156f1b09144c

This is currently just a refactor and creates no change in performance.
It allows new compound types to be added easily in the future to
facilitate experiments with segmentation masks.

Change-Id: If48fed216d482454fabb45a304b4220ada0dbdee

lowres: -0.594% (BDRATE)

tempete: -5.609%
waterfall: -7.227%

Change-Id: Ia68ed6ab3dd6b5b17c3e33658c4b75be28796e8e

Change-Id: I119a6833cf6f71f02d5a40093f7d3bea97b7a7b8

Change-Id: Ie57676814833824b1f3612b60ed9b0485c7ffd52

After https://aomedia-review.googlesource.com/#/c/5589/ ,
the Global_Motion_Params and GLOBAL_MOTION_TYPE types
are redundant, and can be removed in favour of
WarpedMotionParams and TransformationType respectively.

Change-Id: I33f3796e5c469e595fd012ccdb65a672dd1bb86c

BUG=https://bugs.chromium.org/p/aomedia/issues/detail?id=76

Change-Id: I4fb81e606eb995951bd097c1746d1dea76718189

BUG=https://bugs.chromium.org/p/aomedia/issues/detail?id=76

Change-Id: I91c0d86902e3ece290b3b23ba104df426cab51a6

BUG=https://bugs.chromium.org/p/aomedia/issues/detail?id=76

Change-Id: Id4c31aeb4593c80cf6f0c40d2846ec52ba44c3bb

Implement the 3 header checks currently run by configure:
- aom/aom_integer.h
- pthread.h
- unistd.h

Note: the aom_integer.h check is only present to maintain parity
with the behavior of the original build system. The output variable
it sets is not actually used in the aom sources.

BUG=https://bugs.chromium.org/p/aomedia/issues/detail?id=76

Change-Id: I53479a1253a977d6fdb7b0d6e5233686d79315e3