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

Enable tiling with rav1e which allows multi threaded encoding! #32

Closed
wants to merge 2 commits into from

Conversation

@cryptomilk
Copy link
Contributor

cryptomilk commented Nov 26, 2019

Enable tiling with rav1e which allows multi threaded encoding.

Yay, more speed!

cryptomilk added 2 commits Nov 26, 2019
This ignored files of clangd, cscope and ctags.
This allows processing an image with multiple threads.
@joedrago

This comment has been minimized.

Copy link
Collaborator

joedrago commented Nov 26, 2019

libavif most certainly already supports tiling with rav1e already. What does this behavior do instead of the (adjacent settings) tileRowsLog2 / tileColsLog2?

@joedrago

This comment has been minimized.

Copy link
Collaborator

joedrago commented Nov 26, 2019

I read encoder.rs to be sure this is a redunant PR, and .tiles is an alternate path to the same settings, with less control. Please switch your usage of this new code to using tileRowsLog2 and tileColsLog2 instead.

@joedrago joedrago closed this Nov 26, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.