diff --git a/rtchallenge/src/camera.rs b/rtchallenge/src/camera.rs index 1b2cc8f..d64770e 100644 --- a/rtchallenge/src/camera.rs +++ b/rtchallenge/src/camera.rs @@ -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));