1. 04 Sep, 2017 6 commits
    • clang-format's avatar
      apply clang-format · 4eafefe0
      clang-format authored
      Change-Id: If0b48a4ee1f7902d8c6154945ccef68a2b5aabb5
      4eafefe0
    • James Zern's avatar
      .clang-format: update to 4.0.1 · a3a70c38
      James Zern authored
      based on Google style with the following differences:
      3a4
      > # Generated with clang-format 4.0.1
      13c14
      < AllowShortCaseLabelsOnASingleLine: false
      ---
      > AllowShortCaseLabelsOnASingleLine: true
      23c24
      < BraceWrapping:
      ---
      > BraceWrapping:
      43c44
      < ConstructorInitializerAllOnOneLineOrOnePerLine: true
      ---
      > ConstructorInitializerAllOnOneLineOrOnePerLine: false
      46,47c47,48
      < Cpp11BracedListStyle: true
      < DerivePointerAlignment: true
      ---
      > Cpp11BracedListStyle: false
      > DerivePointerAlignment: false
      51c52
      < IncludeCategories:
      ---
      > IncludeCategories:
      78c79
      < PointerAlignment: Left
      ---
      > PointerAlignment: Right
      80c81
      < SortIncludes:    true
      ---
      > SortIncludes:    false
      
      Change-Id: I0a73f0f984a7730e6448c1fadc4cf0b9440b9226
      a3a70c38
    • Rupert Swarbrick's avatar
      Check for early end of data when reading tiles · cd75739f
      Rupert Swarbrick authored
      BUG=aomedia:709
      
      Change-Id: I26f8938a744f7ebfd9734929502730b17de348f9
      cd75739f
    • Rupert Swarbrick's avatar
      Replace an assertion with a proper error on bad bitstream · 5c73c003
      Rupert Swarbrick authored
      The example in bug 712 is a bitstream that signals a global motion
      type of ROTZOOM, but its second frame has shear parameters that fail
      the is_affine_shear_allowed check at warped_motion.c:754. This is
      quite possible (and it's not obvious how to change the bitstream
      format so that you can't signal something like this).
      
      This patch replaces the failing assertion with a proper "no you
      can't!" error.
      
      BUG=aomedia:712
      
      Change-Id: I6a32632d17031b777acd2f78a887491a40177785
      5c73c003
    • Rupert Swarbrick's avatar
      Fix argument to finer_search_pixel_proj_error · 1c8cdef5
      Rupert Swarbrick authored
      Patch https://aomedia-review.googlesource.com/c/aom/+/20200, merged as
      32d150b6, converted several functions in pickrst.c to take a
      "use_highbitdepth" flag as well as (or instead of) the actual bit
      depth.
      
      Unfortunately, I missed a call site and the code can end up passing
      the number 8 as the flag for use_highbitdepth (and, since 8 != 0, this
      ends up using the high bit depth patch).
      
      BUG=aomedia:714
      
      Change-Id: Ie4dbad92f57ea1bacc4d99aad15454d9e5b6ff47
      1c8cdef5
    • Rupert Swarbrick's avatar
      Fix loop-restoration with 8-bit data on highbd path · 32d150b6
      Rupert Swarbrick authored
      The code was incorrectly using "bit_depth == 8" as a test for whether
      to use the highbd path or not.
      
      BUG=aomedia:714
      
      Change-Id: Ib3995dcda949adfe9307bc4c8273c6c375c5a2c7
      32d150b6
  2. 03 Sep, 2017 1 commit
    • Rupert Swarbrick's avatar
      Move loop restoration coefficients to within the frame · 6c545216
      Rupert Swarbrick authored
      Rather than encoding the loop restoration coefficients at the start of
      the frame header, this patch moves them to occur just after certain
      top-level superblocks.
      
      You might hope that we could just encode coefficients on top-level
      superblocks where the top-left corner of the superblock was also the
      top-left corner of the loop restoration tile. Unfortunately, this
      can't work with the superres experiment, where the loop restoration
      tiles don't necessarily line up with the superblocks. Indeed, in
      general there can be multiple different loop restoration coefficients
      that apply in a given top-level superblock. This patch defines a
      function, av1_loop_restoration_corners_in_sb, which yields the
      rectangle [rrow0, rrow1) x [rcol0, rcol1) of loop restoration tiles
      whose top left corners lie in this top-level superblock.
      
      The total file size should be unchanged by this patch: the bits have
      just been moved from the frame header and spread out among the rest of
      the frame.
      
      Change-Id: Icf43b0560964a63dea0d2cd801313f04139188d7
      6c545216
  3. 02 Sep, 2017 3 commits
  4. 01 Sep, 2017 8 commits
    • Ryan's avatar
      this update fixes the bug described in bug report 723 · a97c897b
      Ryan authored
      link is https://bugs.chromium.org/p/aomedia/issues/detail?id=723
      
      BUG=aomedia:723
      
      Change-Id: Iece3abcd88de69ab410674615965687abb5e4579
      a97c897b
    • Tom Finegan's avatar
      Fix CMake MSVC generation. · 00b4df73
      Tom Finegan authored
      Move the add_dependencies() call for aom_pc to within
      the if(MSVC) block, where it belongs.
      
      BUG=aomedia:722
      
      Change-Id: If25a6c83a981afc6d900e89e31db528af13b9e93
      00b4df73
    • Tom Finegan's avatar
      Fix test/hiprec_convolve_test.cc when SSE2 is unavailable. · d72ea329
      Tom Finegan authored
      Change-Id: I934a03bb5da1e266291ff66a056ff7306bd9e01f
      d72ea329
    • Jingning Han's avatar
      Fix zero block rate estimate in lv-map · e9814915
      Jingning Han authored
      Derive the rate cost from the probability model sets effectively
      in use.
      
      Change-Id: Ief1851a32ea7a3a52e82f52050cdc3dae6c72287
      e9814915
    • David Barker's avatar
      Miscellaneous fixes for var-tx · 16c64e33
      David Barker authored
      Lots of small bug fixes, mainly around the transform size coding:
      
      * The loop filter was accidentally using the non-subsampled
        block size for the V plane, due to comparing a plane index
        (0, 1, or 2) against PLANE_TYPE_UV (== 1)
      
      * We allowed an initial update of the transform partition probabilities
        even on frames where we know they will never be used
        (because tx_mode != TX_MODE_SELECT).
        Further, these probabilities would not be reverted at the end
        of the frame, leading to the probability delta persisting across frames.
      
        Change this to behave more like the non-var-tx transform size coding,
        where probability deltas are only coded for frames with
        tx_mode == TX_MODE_SELECT, and the deltas only apply for one frame.
      
      * Fix decoder for the case where the video as a whole isn't lossless,
        and we have tx_mode == TX_MODE_SELECT, but the current segment
        *is* lossless.
        Note that the encoder already does the right thing in this case.
      
      * Don't allow the transform splitting to recurse "below" 4x4.
        This is really just a refactor, but means we can increase the
        maximum depth when subdividing rectangular transforms if we
        want to, whereas the previous code would have needed special cases
        for 4x8 and 8x4 transforms.
      
      * Finally, when we hit the maximum splitting depth, don't update
        the counts as if we had coded a 'no split' symbol.
      
      Change-Id: Iaebdacc9de81d2e93d3c49241e719bbc02e32682
      16c64e33
    • Rupert Swarbrick's avatar
      Encode loop restoration coefficients per tile · 09b5b168
      Rupert Swarbrick authored
      This is a baby-step towards encoding the coefficients at the start of
      superblocks at the top-left of loop restoration tiles. Note that this
      patch causes us to reset "wiener_info" and "sgrproj_info" at each tile
      boundary, which will cause a performance drop.
      
      This is necessary because, in order for tiles to be processed in
      parallel, we cannot delta-encode coefficients across tile boundaries
      if the coefficients are signalled within tiles. We could probably do
      better than the current patch by, say, delta-encoding against previous
      frames.
      
      This patch also fixes up the costing in pickrst.c to match
      
      Change-Id: I5b8b91d63aaf49627cde40219c31c0ac776dfd38
      09b5b168
    • James Zern's avatar
      cmake: drop HAVE_EDSP · 43fb4f57
      James Zern authored
      it was removed in:
      12a27c01 Remove armv6 references from the build system.
      
      Change-Id: I485548367c5cd8b1a94e4f875ebf9badcfddf344
      43fb4f57
    • James Zern's avatar
      cmake: set default CMAKE_BUILD_TYPE to Release · db0cc5b2
      James Zern authored
      this matches configure
      
      Change-Id: I95f6f073b9dba1ae2c2b48b6c89a762277777dfe
      db0cc5b2
  5. 31 Aug, 2017 16 commits
  6. 30 Aug, 2017 6 commits
    • Yunqing Wang's avatar
      Refactor setup_ref_mv_list · d797ea9e
      Yunqing Wang authored
      This patch eliminates the is_inside() checking for each neighouring
      block in scan_row_mbmi() and scan_col_mbmi(). Instead, in
      setup_ref_mv_list(), find maximum above row_offset and left col_offset
      for current block, and use them to decide which above rows and left
      columns to search on. This patch doesn't change bitstream. No
      noticeable speedup is seen.
      
      Change-Id: Ic4ae74412605d86e9e675f86d23de3a69c04e8f3
      d797ea9e
    • Yushin Cho's avatar
      Delete daala-dist and cdef-dist experiments in configure · 6e439653
      Yushin Cho authored
      Since those two experiments have been merged into the dist-8x8 experiment and
      can be turned on by 'aomenc' ccommand line option,
      "--tune=[daala-dist|cdef-dist]".
      
      Change-Id: If21542a6868147538f5c3be6ad49368579db4b53
      6e439653
    • Yi Luo's avatar
      Highbd parallel_deblocking sse2 optimization · 6f5569f3
      Yi Luo authored
      - Decoder speed improves ~13.7% (baseline + parallel_deblocking).
      - Highbd loopfilter AVX2 version works when this experiment is
        disabled.
      
      Change-Id: I5d56b137a1d52236a4735656c370d57ef71ae043
      6f5569f3
    • Tom Finegan's avatar
      Avoid download of unnecessary test files in the CMake build. · 0af2732e
      Tom Finegan authored
      Follow the same rules as configure: obey CONFIG_DECODE_PERF_TESTS
      and CONFIG_ENCODE_PREF_TESTS when building the file lists.
      
      BUG=aomedia:699
      
      Change-Id: I915c528ce6e14e050da1f1810f6ed4c43d136231
      0af2732e
    • Tom Finegan's avatar
      Move rtcd target setup function to aom_optimization.cmake. · 515aca9f
      Tom Finegan authored
      Addresses a TODO. Also clean up some minor formatting nits.
      
      Change-Id: Iced790a214cfdd703eef21f2a9976f8a07517897
      515aca9f
    • Tom Finegan's avatar
      Keep version info up to date in the CMake build. · 872e5856
      Tom Finegan authored
      Add commands and targets for generating aom_version.h and
      aom.pc when necessary.
      
      BUG=aomedia:703
      
      Change-Id: Idc4d8203f950f73f151cbe32806d188bc96a0ee4
      872e5856