wathiede/raytracers
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

camera: update docstring to reflect new findings

Browse Source
This commit is contained in:
Bill Thiede 2021-07-18 12:36:15 -07:00
parent 7e19d7e61b
commit 02ef8634c3
1 changed files with 4 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
rtchallenge/src/camera.rs
View File

@ -287,10 +287,10 @@ impl Camera {
}
/// This renderer use rayon to split each row into a seperate thread. It
/// seems to have really bad performance (only ~6x speedup over serial), and
/// the flame graph looks a mess. A strength over
/// `render_parallel_one_tread_per_core` is that it doesn't require `Camera`
/// and `World` to be cloneable.
/// seems to have more consistent performance than worker pool, equally fast
/// as WP at WP's fastest. The downside is the flame graph looks a mess. A
/// strength over `render_parallel_one_tread_per_core` is that it doesn't
/// require `Camera` and `World` to be cloneable.
fn render_parallel_rayon(&self, w: &World) -> Canvas {
let image_mu = Mutex::new(Canvas::new(self.hsize, self.vsize, BLACK));