Skip to content

switch to Rayon for parallel code#5

Open
birkenfeld wants to merge 1 commit intoabour:masterfrom
birkenfeld:rayon
Open

switch to Rayon for parallel code#5
birkenfeld wants to merge 1 commit intoabour:masterfrom
birkenfeld:rayon

Conversation

@birkenfeld
Copy link
Copy Markdown

and a few misc other simplifications

@abour
Copy link
Copy Markdown
Owner

abour commented Nov 24, 2020

Thank you for your merge request birkenfeld.

I did a performance test on my configuration with your code on a 4096*4096 image. I have the exact same result than with the actual tokio implementation (49s). I don't have any performance improvement. Did I miss something ?

@birkenfeld
Copy link
Copy Markdown
Author

The differences I mentioned on Reddit were measured against the code checked out then. With current master I also have no seen no difference in timing. The remaining advantage is the greatly reduced amount of code.

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.

2 participants