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

Saumya #14

Open
wants to merge 8 commits into
base: main
Choose a base branch
from
Open

Saumya #14

wants to merge 8 commits into from

Conversation

renauga
Copy link
Contributor

@renauga renauga commented Apr 30, 2021

This isn't complete yet. There are some bugs in the code for Chapter 8. Merge this for now. I will fix them as soon as I can.

- Make vec3 class
- Rename Chapter 1.cpp to main.cpp
- Rename Chapter 1.exe to main.exe
- Use vec3 class in main.cpp
- Created Chpater2.ppm

Renamed files for better VCS intuition. Also, changed a few variables to match the book, to prevent further confusion.
- Implemented chapter 3
- Added lodepng functionality

From now, each chapter will have the code in main file, a corresponding ppm file and a corresponding png file
- Implemented Chapter 4
-Implemented Chapter 5 with just one sphere
- Completed Chapter 5
- Implemented Chapter 6
- Implemented Chapter 7
- fix the issue with sphere.h
- add a Chapter 7 HD photo before gamma correction for fun
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant