Merge pull request #708 from estshorter/fix-issue-694

Fix: Book2: Listing 70 (#694)

Author: hollasch

CHANGELOG.md

@@ -15,6 +15,8 @@ Change Log -- Ray Tracing in One Weekend
   - Fix: Listing 15: In `bvh.h`, add missing `hittable_list.h` include (#690)
   - Fix: Listing 33, 34, 38: Change implicit casts to explicit ones (#692)
   - Fix: Listing 40: Change `perlin.h` in the caption to `texture.h` (#698)
+  - Fix: Listing 70: Add missing `bvh.h` (#694)
+  - Fix: Listing 70 and `main.cc`: Change a fuzz value of a metal sphere to 1.0 which is the maximum value (#694)
 
 
 ----------------------------------------------------------------------------------------------------

books/RayTracingTheNextWeek.html

@@ -3247,6 +3247,10 @@
 why we get caustics and subsurface for free. It’s a double-edged design decision.
 
     ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ C++
+    ...
+    #include "bvh.h"
+    ...
+
     hittable_list final_scene() {
         hittable_list boxes1;
         auto ground = make_shared<lambertian>(color(0.48, 0.83, 0.53));
@@ -3280,7 +3284,7 @@
 
         objects.add(make_shared<sphere>(point3(260, 150, 45), 50, make_shared<dielectric>(1.5)));
         objects.add(make_shared<sphere>(
-            point3(0, 150, 145), 50, make_shared<metal>(color(0.8, 0.8, 0.9), 10.0)
+            point3(0, 150, 145), 50, make_shared<metal>(color(0.8, 0.8, 0.9), 1.0)
         ));
 
         auto boundary = make_shared<sphere>(point3(360,150,145), 70, make_shared<dielectric>(1.5));

src/TheNextWeek/main.cc

@@ -236,7 +236,7 @@ hittable_list final_scene() {
 
     objects.add(make_shared<sphere>(point3(260, 150, 45), 50, make_shared<dielectric>(1.5)));
     objects.add(make_shared<sphere>(
-        point3(0, 150, 145), 50, make_shared<metal>(color(0.8, 0.8, 0.9), 10.0)
+        point3(0, 150, 145), 50, make_shared<metal>(color(0.8, 0.8, 0.9), 1.0)
     ));
 
     auto boundary = make_shared<sphere>(point3(360,150,145), 70, make_shared<dielectric>(1.5));