Skip to content

Commit 35c1539

Browse files
authored
Merge branch 'dev-minor' into overrides
2 parents d73a054 + af4b833 commit 35c1539

File tree

1 file changed

+25
-19
lines changed

1 file changed

+25
-19
lines changed

CHANGELOG.md

Lines changed: 25 additions & 19 deletions
Original file line numberDiff line numberDiff line change
@@ -6,10 +6,15 @@ Change Log -- Ray Tracing in One Weekend
66
### Common
77
- Bug: Found a bug in book 3 source `isotropic::scatter()` method. Commented out, using default
88
(as it was previously). (#669)
9-
- Removed: now that the code handles ray-surface intersection from either side, we no longer need
10-
the `flip_face` class, so we've deleted it from the text and from the code (#482, #270)
9+
- Delete: vestigial `vec3::write_color()` method (now in color.h)
10+
- Change: All images and figures renamed to follow more logical convention, using the following
11+
pattern: `{fig,img}-<book>.<sequence>-<title>.<filetype>` (#495)
12+
- Change: `main()` function gets organized into image, world, camera, and render chunks
1113
- Change: Added header guards to the text of all three books whenever a new header file was
12-
introduced
14+
introduced, consistent with source code (#645)
15+
- New: Added constructors that take `color` arguments in addition to the constructors
16+
taking `shared_ptr<texture>` arguments, simplifying calling code. Applies to `checker_texture`,
17+
`constant_medium`, `diffuse_light`, `lambertian`, and `isotropic` (#516, #644)
1318
- Change: Added `override` keywords throughout. This keyword marks a subclass method as one that
1419
is intended to override a superclass method. It makes the code a bit easier to understand, and
1520
ensures that your function is actually overriding the method you think it is. Which is good,
@@ -20,43 +25,44 @@ Change Log -- Ray Tracing in One Weekend
2025
### _In One Weekend_
2126
- Change: Updated all rendered images except for 1.13, 1.14 (#179, #547, #548, #549, #550, #551,
2227
#552, #553, #554, #555, #556, #557, #560, #561, #562, #563, #564, #565, #566)
23-
- Change: Standard render width changed to 400
28+
- Change: Standard working render width changed to 400 pixels
2429
- Change: Image 6 is now a before-and-after pair to illustrate antialiasing
2530
- Change: Listing 48: Refactored material and geometry declarations
2631
- Change: Listing 52: Refactored assignment of `etai_over_etat`
2732
- Change: Listing 56: Refactored material declarations
2833
- Change: Listing 61: Refactored material and geometry declarations
29-
- Fix: Rewrote refracted ray perpendicular and parallel components for correctness
3034
- Fix: Corrected various missed change highlights in code listings
3135
- Fix: Listing 7: Added missing `color.h`, `vec3.h` includes
3236
- Fix: Listing 18: Add missing `double t` member of struct `hit_record` (#428)
3337
- Fix: Listing 24: Add missing `color.h` include
3438
- Fix: Listing 30: Add missing `camera.h` include
3539
- Fix: Listing 42: Don't need to include `ray.h` when using `rtweekend.h`
3640
- Fix: Listing 48: Add missing `material.h` include
37-
- Fix: Listing 51: `refract()` function was missing `fabs()` on `sqrt()` argument (#559
41+
- Fix: Listing 51: `refract()` function was missing `fabs()` on `sqrt()` argument (#559)
3842
- Fix: Listing 61: Include updated `cam` declaration, show context w/highlighting
3943
- Fix: Listing 62: Highlight rename of `camera::get_ray()` parameters to s, t (#616)
4044
- Fix: Listing 63: Show reverted scene declarations
4145
- Fix: Listing 68: Show final scene render parameters with highlighting
46+
- Fix: Rewrote refracted ray perpendicular and parallel components for correctness (#526)
4247
- New: Listing 50: Show the updated material definitions
43-
- New: Add new isotropic constructor taking color argument (#644)
4448

4549
### _The Next Week_
46-
- Removed: Deleted the section covering the old `flip_face` class, renumbered images as this
47-
eliminated the rendering with missing Cornell box faces (#482, #661)
48-
- Change: Renamed and explicitly numbered book images and figures (#495)
49-
- Fix: Reduced code duplication in dielectric::scatter() function
50-
- New: Added alternative constructors that take color arguments in addition to the constructors
51-
that take `shared_ptr<texture>` arguments, simplifying calling code. This applies to the
52-
following classes: `checker_texture`, `constant_medium`, `diffuse_light`, and `lambertian`.
53-
(#516)
50+
- Delete: Deleted the section covering the old `flip_face` class, renumbered images as this
51+
eliminated the rendering with missing Cornell box faces (#270, #482, #661)
52+
- Delete: scenes 7 & 9 from the original (`cornell_balls` and `cornell_final`), as these were not
53+
covered in the book. Made the source and book consistent with each other. There are now a total
54+
of eight scenes for the second book (#653, #620)
55+
- Change: Updated most rendered images for book 2: 2.01-2.03, 2.07-2.13, 2.15-2.22.
56+
- Change: Listing 10: Separate out world & camera definitions in main (#646)
57+
- Fix: Reduced code duplication in `dielectric::scatter()` function
5458
- Fix: "Intance" typo in Chapter 8.1 to "Instance" (#629)
5559
- Fix: Listing 7: Show reverted viewing parameters from book 1 final scene
56-
- Change: Listing 10: Separate out world & camera definitions in main (#646)
57-
- New: Add new isotropic constructor taking color argument (#644)
58-
- Removed: scenes 7 & 9 from the original (`cornell_balls` and `cornell_final`). There are now a
59-
total of eight scenes for the second book (#653, #620)
60+
- Fix: Typo in listing caption for filename `moving-sphere.h`
61+
62+
### _The Rest of Your Life_
63+
- Change: use `vup` for camera, as in other two books
64+
- Fix: world and camera setup in `main()`, and include full body in book listing (#646)
65+
- New: `flip_face` moved to book 3, where it's needed for the light source (#661)
6066

6167

6268
----------------------------------------------------------------------------------------------------

0 commit comments

Comments
 (0)