Skip to content

Project 5 - Taylor Nelms #38

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

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

Conversation

taylornelms15
Copy link

  • Repo Link
  • Features:
    • Successfully filled in code until I got a scene rendered
    • No features past base functionality
  • Feedback:
    I feel like I learned nearly nothing from the process of doing this project, with the exception of (1) the concept of "metaballs" and how to think about/render them, and (2) the fact that ray tracing exists on DirectX. If you were to ask me to use DXR in any fashion going forward, I would start off now with no better footing than when this project began. The educational process of "we deleted 10% of the lines of code from a working project, fill them in based on hints" was frustrating at best, and cryptic and useless at worst. I would recommend either dropping the CPU-side code of the project entirely, or actually preparing students, via IN-CLASS instruction, on how it operates.

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