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: Paul (San) Jewell #15

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

Conversation

sona1111
Copy link

@sona1111 sona1111 commented Oct 9, 2021

Repo Link

Features Implemented:

-Core Rendering thing / requirements
-Antialiasing by camera ray jitter
-mesh loading from OBJ files (with some sub features like camera setup, bounding boxes, etc)
-Intel open source de-noiser integration

-This was a frustrating, but fairly fun project at times. I was greatly slowed down by mistakenly using debug mode instead of release mode... I wish I had more time to experiment with optimizing the mesh loading and camera manipulation. The intel de noiser was very satisfying to use. I also spent some time getting the vulkan repository to work, but after weighing the time i had available I wasn't confidant I would be able to complete the integration in time, so that was shelved. Overall pretty fun time, I suppose~

Used two late days

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.

2 participants