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 4: Carlos Lopez Garces #20

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

Conversation

carlos-lopez-garces
Copy link

@carlos-lopez-garces carlos-lopez-garces commented Oct 19, 2024

Repo Link

  • Naive renderer.
  • Forward+ renderer.
  • Clustered deferred renderer.
  • Extra credit: toon shading with edge detection for outlines (only for clustered deferred renderer).

I had to use late days to work on the README. The code changes that I made after the deadline were (1) performance instrumentation to use GPUQuerySet and be able to measure render/compute pass execution time and (2) a fix to how I obtain the z-depth of a cluster given the z index (I discovered the issue while inspecting the debug viz images that I included in the README).

Most of the commits I made after the deadline were direct uploads in the Github repo website because I got this error when trying to push from local: batch response: This repository is over its data quota. Account responsible for LFS bandwidth should purchase more data packs to restore access.

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