-
Notifications
You must be signed in to change notification settings - Fork 843
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
Updates from book 3 complete progression #1545
Conversation
src/TheRestOfYourLife/pdf.h
Outdated
//============================================================================================== | ||
// Originally written in 2016 by Peter Shirley <ptrshrl@gmail.com> | ||
// | ||
// To the extent possible under law, the author(s) have dedicated all copyright and related and | ||
// neighboring rights to this software to the public domain worldwide. This software is | ||
// distributed without any warranty. | ||
// | ||
// You should have received a copy (see file COPYING.txt) of the CC0 Public Domain Dedication | ||
// along with this software. If not, see <http://creativecommons.org/publicdomain/zero/1.0/>. | ||
//============================================================================================== |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed in next commit.
bool scatter( | ||
const ray& r_in, const hit_record& rec, color& attenuation, ray& scattered, double& pdf | ||
) const override { | ||
double pdf_value; // TODO: What are we supposed to do with the returned PDF? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is this TODO for the reader?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No, things are bad enough at this point that I wanted to make sure all instances of this problematic code are flagged, so we don't miss a copy before shipping. It's also why I've updated images with obviously bad renders.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Slowly getting better!
Also includes some minor fixes.
Note that this shows the current state of some buggy renders, which render the entire scene as black, except for the single square light source.
Resolves #988
Resolves #1317