Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Make code compatible with SVT-AV1 3.0 #2607

Merged
merged 4 commits into from
Feb 13, 2025
Merged

Conversation

vrabaud
Copy link
Collaborator

@vrabaud vrabaud commented Feb 11, 2025

No description provided.

@vrabaud
Copy link
Collaborator Author

vrabaud commented Feb 11, 2025

@wantehchang , we can probably merge the C code without bumping the version for now. That will fix #2589 . If more API changes happen, we will clean them along the way

@y-guyon y-guyon added this to the v1.2.0 milestone Feb 11, 2025
@vrabaud
Copy link
Collaborator Author

vrabaud commented Feb 12, 2025

@wantehchang
Copy link
Collaborator

Yes, I consulted libavif_api_job_fix.patch when I reviewed your pull request.

@vrabaud
Copy link
Collaborator Author

vrabaud commented Feb 12, 2025

Merging the .c fix will break their CI because SVT_AV1_VERSION_MAJOR has not been bumped. Let's wait for the last minute before our 1.2 release.

@vrabaud vrabaud changed the title WIP: try SVT-AV1 pre 3.0 Make code compatible with SVT-AV1 3.0 Feb 12, 2025
Copy link
Collaborator

@wantehchang wantehchang left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

Copy link
Collaborator

@wantehchang wantehchang left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@hassount @1480c1 Please review. Thanks!

@vrabaud vrabaud merged commit e649830 into AOMediaCodec:main Feb 13, 2025
35 checks passed
@vrabaud vrabaud deleted the svt branch February 13, 2025 09:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants