1. 10 Jan, 2018 9 commits
    • Andrey Norkin's avatar
      CICP colorspace signaling · 9e69463f
      Andrey Norkin authored
      Change-Id: I0f0cdf479091c308594a0288a0546dbdb76e4450
      9e69463f
    • Sarah Parker's avatar
      Remove variable widths for bins 1 and 2 in new-quant · 54127da3
      Sarah Parker authored
      This is the first phase of a new-quant cleanup. We are now only
      using a variable width zero bin, so we can simplify the
      implementation.
      
      Change-Id: I4da6c663a9d40b4fd08c14818cf7d7fe9b75844c
      54127da3
    • Michelle Findlay-Olynyk's avatar
      hash_based_trellis speed feature · fbab0621
      Michelle Findlay-Olynyk authored
      Add speed feature that uses hash tables to
      reuse previously found optimized coefficients
      in av1_optimize_txb. This skips some expensive
      optimize_txb calls.
      
      Currently shows no significant quality
      degredation or speed improvement, and set to off
      by default. Requires hash_me, lv_map and
      lv_map_multi. Adding to speed features required
      changing AV1_COMMON *cm to AV1_COMP *cpi in a
      chain of functions.
      
      Variations that have been tried:
      -varying the maximum eob on which the feature
      activates: 16, 32, 64. 16 currently used. 64
      has best hit rate but longer execution time.
      -varying the data hashed and the length of hashes
      (first hash is 16 bit and based on context data,
      while second hash is 16 bit and based only on
      pre-optimized qcoeff values.)
      -softening the data used for the hashes: ideally
      this would raise the number of hits, without
      compromising quality too much.
      
      Change-Id: I94f22be82f3a46637c0489d512f2e334a307575f
      fbab0621
    • Cheng Chen's avatar
      Skip unnecessary computation in trellis · 37d88736
      Cheng Chen authored
      When current coeff is found to be set as eob, nz_rd is not needed
      in the computation of rdcost. Skip it.
      
      Similarly, when has_nz_tail >= 2, computation for nz_rd should be
      skipped.
      
      Local runs show ~3% speed up.
      
      Change-Id: I176abf1d1043cab40788934b70558b816af72732
      37d88736
    • Hui Su's avatar
      Move sum_intra_stats into update_stats() · ea7320e1
      Hui Su authored
      Change-Id: Ied3252c3075e84252d8268a13d3f50066b3418e1
      ea7320e1
    • Hui Su's avatar
      intrabc: correctly store blk_skip[] · 29bd5895
      Hui Su authored
      BUG=aomedia:1203
      
      Change-Id: I03ca925734535c31a1ed6bdeb7f29b5b5d5f476c
      29bd5895
    • Sebastien Alaiwan's avatar
      Prevent warp if global_motion_params are invalid · 742f646f
      Sebastien Alaiwan authored
      BUG=aomedia:1195
      
      Change-Id: Ib643ed9d3a23b83e91d908940f5777f6d13b8b19
      742f646f
    • Jingning Han's avatar
      Check error resilient flag for use_ref_frame_mvs · 2830fd9f
      Jingning Han authored
      BUG=aomedia:1209
      
      Change-Id: If4ba99d6784c55768af57cac2b197db62ea2317e
      2830fd9f
    • Urvang Joshi's avatar
      joint_motion_search: Make assumption explicit. · 48d3e633
      Urvang Joshi authored
      Use assert to test the assumption noted in the comment.
      
      Ran some encodes with varying per-frame resolutions (using
      SUPERRES_RANDOM and RESIZE_RANDOM modes) to ensure that these
      asserts are indeed valid.
      
      Change-Id: I30021fff05f9793f80e42c5319bed02b0ae04cbd
      48d3e633
  2. 09 Jan, 2018 3 commits
  3. 08 Jan, 2018 10 commits
  4. 06 Jan, 2018 9 commits
  5. 05 Jan, 2018 8 commits
    • Yushin Cho's avatar
      [dist-8x8] Remove asserts for sub8x8 distortions · 796d4e29
      Yushin Cho authored
      Remove the asserts in rd_pick_partition(), which check
      whether the sum of distortions of sub8x8 partitions matches to
      the 8x8 distortion measured in final reconstructed 8x8 pixels,
      because the sum of the sub8x8 distortions returned from rd_pick_sb_modes()
      is sometimes different from what is measured with final reconstructed 8x8 pixels.
      
      The asserts have been provided for the purpose of sanity check with PSNR
      and triggered with "--enable-dist-8x8=1" encoder option.
      
      Change-Id: I08e331900b19077cbb325ef5117688f05095dcc7
      796d4e29
    • Jingning Han's avatar
      Make txk-sel support cfl in key frame coding · 60db3c95
      Jingning Han authored
      Properly store and update the luma component result in the key
      frame rate distortion optimization process for chroma component
      prediction. The txk-sel provides 0.15% gains for key frame coding
      on top of lv-map/multi and all other default experiments.
      
      Change-Id: I694860607b318d0a84333ed928d3de85c7676623
      60db3c95
    • Jingning Han's avatar
      Replace full txfm rd search with luma writing · 159f72c3
      Jingning Han authored
      The cfl route needs the reconstructed luma pixels in the chroma
      RD search. Replace an extra route of txfm_rd_in_plane with the
      use of av1_encode_intra_block_plane to directly write out the
      needed luma pixels in the key frame rate-distortion optimization
      process.
      
      The compression stats remain identical. The key frame encoding
      speed is on average 3% faster for a few tested 1080p clips.
      
      Change-Id: I65760b657a6d4b45bed9c4f1eca0cff1a2c274a1
      159f72c3
    • Jingning Han's avatar
      Fix txk-sel rd search in the key frames · c5c37032
      Jingning Han authored
      If the transform block selects a transform kernel type that makes
      it all zero coefficients and hits the best rd cost, store such
      decision and force the transform block to be coded as all zero
      block in the final coding pass. This makes txk-sel improve the
      kf coding performance by 0.15% now.
      
      Change-Id: I4bc3c185b9c68daefb856276da45bbf9e317d02e
      c5c37032
    • Sebastien Alaiwan's avatar
      Revert "Turn lv_map lv_map_multi on by default" · 76658d2f
      Sebastien Alaiwan authored
      This reverts commit 2944f9d1.
      
      Reason for revert: it breaks encode+decode with error message:  
      "Warning: Failed to decode frame 1: Bitstream not supported by this
      decoder".
      
      BUG=aomedia:1193
      
      Change-Id: I43e0adae0f000982455ed4d838c0f8e841f77af0
      76658d2f
    • Zoe Liu's avatar
      Reduce the unnecessary array size · 316a6235
      Zoe Liu authored
      For partition types pruning in ext-partition-types, horz_rd and
      vert_rd are used to store the RD costs for the two blocks in
      horizontal partition and vertical partition. Hence array size of
      2 is sufficient for either array.
      
      Change-Id: I3471ac573665410af4c5b4af3c54bc60fe8c0195
      316a6235
    • Frederic Barbier's avatar
      Remove experimental flag of SIMPLE_BWD_ADAPT · f57a937c
      Frederic Barbier authored
      This experiment has been adopted, we can simplify the code
      by dropping the associated preprocessor conditionals.
      
      Change-Id: I09423f67995ea4bea6f1de71bfeef7f6f1c2ae19
      f57a937c
    • Debargha Mukherjee's avatar
      Resolve a static analysis unstable build · e2f6b166
      Debargha Mukherjee authored
      Change-Id: Idfb819b0500aa8a7f451fc57dc35ccdba63dd836
      e2f6b166
  6. 04 Jan, 2018 1 commit