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

Use rayon for the raytrace-parallel example #1349

Closed
alexcrichton opened this issue Mar 14, 2019 · 0 comments · Fixed by #1380
Closed

Use rayon for the raytrace-parallel example #1349

alexcrichton opened this issue Mar 14, 2019 · 0 comments · Fixed by #1380
Assignees
Labels
assigned This issue has been assigned to a contributor sprint

Comments

@alexcrichton
Copy link
Contributor

Currently the multithreading example we have with raytracing uses a home-grown solution for parallelization, but we should be showing off rayon!

I've got a work in progress branch for gaming this out, but there's a few notable points:

  • Supporting this depends on changes to Rayon, which we're still proving out on our end.
  • Currently the demo is crashing a lot due to lock contention for memory allocation on the main thread, and this is hoped to be patched in the upstream standard library
  • The demo was seemingly crashing for other reasons, but I figured it'd be easier to get to the bottom of it once the allocator issue was solved.

In any case wanted to make sure we had a tracking issue for this!

@alexcrichton alexcrichton self-assigned this Apr 5, 2019
@alexcrichton alexcrichton added the assigned This issue has been assigned to a contributor label Apr 5, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
assigned This issue has been assigned to a contributor sprint
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant