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 6: Peyman Norouzi #23

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

Conversation

pnorouzi
Copy link

@pnorouzi pnorouzi commented Nov 6, 2019

  • Repo Link
  • Features:
    • Forward
    • Forward Plus
    • Clustered
    • All of the rest required parts (Blinn-Phong shading,...)
  • I used the remaining of my late days to complete this project.
  • Feedback: I did not like this project. I feel like most of my time was wasted on learning on JS. I am probably never going to use JS so I think this project did not teach me anything. I would say the class was very good until project 4. The last two projects seemed pretty out of place. Besides the last two projects, the course was great overall. Thanks.

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