Compute Pipeline

In Anlehnung an die Graphics Pipeline soll eine entsprechende Compute Pipeline entwickelt werden. In dieser Anlehnung wird auch eine Beispielanwendung programmiert die an einem einfachen Beispiel die Nutzung der Compute Pipeline bzw. auch von Compute Shadern zeigt.

• Compute Shader Specs
• Compute Pipeline Specs

changed milestone to %CV-Tag

added 🌈 Rendering 🎉 CV-Tag 👍 Feature labels

removed 🎉 CV-Tag label

added 🍪 ToDo label

added 💻 Doing label and removed 🍪 ToDo label

changed the description

created merge request !58 (merged) to address this issue

mentioned in merge request !58 (merged)

mentioned in commit 8331c9cf

mentioned in commit bfe7c164

mentioned in commit c0d3d3c4

mentioned in commit 2ebaeebb

mentioned in commit 52eb4752

Das Erstellen der Compute Pipeline ist soweit gesetzten. Eine Beispielanwendung würde beim Debuggen und ggf. erweitern helfen. Ein ich habe ein einfaches Beispiel gefunden, von dem man den Compute Shader klauen könnte.

mentioned in commit 6a93f420

mentioned in commit 3c77a6b5

Die ComputePipeline kann jetzt über den Core erstellt werden. Ich habe die Testanwendung first_triangle entsprechend angepasst. Zur Laufzeit wirft der Validation Layer jedoch einen Fehler:

[...]
 vkCreatePipelineLayout() call has push constants index 0 with size 0. Size must be greater than zero. The Vulkan spec states: size must be greater than 0
[...]

mit einem Hinweis auf die vkCreatePipelineLayout Specs bzgl. der VkPushConstantRange-size.

Ich bin fast davon ausgegangen, dass es an dieser Stelle zu einem Fehler kommt. Hätte mich überrascht wenn nicht. Die range-size ziehe ich über:

const size_t matrixPushConstantSize = shaderProgram.getPushConstantSize();
const vk::PushConstantRange pushConstantRange(vk::ShaderStageFlagBits::eAll, 0, matrixPushConstantSize);

Die ist dann wohl 0 für das computeShaderProgram.

mentioned in commit 6b163c82

added 🙏 Review label and removed 💻 Doing label

mentioned in commit fca7acf4

mentioned in issue #71 (closed)

mentioned in commit f1ba9662

mentioned in commit 639c44a0

mentioned in commit f6111135

closed with merge request !58 (merged)