1. 21 Dec, 2017 6 commits
    • Hui Su's avatar
      intrabc: enable 16x4 and 4x16 blocks · eb2fd5c5
      Hui Su authored
      0.15% gain on the screen_content testset.
      
      BUG=aomedia:998
      
      Change-Id: Ia6484a90b92a00bb0073ecf988b5c164fe8ba84c
      eb2fd5c5
    • Hui Su's avatar
      Palette: modify the context slightly · c1f411bc
      Hui Su authored
      Use the number of pixels in a block as context, rather than the bsize
      itself. The rectangular blocks therefore share the same context, e.g.
      BLOCK_8X16 and BLOCK_16X8.
      
      The number of contexts is reduced from 10 to 7.
      Almost no coding performance changes.
      
      Change-Id: Ib3241194580c2b93ad0e953957cdc9188393d055
      c1f411bc
    • Angie Chiang's avatar
      Remove all_zero check in read/write_inter_mode · ec9bebc1
      Angie Chiang authored
      This is a bitstream simplification.
      It will reduce motion vector context model generating latency.
      
      Change-Id: I98a496f5d72402ff51a478d5387a0653fa306dc1
      ec9bebc1
    • Steinar Midtskogen's avatar
      Remove CDEF_SINGLEPASS defines · 8322ff04
      Steinar Midtskogen authored
      The experiment has been adopted and has been enabled by default for a
      while and the alternative code path has not been maintained for a long
      time, which is now removed.
      
      Change-Id: Iaf22f2969b45b71b2bf67707e131ab4c439b7fa6
      8322ff04
    • Debargha Mukherjee's avatar
      Remove DISABLE_VARTX_FOR_CHROMA = 2 option. · 27b5136f
      Debargha Mukherjee authored
      Removing code for this option since it is not better than the
      DISABLE_VARTX_FOR_CHROMA = 1 option and is more complex.
      
      Change-Id: Id39d23bc6130bbed0ac008c1c76a2ba5aaee4d22
      27b5136f
    • Jingning Han's avatar
      Make ext-tile and obu work togather · da11e697
      Jingning Han authored
      This commit resolves the broken coding pipeline in ext-tile when
      obu is enabled. It fixes the unit test failure due to enc/dec
      mismatch.
      
      BUG=aomedia:1143
      
      Change-Id: I7f083c90959939048f59540f4427dc94c709a315
      da11e697
  2. 20 Dec, 2017 6 commits
    • Cyril Concolato's avatar
      Moving tile_info upwards in the frame header · 3b5afc1e
      Cyril Concolato authored
      Change-Id: Idce7902ed4f0a283d8ae6643402a450f3e5caf9a
      3b5afc1e
    • Yue Chen's avatar
      Clean ups and bug-fixes for filter_intra · 994dba25
      Yue Chen authored
      Clean up: clean up unused entropy related arrays
      Bug-fixes: resolve compiling errors when it is enabled with
      entropy_stats and ext_skip
      
      Change-Id: I2893e1b5c09e847fb4677f6e62709464208b76f8
      994dba25
    • Debargha Mukherjee's avatar
      Correct/Cleanup min_tx_size computation · 3ebb0d00
      Debargha Mukherjee authored
      Change-Id: I17fd041e9e1cead6b2b6339f0efa55c1ee9938af
      3ebb0d00
    • Imdad Sardharwalla's avatar
      Fix inconsistent use of ref_mv_idx. · c23ad632
      Imdad Sardharwalla authored
      "+ 1" has been added in a few locations to match the indices for the
      context for the CDFs and the motion vectors.
      
      Suggested fix (of removing +1s entirely) can then be implemented
      without further affecting the overall behaviour of the encoder and
      decoder.
      
      BUG=aomedia:858
      
      Change-Id: I650a618d3fda1d27bd3550d25ad1ac52e4365061
      c23ad632
    • Steinar Midtskogen's avatar
      Enable CDEF for chroma for 4:2:2 and 4:4:0 · ab6c9c77
      Steinar Midtskogen authored
      This enables CDEF for chroma when the horizontal and vertical
      subsampling differs.  Since the direction search is only performed in
      luma, the following corrections to the direction index are done for
      chroma to correct the distortion resulting from the subsampling:
      
      4:2:2:   4:4:0:
      0 -> 7   0 -> 1
      1 -> 0   1 -> 2
      2 -> 2   2 -> 2
      3 -> 4   3 -> 2
      4 -> 5   4 -> 3
      5 -> 6   5 -> 4
      6 -> 6   6 -> 6
      7 -> 6   7 -> 0
      
      This improves the chroma PSNR BDR by about 2% for 4:2:2 content at
      cpu-used=4, low delay configuration.  4:2:0 and 4:4:4 content is
      unchanged.
      
      Change-Id: Iee92d3697bc5a6fc9b1f340a63243a334935b433
      ab6c9c77
    • Frederic Barbier's avatar
      Remove EXT_PARTITION_TYPES_AB experiment · 3f53da76
      Frederic Barbier authored
      This experiment has been abandonned for AV1.
      
      Change-Id: Ic706b3533ecf502ab316505ba0a9a8eeeb296612
      3f53da76
  3. 19 Dec, 2017 4 commits
    • Zoe Liu's avatar
      Exclude distance-weighted prediction for ext-skip · 5664419e
      Zoe Liu authored
      For ext-skip, the use of distance-weighted prediction has only shown
      a coding gain in the AMCY low delay scenario. It is hence not being
      considered and compound-average is used exlusively.
      
      Change-Id: I57439eda1246e179d6404bbec366c6c610af3a36
      5664419e
    • Lei's avatar
      add new experiment loopfiltering_across_tiles_ext · 7bb501d5
      Lei authored
      based on the latest discussion in the HW working group about how loop
      filter should be integrated with tiles, the following decisions have been
      made:
      1. two seperated flages should be added for
      loop_filter_across_tiles_enabled for horizontal tile boundary and
      vertical tile boundary.
      2. encoder and decoder should only check these two flags to determine
      whether loop filtering (including deblocking, CDEF and loop restoration)
      should cross tile boundaries (vertical and/or horizontal) or not
      regardless the horitontal depepdent tile flag.
      
      This change list implemented the support for two seperated
      loop_filter_across_tiles_enabled flags for vertical and horizontal tile
      boundaries. The new experiment is disabled as default before it is
      adopted.
      
      Change-Id: I814377947517f5419c08b004a3b71b950d01eadd
      7bb501d5
    • Yue Chen's avatar
      Remove filter_intra mode info for UV plane · b0571874
      Yue Chen authored
      Change-Id: I92f73b78dfd0e25a61897e8b6e9e28ff210376d5
      b0571874
    • Sebastien Alaiwan's avatar
      Remove ADAPT_SCAN experiment · 23c511e7
      Sebastien Alaiwan authored
      This experiment has been abandonned for AV1.
      
      Change-Id: Ib3bb07d62f2025ce50dc9bc1b3f9fc22488519a7
      23c511e7
  4. 18 Dec, 2017 3 commits
    • Luc Trudeau's avatar
      [CleanUp] Remove block from av1_mode_context_analyzer · 15a18e3d
      Luc Trudeau authored
      av1_mode_context_analyzer is always called with block == -1.
      
      Change-Id: Ibc9e7e87ea95f3651f7505b82674a6fd795ae985
      15a18e3d
    • Zoe Liu's avatar
      Expand skip mode support for all comp frames · 4b847e19
      Zoe Liu authored
      All compound predicted frames now have the choice to either turn on
      or off the use of the skip mode. The current encoder design only
      turns on the use of skip mode when the temporal distances of the two
      reference frames to the current frame are only different by 1.
      
      This patch also includes a fix on the calculating of the temporal
      distance between the two reference frames to the current frame for
      skip mode.
      
      Performance wise, this patch does not have noticeable impact as the
      encoder stays with the same choice with the frame-level skip mode
      flag.
      
      Change-Id: I34f370940b3b25d2ab429b8721344133ae6288ad
      4b847e19
    • Cheng Chen's avatar
      LPF_SB: cleanup unused functions and code path · 76224b0e
      Cheng Chen authored
      Since we've decided to use guess filter levels for lpf_sb, I remove
      unused searching functions and coding paths to make codebase clean
      and easy to manage.
      
      Change-Id: I3497add59fc1eaf36acdd0acb85e3bcb6486d01d
      76224b0e
  5. 17 Dec, 2017 1 commit
  6. 16 Dec, 2017 2 commits
    • Jingning Han's avatar
      Deprecate the use of bmi structure from av1 codec · 2fac8a41
      Jingning Han authored
      Change-Id: I7f5010ae3b9a014b3dca0425c9eada3b9e2c0ab3
      2fac8a41
    • Zoe Liu's avatar
      Support ext-skip for both low delay and high delay · 104d62e1
      Zoe Liu authored
      For both low delay and high delay scenarios, the reference pair in
      skip mode are specified as the closest fwd ref, together with the
      closest bwd ref if there is any bwd ref, otherwise with the two
      closest fwd refs.
      
      Skip mode by default uses COMPOUND_AVERAGE. When all the reference
      frames are on the same side, temporal-distance weighted compound is
      considered, and a compound index is signaled to indicate whether
      distance-weighted compound or compound-average is usd.
      
      Whether to use distance-weighted compound for skip mode is still
      under experimenting, hence a flag is temporarily added:
      SKIP_MODE_WITH_JNT_COMP.
      
      Following experimental results are obtained over 30 frames, using the
      setup of --disable-ext-partition --disable-ext-partition-types
      --disable-txmg --enable-jnt-comp --enable-mfmv --enable-ext-skip:
      
      (1) High Latency:
      For Google test sets (lowres/midres), the BDRate coding gain is ~0.2%;
      For AWCY, the coding gain is ~0.1%.
      (2) Low Latency:
      No gain has been observed over Google sets and ~0.1% gain is obtained
      only when temporal-distance weighted prediction is used.
      
      Change-Id: I8c433357adebed0126ebfdd5c4d51aa71e64be57
      104d62e1
  7. 15 Dec, 2017 5 commits
  8. 14 Dec, 2017 4 commits
    • Jingning Han's avatar
      Fix enc/dec mismatch bewteen mfmv and obu · 923f827e
      Jingning Han authored
      BUG=aomedia:1123
      
      Change-Id: I3886bd74cf3ee266eee13020c8821aff5b59e11d
      923f827e
    • Luc Trudeau's avatar
      [Clean Up] Remove get_y_mode() · 2eb9b845
      Luc Trudeau authored
      The get_y_mode function, is superfluous, not used consistently, and requires a
      useless block_idx parameter than gets pass around a lot inside the
      codebase for no apparent reason.
      
      The block parameter is misleading, as it could cause people to think all these
      functions actually use this value.
      
      Change-Id: I7ae0a8d1282c009b9114c83771cce10f5c2ee397
      2eb9b845
    • Hui Su's avatar
      fix conflict between intrabc and obu · ad7536be
      Hui Su authored
      Change-Id: Ieed2451060f68fee6e67bca7d836d27bc6483c0f
      ad7536be
    • Yaowu Xu's avatar
      Revert "enable obu (HLS R19) by default" · a14de5b3
      Yaowu Xu authored
      This reverts commit 0dc0ddc0.
      
      Reason for revert: enc/dec mismatch was found and tracked down to this commit. 
      
      BUG=aomedia:1123
      
      Change-Id: I47ac289426e18d2118117d56c2cbc8930b78bda2
      a14de5b3
  9. 13 Dec, 2017 5 commits
  10. 12 Dec, 2017 3 commits
    • Rupert Swarbrick's avatar
      Fix calculation of tx_type cost for rectangular transforms · ffbff57d
      Rupert Swarbrick authored
      This patch fixes up av1_tx_type_cost to match the code in
      av1_write_tx_type. Beforehand, we wrongly assumed a 32x16 block needed
      to signal its transform size (with rect-tx-ext & rect-tx-ext-intra)
      because we were passing 16x16 to get_ext_tx_types.
      
      I've also changed av1_write_tx_type to use get_min_tx_size rather than
      inlining its body. No functional change, but it's probably better to
      use the same helper function both times.
      
      Change-Id: Iff6ee0bff2d332d5270fe0219db88c95e0b051d0
      ffbff57d
    • Cheng Chen's avatar
      Make lpf_sb work with loopfilter_level · 765e34e3
      Cheng Chen authored
      Make lpf_sb compatible with loopfilter_level, when USE_GUESS_LEVEL = 1.
      
      Filter levels will be selected based on q index and applied for
      filtering on Y, U, V planes separately.
      
      Current model only allows to guess one filter level.
      Now Y_vert = Y_horz = U = V. In the future, we need to retrain the
      model and get filter levels for Y_vert, Y_horz, U and V separately.
      
      When USE_GUESS_LEVEL = 0, lpf_sb can't work with loopfilter_level yet.
      
      Change-Id: Icd774a147c07a4035cf8204a8754b2a99668bbfd
      765e34e3
    • Cheng Chen's avatar
      LPF_SB: set row delay to work with intrabc · f18ba021
      Cheng Chen authored
      Intrabc search area can't use pixels of the last 8 rows in the current
      superblock row. Add a delay so that lpf_sb can work with intrabc.
      
      Let lpf_sb on with intrabc while other loop filtering off (cdef, loop
      restoration).
      
      Change-Id: I98ab093a17096387b6233182da2656f90dfa09eb
      f18ba021
  11. 11 Dec, 2017 1 commit