Project 6: Liam Dugan -- 82 Late Days #26
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Repo Link: https://github.com/liamdugan/Project6-Vulkan-Grass-Rendering
Completed Features:
Comments:
This was the single most difficult assignment I've ever had to do at Penn (100+ hours). There was practically zero guidance on how to set up the host code or how to bind descriptor sets from either the slides, the writeup, or the linked tutorials. The easy items are very well explained, the more difficult ones, not enough. The official vulkan tutorial is very good, but it's impossible to learn everything you need to know for this project on there. To make matters worse, the code for implementing this paper is linked within the paper (not to mention the other public students' repos on github are also public) so this complete lack of guidance for this project is very likely driving students to be academically dishonest. I very much enjoyed this assignment once I finished, but please, in the future make the writeup have: a section on explaining what descriptor sets are, a section on tessellation (with an actually working HelloTessellation sample), a section on the differences between GLSL and SPIR-V, a section on which atomics are usable in vulkan, and finally a section on how to allocate uniform buffers and how they are different from vertex buffers.