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: Yue Zhang #16

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

Conversation

YueZhang1027
Copy link

@YueZhang1027 YueZhang1027 commented Oct 9, 2023

Two late day used. One late day for readme figure.

  • Repo Link
    Core feature implemented.
  • Extra credits:
    • 2️⃣ Refraction (e.g. glass/water) with Frensel effects evaluation.
    • 2️⃣ Physically-based depth-of-field.
    • 2️⃣ Stochastic Sampled Antialiasing.
    • 2️⃣ + Direct lighting, MIS and full integrator with environment lighting.
    • 4️⃣ Subsurface scattering.
    • 2️⃣ OBJ mesh loading.
    • 6️⃣ Kd tree hierarchical spatial data structures.
    • 3️⃣ [Better random number sequences for Monte Carlo ray tracing]

CMakeList Change: I organize the code structure by seperate some features into single header. Also, I use some new files for other external functions, here is the change list:

  • src/material.h
  • src/light.h
  • src/sample.h
  • src/medium.h
  • src/hdrloader.h, src/hdrloader.cpp
  • src/tiny_obj_loader.h, src/tiny_obj_loader.cc

Feedback: I really focus on this project and I almost missed another course's work (lol). For improvement, I do think several visual output figure with some sample scene or baseline running speed data will help debugging and comparision at first. There can be more decoupling on the original code structure.
Anyway, it's a project of infinite possiblity. I think I will restart it again after this semester (though with late day, the time is still limited for me to produce ideal visual effect.)

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