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.

Sign up for GitHub

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

Jump to bottom

[Function Request] avifenc supports svt and rav1e encoder options key-value #2108

Closed
sommio opened this issue Apr 13, 2024 · 2 comments · Fixed by #2126
Closed

[Function Request] avifenc supports svt and rav1e encoder options key-value #2108

sommio opened this issue Apr 13, 2024 · 2 comments · Fixed by #2126

Comments

@sommio
Copy link

sommio commented Apr 13, 2024

Right now avifenc only supports aom's encoder options, and I'm hoping that the faster encoding svt and rav1e will be able to use it as well. Thanks!
@vrabaud
Copy link
Collaborator

vrabaud commented Apr 15, 2024

Hi, indeed, it was not added. It seems easy for SVT through svt_av1_enc_set_parameter ( https://gitlab.com/AOMediaCodec/SVT-AV1/-/blob/master/Source/API/EbSvtAv1Enc.h?ref_type=heads#L973) and rav1e_config_parse_int for rav1e. I'll look into it.
BTW, we are always looking for benchmark data, where di you find those speed results? Thx!

@sommio
Copy link
Author

sommio commented Apr 16, 2024

benchmark: https://www.reddit.com/r/AV1/comments/i4xdv6/av1_encoders_comparison/

vrabaud added a commit to vrabaud/libavif that referenced this issue Apr 19, 2024
@vrabaud
Add option parsing for rav1e and SVT-AV1.
041eeed 
This fixes AOMediaCodec#2108
@vrabaud vrabaud mentioned this issue Apr 19, 2024
Merged
vrabaud added a commit that referenced this issue Apr 22, 2024
@vrabaud
Add option parsing for rav1e and SVT-AV1. (#2126)
ca071c4 
This fixes #2108
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Add option parsing for rav1e and SVT-AV1. vrabaud/libavif
2 participants
@vrabaud @sommio