1. 23 Aug, 2017 11 commits
  2. 22 Aug, 2017 18 commits
    • Sarah Parker's avatar
      Refactor palette functions to write a general color map · 0cf4d9f1
      Sarah Parker authored
      This will allow the mrc-tx experiment to pass in its own color map
      and set of probabilities to the tokenize and bit packing functions.
      The corresponding change on the decoder side will come in a followup
      shortly.
      
      No change in performance.
      
      Change-Id: I1872b7f1b1cf5b102e6289388ce1ae449e67cb4d
      0cf4d9f1
    • Rupert Swarbrick's avatar
      Refactor iteration over neighbours for OBMC · c0cea7f3
      Rupert Swarbrick authored
      There are six pieces of code in reconinter.c and two in rdopt.c which
      iterate over the blocks along the top or left edge of the current
      block for OBMC. Before this patch, each bit of code has its own
      implementation of the iteration, which is reasonably finicky to get
      right.
      
      This patch factors out that logic into a pair of helpers
      (foreach_overlappable_nb_above and foreach_overlappable_nb_left). The
      functions take a "fun" parameter, which contains the loop body. Note
      that the iteration is too complicated for us to be able to define a
      macro that could be used like
      
        FOREACH_NB_ABOVE(rel_pos, nb_size, nb_mi) { ... }
      
      While C's syntax doesn't seem to let you do that, once the compiler's
      optimisation pass is done inlining everything, the results are
      essentially the same.
      
      The iteration logic is also slightly generalised: the old code checked
      whether a block was shorter or narrower than 8 pixels by comparing a
      block size with BLOCK_8X8. This doesn't work when you have a 4x16 or
      16x4 block because e.g. BLOCK_16X4 is not less than BLOCK_8X8. This
      generalisation is (unsurprisingly) needed in order to to support 16x4
      or 4x16 blocks.
      
      This patch doesn't address the CONFIG_NCOBMC functions in reconinter.c
      that do prediction from right and bottom edges.
      
      This patch shouldn't affect the generated bitstream in any way: the
      code is supposed to be equivalent.
      
      Change-Id: I9e5a116b012c18645604a7d98fb98be99697d363
      c0cea7f3
    • Sebastien Alaiwan's avatar
      av1_txfm_test.cc: factorize 'dct_matrix' · bdb7e9cd
      Sebastien Alaiwan authored
      Make it explicit that:
      - forward tx is using the matrix
      - inverse tx is using the transposed matrix
      
      Change-Id: Ia9507f4f68397b07b6fd1868bb421bb9d5dd817d
      bdb7e9cd
    • David Barker's avatar
      Fix ASan errors in SSE4.1 selfguided filter tests · 67a5e148
      David Barker authored
      The selfguided filter code was sometimes fetching 8 bytes of data
      when it only needed 4. This was normally fine, but lead to problems
      in the selfguided filter test when compiling for x86-32, where we
      accidentally read off the end of the input buffer.
      
      Fix this by only reading the amount of data we actually need.
      
      BUG=aomedia:700
      
      Change-Id: I2448b7b0d9cb2f9292a092675a66da64c89f913c
      67a5e148
    • Yaowu Xu's avatar
      Revert "Prevent access of array using -1 index" · 03dc6799
      Yaowu Xu authored
      This reverts commit 38cdce8e.
      
      Reason for revert: <INSERT REASONING HERE>
      
      It is reported that this introduced a segment fault, revert now to investigate. 
      
      Change-Id: I229dcdc09462ae4eb9cf2a3da4ce59aa9eafe65e
      03dc6799
    • Cheng Chen's avatar
      Silence warnings when warped_motion is disabled · 811bba79
      Cheng Chen authored
      Change-Id: I5da893be07b3b9aa4351b5bb1af987f4992370ef
      811bba79
    • Sebastien Alaiwan's avatar
      idct.c: fix left shift of negative value · 77323261
      Sebastien Alaiwan authored
      This is undefined behaviour in C and might confuse the optimizer, leading to incorrect code.
      
      Change-Id: Ia4bb60478068da678f013bdd6ab6a49814d89ebe
      77323261
    • Debargha Mukherjee's avatar
      Change 4-tap filter to be smoother · 6ec3d6e4
      Debargha Mukherjee authored
      Improves reconstruction performance.
      
      Change-Id: Ia30a345e1f5d4e450138c1dd3a7b700119c7de31
      6ec3d6e4
    • Lester Lu's avatar
      Refactor lgt · 918fe698
      Lester Lu authored
      Change get_lgt in order to integrate a later experiment
      lgt_from_pred with lgt. There are two main changes.
      
      The main purpose for this change is to unify get_fwd_lgt and
      get_inv_lgt functions into a get_lgt function so the lgt basis
      functions can always be selected through the same function in
      both forward and inverse transform paths. The structure of those
      functions will also be consistent with the get_lgt_from_pred
      functions that will be added in the lgt-from-pred experiment.
      
      These changes have no impact on the bitstream.
      
      Change-Id: Ifd3dfc1a9e1a250495830ddbf42c201e80aa913e
      918fe698
    • Jingning Han's avatar
      Account for cdf model update in lv-map coding · 8f661605
      Jingning Han authored
      Make the lv-map rate-distortion optimization account for the
      per symbol cdf update. This improves the level map coding
      performance by 0.4%.
      
      Change-Id: I1cbecafbad4e6642abc3782c9741636d92d87eb7
      8f661605
    • Jingning Han's avatar
      Add jnt_comp experimental flag · dca9818c
      Jingning Han authored
      Change-Id: Icdce2ac2203bacdcbbbf17d1fc3308dc1ed15974
      dca9818c
    • Jingning Han's avatar
      Add mfmv experiment flag · c4fcd626
      Jingning Han authored
      Change-Id: Ife4e7e4f62b217fd55db1ec3ef69c243f1651777
      c4fcd626
    • Jingning Han's avatar
      Support per symbol cdf update in lv-map encoding · 24b0cf96
      Jingning Han authored
      Allow per symbol cdf probability model update in the lv-map
      encoding process.
      
      Change-Id: Id0c4a659c04ffa981d8f04537eda2f1304d1682a
      24b0cf96
    • Jingning Han's avatar
      Make lv-map decoding support per symbol cdf update · bb72cafa
      Jingning Han authored
      Support the per symbol cdf update at the level map syntax element
      decoding process.
      
      Change-Id: Id58bf86fb1c7c72dd14ddaa3781caaa27f446ef7
      bb72cafa
    • Jingning Han's avatar
      Initialize lv-map syntax probability model · fdaa55ed
      Jingning Han authored
      Initialize the cdf model for level map syntax elements.
      
      Change-Id: I3865e07c126eb4c856803c12485b05782dea6526
      fdaa55ed
    • Debargha Mukherjee's avatar
      Simple upscaler w/ frame superres + loop-rest · 3897e2df
      Debargha Mukherjee authored
      When frame-superres is used with loop-restoration use a 4-tap
      upscaler to reduce complexity.
      
      Change-Id: Idf5712b9a006763c4000e264176b04aa8331b12f
      3897e2df
    • James Zern's avatar
      rdopt: make ADST_FLIP_SVM array static · 67932797
      James Zern authored
      Change-Id: Iec280ba1e314849ac027085f61e62a48dd224be4
      67932797
    • Debargha Mukherjee's avatar
      Adds the option to use 5x5 Wiener for chroma · 1cb757cb
      Debargha Mukherjee authored
      Change-Id: I1b789acc18f1e69fb5db069ccd8bd17815938e9d
      1cb757cb
  3. 21 Aug, 2017 11 commits
    • Sebastien Alaiwan's avatar
      Add inverse accuracy test for inv_tx_fm1d · d02642f2
      Sebastien Alaiwan authored
      Change-Id: I3cc9cb8135ec5e8aa20c8cff901a2f37b3b459ee
      d02642f2
    • Yaowu Xu's avatar
      Prevent access of array using -1 index · 38cdce8e
      Yaowu Xu authored
      BUG=aomedia:685
      
      Change-Id: I0698b281fbaff2e77a50f5dc9a10d752de6c6f79
      38cdce8e
    • Debargha Mukherjee's avatar
      Silence a compiler warning in detokenize · 415d82ca
      Debargha Mukherjee authored
      Change-Id: Idcc4c4722bd0379c8667b60115bfd85cf779a6ac
      415d82ca
    • Rupert Swarbrick's avatar
      Obey do_average flag when doing convolve_round · 07089c68
      Rupert Swarbrick authored
      Doing this means that we don't have to memset temporary buffers to
      zero in reconinter.c, which was taking ~5% of cycles in a short
      encoding test (using perf to attach to a running encode).
      
      Change-Id: Ibb6e31920000b876c6ee99f454d89c8a97e9fb31
      07089c68
    • Angie Chiang's avatar
      Let DISABLE_TRELLISQ_SEARCH configurable · 91a1cf91
      Angie Chiang authored
      Change-Id: I48ebf352c6c28e5c0c0e477b24828f0e3fe1dedb
      91a1cf91
    • Angie Chiang's avatar
      Change av1_cost_coeffs_txb's interface · 3627de2c
      Angie Chiang authored
      Change-Id: Ie7c216218bd233e74970b261186df8f08aca6193
      3627de2c
    • Angie Chiang's avatar
      Use table lookup for computing base ctx · def11251
      Angie Chiang authored
      This will speed up lv_map by 4.3%
      
      Change-Id: I886f67b9f1ae2e5f567e114fa34cf3ba437b8381
      def11251
    • Angie Chiang's avatar
      Use || to replace && logics in try_level_down() · 0c89dca7
      Angie Chiang authored
      Change-Id: I6e9e1c94f688ed84c49d3b861dbac15c37248827
      0c89dca7
    • Joe Young's avatar
      [cb4x4][refmv] scan_row_mbmi/scan_col_mbmi · 8e4c3a74
      Joe Young authored
      For sub-8x8 blocks, scan loop should have at least one iteration.
      Previously, loop would be skipped if col_offset == 1 and n8_w == 1.
      
      This change appears to have neutral effect on bdrate (+0.01).  This could
      mean that extra MV candidates are not helping that much for sub-8x8.
      
      Change-Id: Ic778312b39113ab8f348f5c7c18813593a831b7c
      8e4c3a74
    • Rupert Swarbrick's avatar
      Hide definitions behind #if ! CONFIG_EXT_PARTITION · a74fc4c1
      Rupert Swarbrick authored
      The orders_32x128 and orders_128x32 variables are only used when
      CONFIG_EXT_PARTITION is false. Only define them in that case to avoid
      a compiler warning.
      
      Change-Id: Iebe297eff36b139caa83d039101cf16eb88ca8f8
      a74fc4c1
    • Zoe Liu's avatar
      Tune the decision on the GF group interval · 4c3d4767
      Zoe Liu authored
      The change is purely an encoder-side tuning, which constrains a GF
      group interval to be either 8, or 12, or a number larger than 12
      (maximum of 16). The change is under ext_refs.
      
      This work has been part of the summer internship effort by chendixi@.
      
      Further, this CL removes the experiment of flex_refs and merges it
      into ext_refs, as flex_refs has been completely focused on the encoder
      side optimization for ext_refs.
      
      On Google test sets, above (stillness + interval enforcement) achieve
      a coding gain in BDRate as follows, compared against the AV1 baseline
      with all tools enabled by default (excluding ext-tx, global-motion,
      and convolve-round for speed concern):
      
      lowres: avg_psnr -0.193%; ovr_psnr -0.370%; ssim -0.332%
      midres: avg_psnr -0.556%; ovr_psnr -0.680%; ssim -0.798%
      
      Change-Id: Ia992d8ef0d82cc0ef2caba5eec9b26f0358e6bde
      4c3d4767