Resolve "Compute Pipeline"

Closes #65 (closed)

changed milestone to %CV-Tag

added 🌈 Rendering 👍 Feature 💻 Doing labels

added 3 commits

• 8331c9cf - [#65 (closed)][Add] Function for creating compute pipeline
• c0d3d3c4 - [#65 (closed)][Add] Compute Shader Module
• bfe7c164 - [#65 (closed)][Add] Compute Shader Stage Info Struct

Compare with previous version

added 1 commit

• 2ebaeebb - [#65 (closed)][Add] Pipeline Layout for Compute Shadetr

Compare with previous version

added 1 commit

• 52eb4752 - [#65 (closed)][Add] Create Compute Pipeline

Compare with previous version

added 1 commit

• 6a93f420 - [#65 (closed)][Add] Create Compute Pipeline to Core class

Compare with previous version

added 1 commit

• 3c77a6b5 - [#65 (closed)][Update] Extend first_triangle by a Compute Pipeline

Compare with previous version

added 1 commit

• 6b163c82 - [#65 (closed)]add some comments

Compare with previous version

added 1 commit

• fca7acf4 - [#65 (closed)] Fix startup crash

Compare with previous version

Dieser Branch kann an sich jetzt so wie es ist gemerged werden. Strukturänderungen nähme ich dann im Refactoring Issue vor. Eine Frage habe ich dieser Hinsicht aber noch:

Die GraphicsPipeline wird über einen Parameter PipelineConfig erstellt. Für die ComputePipeline macht das aber recht wenig Sinn weil eigentlich nur das ShaderProgramausschlaggebend für die Pipeline ist. Soll das aber dennoch angeglichen werden?

Die Frage wollte ich schon mal in den Raum werfen, die Diskussion würde ich dann in #71 (closed) führen.

marked this merge request as ready

Ich habe noch Validation Layer Fehler im first-triangle.

Oh shit, kk, dann hatte ich dich falsch verstanden. Dachte, dass du's gefixt hast. Ich habe den Merge Request wieder zurück gezogen und mache das dann erstmal fertig.

Nope, ich hab nur den Crash gefixed.

marked this merge request as draft

added 12 commits

• fca7acf4...887c6f2a - 10 commits from branch develop
• b63f9c68 - Merge branch 'develop' into 65-compute-pipeline
• f1ba9662 - [#65 (closed)] Set compute pipeline descriptor set layout and push constant ranges correctly

Compare with previous version

added 1 commit

• 639c44a0 - [#65 (closed)] Added dispatch recording commands and adjusted test compute shader

Compare with previous version

added 6 commits

• 639c44a0...326c8c04 - 5 commits from branch develop
• 820a1981 - Merge branch 'develop' into 65-compute-pipeline

Compare with previous version

marked this merge request as ready

approved this merge request

requested review from @tfrisch

approved this merge request