1. 01 Jun, 2017 7 commits
  2. 31 May, 2017 5 commits
  3. 30 May, 2017 1 commit
  4. 29 May, 2017 2 commits
  5. 28 May, 2017 2 commits
  6. 27 May, 2017 1 commit
  7. 26 May, 2017 5 commits
    • Jingning Han's avatar
      Fix enc/dec mismatch in filter-intra and chroma-sub8x8 · 62946d17
      Jingning Han authored
      BUG=aomedia:561
      
      Change-Id: I6dc0fd97f29bfa0302abb6e90802674c1aa65674
      62946d17
    • Jingning Han's avatar
      Cover intra tx size count in rect-tx · 2d4fafac
      Jingning Han authored
      Account for intra tx size case when coding block is in skip mode.
      
      Change-Id: I030bc05d890c4a7c8d4628a7f9bdebaba6083a03
      2d4fafac
    • Yue Chen's avatar
      ext_inter: remove unnecessary rate parameters · 5e606544
      Yue Chen authored
      Only use compmode_interinter_cost and compmode_interintra_cost in
      handle_inter_mode() since outside the function they can be reflected
      by the total rate.
      Plus adding config flags to separate wedge/compound_segment from
      ext_inter
      
      Change-Id: I56b402aab3cf2337bae8d7e682f93cd8557f69a4
      5e606544
    • Yi Luo's avatar
      Improve the quantization table interface · c621023e
      Yi Luo authored
      - Add av1_build_quantize() function so that quantization
        related unit test would be able to access quant/dequant
        table without starting an encoder instance and directly
        accessing cpi.
      
      Change-Id: I8ba429e5deb7a4e7f967996aaec1d20bff0feb3c
      c621023e
    • Jingning Han's avatar
      Initialize min_tx_size in var-tx · 923377b8
      Jingning Han authored
      Avoid the risk of use uninitialized value when TX_2X2 is moved
      behind chroma-2x2 flag.
      
      Change-Id: Ibb3e3700ab82d7f98ebe58fae70afbf64f5773c7
      923377b8
  8. 25 May, 2017 7 commits
    • Debargha Mukherjee's avatar
      Frame level flags to ctrl ext-inter compound modes · 9e2c7a65
      Debargha Mukherjee authored
      Change-Id: I904283119d8f2c1099e6ec2953ea1c10c5e3b280
      9e2c7a65
    • Jingning Han's avatar
      Fix an enc/dec mismatch issue in rect-tx · 1341cbac
      Jingning Han authored
      This commit resolves a rare enc/dec mismatch issue due to the
      intracacy of using rectangular transform block size for intra
      coded blocks.
      
      BUG=aomedia:562
      
      Change-Id: I10c52339a995a89bafa3a3330469ee3f2b784768
      1341cbac
    • Yi Luo's avatar
      Add HBD build to av1_quantize_fp_sse2 · bf8af7e6
      Yi Luo authored
      - This change turns on low bit depth data path for
        this function under default HBD build.
      - Encoder user level encoding time reduces ~12%
        on i7-6700.
      
      Change-Id: I7ce21e8db1a379f972e51c3b4ab305ca10e41efb
      bf8af7e6
    • Nathan E. Egge's avatar
      Remove the DAALA_EC experiment. · 476c63c1
      Nathan E. Egge authored
      This patch forces DAALA_EC on by default and removes the dkbool coder.
      
      Change-Id: Icd2ff08efd7bf467adf554344111473cb357adf8
      476c63c1
    • David Barker's avatar
      Fix bug in av1_optimize_b when tx64x64 is enabled · c747a78e
      David Barker authored
      In both versions of av1_optimize_b, the code to calculate the
      dequantized coefficient from a modified quantized coefficient
      was not compatible with tx64x64.
      
      Specifically, the new dequantized coefficient was calculated as an
      offset applied to the original coefficient. This had a fixup block
      for 32x32 transforms, but not for 64x64 transforms.
      
      For simplicity, instead of extending the fixup block, we now
      recalculate the dequantized coefficient directly.
      
      BUG=aomedia:537
      
      Change-Id: I86c1c8820f324ad3f6afd6a720ff59f754384a19
      c747a78e
    • Alex Converse's avatar
      var_tx+intrabc: Fix infinite recursion · e16b2665
      Alex Converse authored
      Change-Id: Ibbf0c14934d0bff316cfdf5c252c64b3e68c73e7
      e16b2665
    • Jingning Han's avatar
      Access bottom-right 4x4 block in sub8x8 neighbor for obmc · ad586b91
      Jingning Han authored
      When the neighbor block is coded in sub8x8 block size, use the
      bottom-right 4x4 luma block coding mode to produce obmc reference.
      
      Change-Id: Icecccf5ddf95e8b9e65b36fdb6fa176691591a8e
      ad586b91
  9. 24 May, 2017 3 commits
    • David Michael Barr's avatar
      [CFL] Calculate SSE for distinct alpha before RDO · 84a44dbe
      David Michael Barr authored
      Separate the codes into a table of distinct values
      and an index into that table.
      Pull the SSE calculation of the RDO loop and avoid
      repeating for the same alpha values.
      
      Change-Id: I8c4bd7eab6f8000e6aca9687d9190abc3e270c37
      Signed-off-by: default avatarDavid Michael Barr <b@rr-dav.id.au>
      84a44dbe
    • Jingning Han's avatar
      Set mi pointer before setting the skip flag · 9a80e7c4
      Jingning Han authored
      This resolves a potential race issue in multi-threading test when
      chroma-sub8x8 is enabled.
      
      Change-Id: I443f4af10e55046b04d6d739f7594dd8bda5c830
      9a80e7c4
    • David Barker's avatar
      ext-inter: Further cleanup · f19f35f7
      David Barker authored
      * Rename the 'masked_compound_*' functions to just 'masked_*'.
        The previous names were intended to be temporary, to distinguish
        the old and new masked motion search pipelines. But now that the
        old pipeline has been removed, we can reuse the old names.
      
      * Simplify the new ext-inter compound motion search pipeline
        a bit.
      
      * Harmonize names: Rename
        aom_highbd_masked_compound_sub_pixel_variance* to
        aom_highbd_8_masked_sub_pixel_variance*, to match the naming of
        the corresponding non-masked functions
      
      Change-Id: I988768ffe2f42a942405b7d8e93a2757a012dca3
      f19f35f7
  10. 23 May, 2017 1 commit
    • David Barker's avatar
      ext-inter: Delete dead code · 0f3c94e1
      David Barker authored
      Patches https://aomedia-review.googlesource.com/c/11987/
      and https://aomedia-review.googlesource.com/c/11988/
      replaced the old masked motion search pipeline with
      a new one which uses different SAD/SSE functions.
      This resulted in a lot of dead code.
      
      This patch removes the now-dead code. Note that this
      includes vectorized SAD/SSE functions, which will need
      to be rewritten at some point for the new pipeline. It
      also includes the masked_compound_variance_* functions
      since these turned out not to be used by the new pipeline.
      
      To help with the later addition of vectorized functions, the
      masked_sad/variance_test.cc files are kept but are modified
      to work with the new functions. The tests are then disabled
      until we actually have the vectorized functions.
      
      Change-Id: I61b686abd14bba5280bed94e1be62eb74ea23d89
      0f3c94e1
  11. 22 May, 2017 4 commits
    • Jingning Han's avatar
      Use 8x8 block step size in obmc reference setup · 680b09ea
      Jingning Han authored
      The obmc assumes minimum processing block size to be 8x8.
      
      BUG=aomedia:554
      
      Change-Id: Ic0415f433e8e463fffb13559703a315f317607d3
      680b09ea
    • Timothy B. Terriberry's avatar
      chroma_sub8x8: Fix av1_get_entropy_contexts() · 32b8d46f
      Timothy B. Terriberry authored
      This was not clamping the chroma block size to at least 4x4, even
      though it is getting contexts for coding the transform residual
      (which is at least 4x4). Thus it would copy too few contexts.
      
      This is an encoder-only change. It should not change the current
      encoder output, because we currently store twice as many entropy
      contexts as actually required, so we'll copy 1 and then check 2 to
      see if either are non-zero. However, this will break when we
      eliminate the context doubling, as instead we'll copy 0 and check
      1.
      
      Change-Id: I4087529d6b3e7cd5a226746625b90494790983e6
      32b8d46f
    • Yaowu Xu's avatar
      Revert "dependent tile set to default" · 551d0fed
      Yaowu Xu authored
      This reverts commit 05c38874.
      
      BUG=aomedia:553
      
      Change-Id: Ibbfe3e357bbfb8cb7d07482755c47c64afd30092
      551d0fed
    • Jingning Han's avatar
      Support multi-thread encoding for var-tx · 770e6563
      Jingning Han authored
      This commit resolves the encoding results disparity between single
      and multi-thread encoding.
      
      BUG=aomedia:553
      
      Change-Id: I968972e6d53e1977384d1dfd2f006d04af074be6
      770e6563
  12. 21 May, 2017 2 commits
    • Timothy B. Terriberry's avatar
      cb4x4: Don't assume TX units are half MI_SIZE in skip context indexing. · 5e81643d
      Timothy B. Terriberry authored
      Currently the "transform units" used to measure transform sizes
      in some parts of the code are based on the smallest defined
      transform size.
      
      cb4x4 currently defines a 2x2 transform size, even when chroma_2x2
      is not enabled, which means that the scale of the transform units
      was always double that of MODEINFO units.
      
      Several areas of the code were hard-coding this assumption instead
      of converting from one to the other using appropriate constants.
      
      Change-Id: Ibc55671aa5bc3ad272cb8a036f9c4f9621df85ab
      5e81643d
    • Timothy B. Terriberry's avatar
      cb4x4: Fix reset_skip_context() without chroma_2x2 · a2d5cdef
      Timothy B. Terriberry authored
      reset_skip_context() was always clearing the entropy contexts for
      all three color planes, using a block size that corresponded with
      the luma plane.
      
      However, when chroma_2x2 is disabled, then for sub-8x8 luma block
      sizes, the corresponding chroma block size is always 4x4, and the
      skip flag only affects the chroma blocks corresponding to the
      upper-left luma block.
      
      This patch makes reset_skip_context() reset the contexts that
      actually correspond to the chroma blocks that are skipped (if any).
      It also moves reset_skip_context() to av1_reset_skip_context() in
      blockd.c, because blockd.h gets included before onyx_int.h, which
      declares the required is_chroma_reference() function.
      reset_skip_context() was too large and used in too many places to
      be a reasonable candidate for inlining, anyway.
      
      AWCY results on objective-1-fast:
      
      cb4x4-fix-base@2017-05-11T06:26:50.159Z -> cb4x4-fix-reset_skip@2017-05-11T06:28:45.482Z
        PSNR | PSNR Cb | PSNR Cr | PSNR HVS |   SSIM | MS SSIM | CIEDE 2000
      0.0301 |  0.1068 |  0.1463 |   0.0359 | 0.0260 |  0.0347 |     0.0479
      
      A regression (near the noise range), but without this fix, the line
      buffer size required by the entropy contexts will be doubled.
      
      Change-Id: I12fa6e60d9c1c7c85927742775a346ea22b3193f
      a2d5cdef