Skip to content
Snippets Groups Projects

feat: enable sub-64px dimensions

Merged Gianni Rosato requested to merge computerbuster/SVT-AV1:min-size into master
All threads resolved!

Description

Issue

This is the seventh Merge Request from the SVT-AV1-PSY team aiming to integrate our features into mainline SVT-AV1 under the oversight of Hassene Tmar and Faouzi Kossentini. This MR enables encoding of source media with dimensions smaller than 64x64px.

@hassount mentioned that this limitation was imposed intentionally in the past due to crashes encountered with small dimensions. We had a user report something like this to SVT-AV1-PSY's bug tracker, and I've included the fix in this commit. I'm not sure if this fixes encoding with small dimensions in all cases, but I haven't personally encountered any crashes since this fix was incorporated.

Author(s)

@juliobbv

Performance impact

  • quality
  • memory
  • speed
  • 8 bit
  • 10 bit
  • N/A

Test set

  • obj-1-fast can be found here
  • other
  • N/A

Merge method

  • Allow the maintainer to squash and merge when PR is ready to create a 1-commit to the master branch. The maintainer will be able to fix typos / combine commit messages to create a more readable 1-commit message or use whatever is stated in the 'Description' section
  • I will clean up my commits and the maintainer shall use 'rebase and merge' to the master branch

Merge request reports

Merge request pipeline #1641804733 passed

Merge request pipeline passed for b44b13a0

Approved by
Test summary results are being parsed

Merged by Christopher DegawaChristopher Degawa 1 month ago (Jan 27, 2025 4:37am UTC)

Merge details

  • Changes merged into master with b44b13a0.
  • Did not delete the source branch.

Pipeline #1642502384 passed

Pipeline passed for b44b13a0 on master

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Gianni Rosato added 1 commit

    added 1 commit

    • eb99797b - fix: get_max_wavefronts return value logic

    Compare with previous version

  • Gianni Rosato added 1 commit

    added 1 commit

    • 47608a36 - chore: document restoration filtering limitation on small inputs in comment

    Compare with previous version

  • PhoenixWorth resolved all threads

    resolved all threads

  • PhoenixWorth approved this merge request

    approved this merge request

  • added 14 commits

    • 47608a36...988e930c - 11 commits from branch AOMediaCodec:master
    • 53dc6459 - feat: enable sub-64px dimensions
    • 5dbd8e52 - fix: get_max_wavefronts return value logic
    • 5d487ee3 - chore: document restoration filtering limitation on small inputs in comment

    Compare with previous version

  • added 18 commits

    • 5d487ee3...0598382a - 15 commits from branch AOMediaCodec:master
    • d59b68e9 - feat: enable sub-64px dimensions
    • c84966bd - fix: get_max_wavefronts return value logic
    • 49e51bd5 - chore: document restoration filtering limitation on small inputs in comment

    Compare with previous version

  • added 1 commit

    Compare with previous version

  • Christopher Degawa reset approvals from @PhoenixWorth by pushing to the branch

    reset approvals from @PhoenixWorth by pushing to the branch

  • Christopher Degawa approved this merge request

    approved this merge request

  • Please register or sign in to reply
    Loading