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

Project 3: Henry Zhu #22

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

Conversation

Maknee
Copy link

@Maknee Maknee commented Oct 3, 2018

  • Repo Link

  • Features

  • Refraction with fresnel effects

  • Reflection

  • Anti-aliasing

  • Motion-blur

  • Model loading (using tinyObj) with toggleable bounding volume intersection culling

Feedback:

  • Project was enjoyable, but was hard to figure out where to start and how to continue. I think adding an example of implementing one or two of the very simple features such as anti-aliasing to pathtracer or walking through the source code would be very helpful.
  • I learned a lot from this project about not only path tracing, but graphics in general. I didn't do refraction, reflection, anti-aliasing or motion-blur or model loading before and it was very fun implementing them!

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