1. 16 Nov, 2012 4 commits
    • Ronald S. Bultje's avatar
      Support 32x32 intra modes in non-keyframe superblocks. · 5b11052a
      Ronald S. Bultje authored
      Change-Id: Icf8ad313c543462e523bff89690e5daa8d49bcc0
      5b11052a
    • Paul Wilkins's avatar
      Further experimentation with the mode context · a57dbd95
      Paul Wilkins authored
      Experiments with a larger set of contexts and some
      clean up to replace magic numbers regarding the
      number of contexts.
      
      The starting values and rate of backwards adaption
      are still suspect and based on a small set of tests.
      Added forwards adjustment of probabilities.
      
      The net result of adding the new context and forward
      update is small compared to the old context from the
      legacy find_near function.  (down a little on derf but
      up by a similar amount for HD)
      
      HOWEVER.... with the new context and forward update
      the impact of disabling the reverse update (which may be
      necessary in some use cases to facilitate parallel decoding)
      is hugely reduced.
      
      For the old context without forward update, the impact of
      turning off reverse update (Experiment was with SB off) was
      Derf - 0.9, Yt -1.89, ythd -2.75 and sthd -8.35. The impact was
      mainly at low data rates.
      
      With the new context and forward update enabled the impact
      for all the test sets was no more than 0.5-1% (again most at
      the low end).
      
      Change-Id: Ic751b414c8ce7f7f3ebc6f19a741d774d2b4b556
      a57dbd95
    • Yaowu Xu's avatar
      changed mv candidate search for superblocks · 415e6bff
      Yaowu Xu authored
      added additional motion vectors at close neighborhood of a superblock
      to the list of candiate motion vectors, and removed a couple that are
      further away.
      
      The change helped std-hd set about .8% (all metrics) and smaller gain
      for derf set.
      
      Change-Id: Iaa69b98614db43420ed3fd4738d0ca5587b90045
      415e6bff
    • Deb Mukherjee's avatar
      Compound inter-intra experiment · 0c917fc9
      Deb Mukherjee authored
      A patch on compound inter-intra prediction.
      
      In compound inter-intra prediction, a new predictor for
      16x16 inter coded MBs are obtained by combining a single
      inter predictor with a 16x16 intra predictor, in a manner
      that the weight varies with distance from the top/left
      boundary. The current search strategy is to combine the best
      inter mode with the best intra mode obtained independently.
      
      Results so far:
      
      derf +0.31%
      yt +0.32%
      std-hd +0.35%
      hd +0.42%
      
      It is conceivable that the results would improve somewhat
      with a more thorough search strategy where all intra modes
      are searched given the best mv, or even a joint search for
      the best mv and the best intra mode.
      
      Change-Id: I7951f1ed0d6eb31ca32ac24d120f1585bcd8d79b
      0c917fc9
  2. 15 Nov, 2012 4 commits
    • Yaowu Xu's avatar
      subpelrefmv for superblocks · 61416aed
      Yaowu Xu authored
      duplicate code clean-up and variable name corrections
      
      Change-Id: Ibc4703228e652ec425125de5e7bc038fa46595c5
      61416aed
    • John Koleszar's avatar
      support building vp8 and vp9 into a single lib · a9c7597a
      John Koleszar authored
      Change-Id: Ib8f8a66c9fd31e508cdc9caa662192f38433aa3d
      a9c7597a
    • John Koleszar's avatar
      make: flatten object file directories · b72373de
      John Koleszar authored
      Rather than building an object file directory heirarchy matching the
      source tree's layout, rename the object files so that the object
      file name contains the path in the source file tree. The intent here
      is to allow two files in different parts of the source tree to have
      the same name and still not collide when put into an ar archive.
      
      Change-Id: Id627737dc95ffc65b738501215f34a995148c5a2
      b72373de
    • John Koleszar's avatar
      detokenize: use SEG_LVL_EOB feature consistently · 6becad42
      John Koleszar authored
      Update decode_coefs() to break when c >= eob, since it's possible that
      c starts the loop from 1 and eob is 0. The loop won't terminate in that
      case.
      
      Add new get_eob() function to consistently clamp the eob based on the
      segment level EOB and the block size. It's possible to code a segment
      level EOB that's greater than the block size, and that leads to an
      out of bounds access.
      
      Change-Id: I859563b30414615cf1b30dcc2aef8a1de358c42d
      6becad42
  3. 14 Nov, 2012 6 commits
  4. 13 Nov, 2012 6 commits
  5. 12 Nov, 2012 2 commits
    • Paul Wilkins's avatar
      New inter mode context · 2669f42b
      Paul Wilkins authored
      This change is a fix / extension of the newbestrefmv
      experiment. As such it is presented without IFDEF.
      
      The change creates a new context for coding inter modes
      in vp9_find_mv_refs(). This replaces the context that
      was previously calculated in vp9_find_near_mvs().
      The new context is unoptimized and not necessarily
      any better at this stage (results pending), but eliminates
      the need for a legacy call to vp9_find_near_mvs().
      Based on numbers from Scott, this could help decode
      speed by several %.
      
      In a later patch I will add support for forward update of
      context (assuming this helps) and refine the context as
      necessary.
      
      Change-Id: I1cd991b82c8df86cc02237a34185e6d67510698a
      2669f42b
    • Paul Wilkins's avatar
      Restrict ref mv search range. · 6fb8953c
      Paul Wilkins authored
      Experiment to test speed trade off of reducing the
      extent of the ref mv search.
      
      Reducing the maximum number of tested candidates to 9 had
      minimal net effect on quality in any of the tests sets.
      
      Reduction to 7 has a small negative impact (worst was STD-HD
      at about -0.2%).
      
      This change is in response to the apparently high number of
      decode cycles reported in regard to mv-ref selection.
      
      Change-Id: I0e92e92e324337689358495a1ec9ccdeb23dc774
      6fb8953c
  6. 10 Nov, 2012 3 commits
    • Ronald S. Bultje's avatar
      Fix data type for eobs[] array in SB 4x4 IDCT code. · dd9d4f9e
      Ronald S. Bultje authored
      This fixes encoder/decoder mismatches with the superblock experiment
      turned on whenever a superblock is encoded using the 4x4 transform.
      
      Change-Id: Iefec7055e8d25f8efdbba66c4261bbd322d335a3
      dd9d4f9e
    • Ronald S. Bultje's avatar
      Remove 'thismb' data pointer when superblock experiment is on. · 73987d14
      Ronald S. Bultje authored
      This should prevent inconsistent results between identical encodes with
      the superblock experiment turned on.
      
      Change-Id: I41a005fae53f2eb59736cc70041185fb7d63cfca
      73987d14
    • Deb Mukherjee's avatar
      New b-intra mode where direction is contextual · d01357bb
      Deb Mukherjee authored
      Preliminary patch on a new 4x4 intra mode B_CONTEXT_PRED where the
      dominant direction from the context is used to encode. Various decoder
      changes are needed to support decoding of B_CONTEXT_PRED in conjunction
      with hybrid transforms since the scan order and tokenization depends on
      the actual direction of prediction obtained from the context. Currently
      the traditional directional modes are used in conjunction with the
      B_CONTEXT_PRED, which also seems to provide the best results.
      
      The gains are small - in the 0.1% range.
      
      Change-Id: I5a7ea80b5218f42a9c0dfb42d3f79a68c7f0cdc2
      d01357bb
  7. 09 Nov, 2012 2 commits
    • Deb Mukherjee's avatar
      Build fix in decoder/decodframe.c · 3f7182cb
      Deb Mukherjee authored
      Missing eobs agrument in vp9_dequant_idct_add_16x16_c
      
      Change-Id: I826b1afa0a4ee6398f7373325aa0c75e6a866937
      3f7182cb
    • Vignesh Venkatasubramanian's avatar
      Packing Altref along with succeeding frame and length encoding frames · bc9670ee
      Vignesh Venkatasubramanian authored
      The altref frame is packed along with the next P frame. So that
      outside of the codec there are now only two types of frames P and I.
      Also, now it is one frame in and one frame out with respect to the
      codec. Apart from that, all the frames are length encoded with the
      length of each frame appended to the frame itself. There are
      two categories of frames and each of them will look as follows:
      
        - Packed frames (an altref along with the succeeding p frame)
          - altref_frame_data | altref_lenngth | frame_data | length
        - Unpacked frames (all frames other than the above)
          - frame_data | length
      
      Change-Id: If1eabf5c473f7d46b3f2d026bd30c803588c5330
      bc9670ee
  8. 08 Nov, 2012 3 commits
    • Jim Bankoski's avatar
      remove macros obfuscating mv costing · c72be96b
      Jim Bankoski authored
      cleanup
      
      Change-Id: I565eee40d900e0441ad211b65ac829fc5b93d94a
      c72be96b
    • Ronald S. Bultje's avatar
      Implement tx_select for superblock encoding. · 1d4fbeb3
      Ronald S. Bultje authored
      Also split superblock handling code out of decode_macroblock() into
      a new function decode_superblock(), for easier readability.
      
      Derf +0.05%, HD +0.2%, STDHD +0.1%. We can likely get further gains
      by allowing to select mb_skip_coeff for a subset of the complete SB
      or something along those lines, because although this change allows
      coding smaller transforms for bigger predictors, it increases the
      overhead of coding EOBs to skip the parts where the residual is
      near-zero, and thus the overall gain is not as high as we'd expect.
      
      Change-Id: I552ce1286487267f504e3090b683e15515791efa
      1d4fbeb3
    • Yunqing Wang's avatar
      Optimize 16x16 dequant and idct · 6c17c9fa
      Yunqing Wang authored
      As suggested by Yaowu, simplified 16x16 dequant and idct. In decoder,
      after detoken step, we know the number of non-zero dct coefficients
      (eobs) in a macroblock. Idct calculation can be skipped or simplified
      based on eobs, which improves the decoder performance.
      
      Change-Id: I9ffa1cb134bcb5a7d64fcf90c81871a96d1b4018
      6c17c9fa
  9. 07 Nov, 2012 2 commits
    • Yaowu Xu's avatar
      merge full pixel refmv experiment · 0cedaa36
      Yaowu Xu authored
      Change-Id: Ib39ad47a7d188f3b45416937b7eeb28c3e79b74c
      0cedaa36
    • James Zern's avatar
      Fix variance (signed integer) overflow · 98473443
      James Zern authored
      In the variance calculations the difference is summed and later squared.
      When the sum exceeds sqrt(2^31) the value is treated as a negative when
      it is shifted which gives incorrect results.
      
      To fix this we force the multiplication to be unsigned.
      
      The alternative fix is to shift sum down by 4 before multiplying.
      However that will reduce precision.
      
      For 16x16 blocks the maximum sum is 65280 and sqrt(2^31) is 46340 (and
      change).
      
      This change is based on:
      16982342 Missed some variance casts
      fea3556e Fix variance overflow
      
      Change-Id: I2c61856cca9db54b9b81de83b4505ea81a050a0f
      98473443
  10. 06 Nov, 2012 8 commits
    • Yaowu Xu's avatar
      fixed function prototype · a879b4e6
      Yaowu Xu authored
      so they are consistent with actual definitions of the functions
      
      Change-Id: Ie4b4e81b3da3e288fc2edbbd2b393a5c54d2556b
      a879b4e6
    • Yaowu Xu's avatar
      group refmv experiment related functions · acadcec5
      Yaowu Xu authored
      Change-Id: Iedaa108ddb65f54d768424f9c47ad4d069b656fd
      acadcec5
    • James Zern's avatar
      fix test builds · 2e3e6857
      James Zern authored
      s/([vV][pP])8/$19/
      additionally dct.h was removed; declare the _c functions that are used
      in the tests. the TODO for conversion to parameterized tests still
      remains.
      
      Change-Id: I73db9425a57075bbb78a92693ba6b320578981cd
      2e3e6857
    • John Koleszar's avatar
      vpx: merge with master · 83b1d907
      John Koleszar authored
      Change-Id: I44b3ad780cef6f448fa17ff8e28fea87ef9cd518
      83b1d907
    • Yunqing Wang's avatar
      Convert 16x16 dct/idct to integer forms · 4626faf1
      Yunqing Wang authored
      Converted vp9_short_fdct16x16_c and vp9_short_idct16x16_c to
      integer versions.
      
      Change-Id: Ie3ec985a890ac0f4f4f5818e6f0122e00c8af69f
      4626faf1
    • James Zern's avatar
      vp9/encoder/bitstream.c: fix unused variable warnings · 0078d2f3
      James Zern authored
      Change-Id: Ibfac7e000509d2017eac9a108060e534a19fec33
      0078d2f3
    • Yaowu Xu's avatar
      silent a lot of MSVC compiler warnings · 8a336b0d
      Yaowu Xu authored
      there are still a couple type of warning left, which are related to
      double constants assigned to float type. As those would be addressed
      by the conversion of transforms into integer version. This commit
      has left those un-dealt with.
      
      Change-Id: I48fd9b489c0c27ad6b543f4177423419f929f2bb
      8a336b0d
    • James Zern's avatar
      rd_pick_inter_mode: prevent signed integer overflow · e47d9f1d
      James Zern authored
      calculate the txfm_cache difference first as both values may be
      INT64_MAX with the intent that they cancel each other out.
      
      Change-Id: I214d072458e1b24f60289974e6302af1aff7b66c
      e47d9f1d