1. 28 May, 2014 1 commit
  2. 23 May, 2014 2 commits
    • Paul Wilkins's avatar
      Further first pass allocation changes. · 1edbaeb0
      Paul Wilkins authored
      Further changes to first pass allocation for gf/arf groups.
      Three variables removed from TWO_PASS structure as only
      now used locally. Dont adjust gf_group_bits in the post
      encode update as this will no longer have any effect.
      
      Change-Id: Iff89b225db923fc856f5d2aedbc899f1d7d68b55
      1edbaeb0
    • Paul Wilkins's avatar
      Re-factor bit allocation in first pass. · 03eb0621
      Paul Wilkins authored
      Restructuring to allocate the bits for each frame in
      a GF group at the time the group is defined.
      
      At the moment the allocation closely mirrors what
      we had before.
      
      Also changes the default rate adjustment method to
      LONG_TERM_VBR_CORRECTION.
      
      Change-Id: Ie5793c46c6b9c888cead5d8790792efd7d60b7c1
      03eb0621
  3. 22 May, 2014 5 commits
  4. 21 May, 2014 1 commit
  5. 20 May, 2014 3 commits
  6. 19 May, 2014 1 commit
    • Paul Wilkins's avatar
      Cosmetic clean up. · f07a96fd
      Paul Wilkins authored
      Use type TWO_PASS instead of "struct twopass".
      
      Change-Id: I9d92920893bd436537b2ca19e9c9d355cca56c7c
      f07a96fd
  7. 16 May, 2014 3 commits
  8. 15 May, 2014 2 commits
  9. 14 May, 2014 2 commits
    • Paul Wilkins's avatar
      Simplify 2 pass KF bitrate allocation · 6122ca87
      Paul Wilkins authored
      Simplify the calculation of KF bitrate in similar way
      to previous patch for GF/arf.
      
      This has no impact on derf or std hd sets but gives a
      small net gain of ~0.1% for yt and yt-hd sets.
      
      Change-Id: Ida64ac1428d9c2a62adb67056fadbf0180eff030
      6122ca87
    • Paul Wilkins's avatar
      Further two pass clean up. · d677ea1f
      Paul Wilkins authored
      The variation in boost calculation for gf and arf groups
      is not significant enough to justify the extra complexity.
      Also removed some other spurious code that no longer
      has much material impact.
      
      The handling of the rare case, where the boost bits
      number is less than the number of bits a that would
      be allocated if a frame was not boosted, will be dealt
      with in a subsequent patch.
      
      This change actually helps on all sets a little by
      ~0.1% - 0.2% with slightly bigger gains on SSIM.
      
      Change-Id: Id42c1ac22a80a8c4993cfa0e51bc733eb9ed4f75
      d677ea1f
  10. 12 May, 2014 2 commits
    • Paul Wilkins's avatar
      Clean up of firstpass.c · 702e2583
      Paul Wilkins authored
      Re-factor duplicate code.
      Add two pass check for use of section_intra_rating as
      it is un-initialised in the 1 pass and rt case.
      
      Change-Id: I93120796f07961b8a21fb26e1a9f0d3d13949994
      702e2583
    • Paul Wilkins's avatar
      First pass clean up. · 00d54aa3
      Paul Wilkins authored
      One of a series of changes to clean up two pass
      allocation as precursor to support for multiple arf
      or boosted frames per GF/ARF group.
      
      This change pulls out the calculation of the total bits
      allocated to a GF/ARF group into a function, to aid
      readability and reduce the line count for define_gf_group().
      
      This change should have no material impact on output.
      
      Change-Id: I716fba08e26f9ddde3257e7d9b188453791883a3
      00d54aa3
  11. 08 May, 2014 1 commit
  12. 01 May, 2014 1 commit
    • Paul Wilkins's avatar
      Switch the default 2 pass vbr variant. · 307b6c22
      Paul Wilkins authored
      On balance Deb's  modified rate control for VBR seems
      to be outperforming especially on some low motion YT
      clips so I have switched this to be the default mode for
      now.
      
      Change-Id: I0713d430cad6425ac5c48fccdf332e12814ee44a
      307b6c22
  13. 30 Apr, 2014 1 commit
  14. 29 Apr, 2014 2 commits
  15. 24 Apr, 2014 2 commits
  16. 23 Apr, 2014 1 commit
    • Adrian Grange's avatar
      Fixed handling of regularly placed keyframes · 086ae992
      Adrian Grange authored
      The encoder was not handling requests to place keyframes at
      fixed intervals, i.e. kf_min_dist == kf_max_dist, correctly.
      
      In this case when looking to place the next keyframe it was
      accumulating stats all the way up to the end of the firstpass
      file. This patch corrects this behavior.
      
      Change-Id: I948ad9f1d7faa0c05861df588136cce3bb61d7e7
      086ae992
  17. 22 Apr, 2014 1 commit
    • Dmitry Kovalev's avatar
      Renaming "onyx" to "encoder". · ef003078
      Dmitry Kovalev authored
      Actual renames:
        vp9_onyx_if.c -> vp9_encoder.c
        vp9_onyx_int.h -> vp9_encoder.h
      
      Change-Id: I80532a80b118d0060518e6c6a0d640e3f411783c
      ef003078
  18. 21 Apr, 2014 1 commit
    • Paul Wilkins's avatar
      Change naming of end_usage parameter. · 4c7af859
      Paul Wilkins authored
      The end_useage parameter  is confusingly named since it
      now actually defines the rate control method used.
      
      Change-Id: I98912caabfe556b7af0b939a645d1336409e4d71
      4c7af859
  19. 18 Apr, 2014 2 commits
  20. 17 Apr, 2014 4 commits
    • Dmitry Kovalev's avatar
      Removing cq_target_quality from VP9_COMP. · 53dbc7f9
      Dmitry Kovalev authored
      Use cq_level (which is identical) from VP9_CONFIG instead.
      
      Change-Id: I1aaf2fcef3d2dbd4577f613d27693ff8a68989fd
      53dbc7f9
    • Paul Wilkins's avatar
      Merge two new VBR adjustment schemes. · 02e5bde3
      Paul Wilkins authored
      To make direct side by side testing this patch combines two
      VBR corrections schemes to allow more direct side by side testing.
      (The other patch was by Debargha chg id I0cd1f7...)
      
      Change-Id: I271c45e5c4ccf8de8305589000218b80d9dc3a25
      02e5bde3
    • Jim Bankoski's avatar
      add a context tree structure to encoder · e890c257
      Jim Bankoski authored
      This patch sets up a quad_tree structure (pc_tree) for holding all of
      pick_mode_context data we use at any square block size during encoding
      or picking modes.  That includes contexts for 2 horizontal and 2 vertical
      splits, one none, and pointers to 4 sub pc_tree nodes corresponding
      to split.  It also includes a pointer to the current chosen partitioning.
      
      This replaces code that held an index for every level in the pick
      modes array including:  sb_index, mb_index,
      b_index, ab_index.
      
      These were used as stateful indexes that pointed to the current pick mode
      contexts you had at each level stored in the following arrays
      
      array ab4x4_context[][][],
      sb8x4_context[][][], sb4x8_context[][][], sb8x8_context[][][],
      sb8x16_context[][][], sb16x8_context[][][], mb_context[][], sb32x16[][],
      sb16x32[],  sb32_context[], sb32x64_context[], sb64x32_context[],
      sb64_context
      
      and the partitioning that had been stored in the following:
      b_partitioning, mb_partitioning, sb_partitioning, and sb64_partitioning.
      
      Prior to this patch before doing an encode you had to set the appropriate
      index for your block size ( switch statement),  update it ( up to 3
      lookups for the index array value) and then make your call into a recursive
      function at which point you'd have to call get_context which then
      had to do a switch statement based on the blocksize,  and then up to 3
      lookups based upon the block size to find the context to use.
      
      With the new code the context for the block size is passed around directly
      avoiding the extraneous switch statements and multi dimensional array
      look ups that were listed above.   At any level in the search all of the
      contexts are local to the pc_tree you are working on (in?).
      
      In addition in most places code that used to call sub functions and
      then check if the block size was 4x4 and index was > 0 and return
      now don't preferring instead to call the right none function on the inside.
      
      
      
      Change-Id: I06e39318269d9af2ce37961b3f95e181b57f5ed9
      e890c257
    • Dmitry Kovalev's avatar
      Replacing cpu_used with speed in VP9_CONFIG. · 2c8c1f53
      Dmitry Kovalev authored
      Change-Id: I86b85b5c11388e84a48f8936330c0d920df5d1f0
      2c8c1f53
  21. 16 Apr, 2014 2 commits
    • Paul Wilkins's avatar
      Add experimental VBR adaptation method. · 085f76e5
      Paul Wilkins authored
      Add code to monitor over and under spend and
      apply limited correction to the data rate of subsequent
      frames. To prevent the problem of starvation or overspend
      on individual frames (especially near the end of a clip) the
      maximum adjustment on a single frame is limited to a %
      of its un-modified allocation.
      
      Change-Id: I6e1ca035ab8afb0c98eac4392115d0752d9cbd7f
      085f76e5
    • Dmitry Kovalev's avatar
      Cleaning up vp9_init_second_pass(). · 961594c9
      Dmitry Kovalev authored
      Change-Id: Ibf360fe0728e07de448e77e3841c49eb25d3ba2d
      961594c9