Changeset 914bb99 in opengl-game for vulkan-game.hpp

Timestamp:

Apr 9, 2021, 3:26:38 PM (3 years ago)

Author:

Dmitry Portnoy <dportnoy@…>

Branches:

feature/imgui-sdl

Children:

a00eb06

Parents:

5049354

Message:

In VulkanGame, specify each vertex explicitly for the model pipeline instead of using the same index multiple times, in order to support the current approach to normal calculation.

File:

• vulkan-game.hpp (modified) (2 diffs)

vulkan-game.hpp

@@ -212 +212 @@
 
 class VulkanGame {
+
    public:
 
@@ -558 +559 @@
 }
 
+// This function sets all the normals for a face to be parallel
+// This is good for models that should have distinct faces, but bad for models that should appear smooth
+// Maybe add an option to set all copies of a point to have the same normal and have the direction of
+// that normal be the weighted average of all the faces it is a part of, where the weight from each face
+// is its surface area.
+
+// TODO: Since the current approach to normal calculation basicaly makes indexed drawing useless, see if it's
+// feasible to automatically enable/disable indexed drawing based on which approach is used
 template<class VertexType>
 vector<VertexType> VulkanGame::addVertexNormals(vector<VertexType> vertices) {