Skip to content
Snippets Groups Projects
  1. Mar 11, 2011
    • Paul Wilkins's avatar
      1 Pass CQ and VBR bug fixes · 2ae91fbe
      Paul Wilkins authored
      Issue 291 highlighted  the fact that CQ mode was not working
      as expected in 1 pass mode,
      
      This commit fixes that specific problem but in so doing I also
      uncovered an overflow issue in the VBR code for 1 pass and
      some data values not being correctly initialized.
      
      For some clips (particularly short clips), the resulting
      improvement is dramatic.
      
      Change-Id: Ieefd6c6e4776eb8f1b0550dbfdfb72f86b33c960
      2ae91fbe
  2. Mar 09, 2011
  3. Mar 08, 2011
  4. Mar 07, 2011
    • John Koleszar's avatar
      Merge "Fix format-string warning" · 1016b856
      John Koleszar authored
      1016b856
    • Ralph Giles's avatar
      Fix format-string warning · fe9a604b
      Ralph Giles authored
      Cast size_t to (unsigned long) and print it with the %lu format
      string, which is more portable than C99's explict %zu for size_t.
      
      This truncates on Windows x64 but otherwise works on 32 and 64 bit
      platforms. In practice the stats file is unlikely to be so large.
      
      Change-Id: I0432b3acf85fc6ba4ad50640942e1ca4614b21cb
      fe9a604b
    • Paul Wilkins's avatar
      Improved key frame detection. · bc9c30a0
      Paul Wilkins authored
      In some cases where clips have been encoded with
      borders (eg. some wide-screen content where there is a
      border top and bottom and slide shows containing portrait
      format photographs (border left and right)) key frames were
      not being correctly detected.
      
      The new code looks to measure cases where a portion of
      the image can be coded equally easily using intra or inter
      modes and where the resulting error score is also very low.
      These "neutral" areas are then discounted in the key frame
      detection code.
      
      Change-Id: I00c3e8230772b8213cdc08020e1990cf83b780d8
      bc9c30a0
    • Paul Wilkins's avatar
      Improved KF insertion after fades to still. · 9fc8cb39
      Paul Wilkins authored
      This code extends what was previously done for GFs, to pick
      cases where insertion of a key frame after a fade (or other
      transition or complex motion)  followed by a still section, will
      be beneficial and will reduce the number of forced key frames.
      
      Change-Id: If8bca00457f0d5f83dc3318a587f61c17d90f135
      9fc8cb39
  5. Mar 04, 2011
  6. Mar 03, 2011
    • Mikhal Shemer's avatar
      Configuration updates:Making a clear distinction between Init and Change · 84f7f209
      Mikhal Shemer authored
      Change-Id: I7b2fb326e1aabc08b032177a7b914a5b8bb7376f
      84f7f209
    • Mikhal Shemer's avatar
      Fixing divide by zero · 1de99a2a
      Mikhal Shemer authored
      Change-Id: I9d8a98a2f7ed1e3116d0bae35164618c41998bac
      1de99a2a
    • John Koleszar's avatar
      Fix drastic undershoot in long form content · 36be4f7f
      John Koleszar authored
      When the modified_error_left accumulator exceeds INT_MAX, an incorrect
      cast to int resulted in a negative value, causing the rate control to
      allocate no bits to that keyframe group, leading to severe undershoot
      and subsequent poor quality.
      
      This error was exposed by the recent change to the rolling target and
      actual spend accumulators in commit 305be4e4 which fixed them to
      actually calculate the average value rather than be re-initialized
      on every frame to the average per-frame bitrate. When this bug was
      triggered, the target bitrate could be 0, so the rolling target
      becomes small, which causes the undershoot. The code prior to 305be4e4
      did not exhibit this behavior because the rolling target was always
      set to a reasonable value and was independent of the actual target
      bitrate. With this patch, the actual target bitrate is calculated
      correctly, and the rate control tracks as expected.
      
      This cast was likely added to silence a compiler warning on a comparison
      between a double (modified_error_left) and an int (0). Instead, this
      patch removes the cast and changes the comparison to be against 0.0,
      which should prevent the warning from reoccuring.
      
      This fixes issue #289. Special thanks to gnafu for his efforts in
      reporting and debugging this fix.
      
      Change-Id: Ie5cc1a7b516c578a76c3a50c892a6f04a11621fe
      36be4f7f
Loading