Shader Program Reflection

Auslesen von:

• VertexLayout
• DesccriptorSetLayouts
• OutputAttachments

@mmints @mvogt @shermann04 @lfranken @awasmut

changed milestone to %Texture Cube

added 🌈 Rendering 🍪 ToDo labels

mentioned in issue #33 (closed)

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

mentioned in merge request !28 (merged)

mentioned in commit 636f8131

removed 🍪 ToDo label

added 💻 Doing label

mentioned in commit c0d144a2

mentioned in commit aad45984

marked the checklist item VertexLayout as completed

mentioned in commit e6b51467

mentioned in commit 779aa495

mentioned in merge request !29 (merged)

mentioned in commit f05dcda6

closed with merge request !28 (merged)

reopened

removed milestone %Texture Cube

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

mentioned in merge request !49 (merged)

mentioned in commit 827a2edf

mentioned in commit 996a1ffb

Leonie: Wir reden gerade darüber, ob es vielleicht sinnvoll wäre die DescriptorSet Reflection direkt mit den Descriptoersets zu verbinden (DescriptorsetConfig.hpp) hat jemand von euch Zeit dazuzukommen und uns zu erklären, ob das sinnvoll ist oder nicht? Stand jetzt sind Set (ID) und Binding (ID) nicht im DescriptorBinding enthalten was ja (wenn ich das richtig verstanden habe) das ist, was wir reflecten möchten.

Ich selbst hätte das auf jeden Fall jetzt so gemacht, dass aus der ShaderReflection direkt die Erstellung der vkcv::DescriptorSets resultiert und diese dann im ShaderProgram als Member gespeichert werden. In dem Fall benötigt man von der Shader Reflection bei den Deskriptoren momentan eben die Daten, die auch ein vkcv::DescriptorBinding benötigt (type, count, shaderStage). Die SetID ist zudem wichtig dafür, welche vkcv::DescriptorBindings in ein vkcv::DescriptorSet zusammengepackt werden. BindingID und Name könnte man sonst halt auch noch reflecten. Das müsste man in der vkcv::DescriptorConfig dann halt noch ergänzen. BindingID war aber auch schon mal in der vkcv::DescriptorBinding drin und wurde dann wieder rausgenommen.

mentioned in commit 5a812555

mentioned in commit d4e5049d

mentioned in commit 30ac3c3f

mentioned in commit 8029ec84

mentioned in commit b9e760f8

marked the checklist item DesccriptorSetLayouts as completed

mentioned in commit 5027e837

mentioned in commit 887c6f2a

closed with merge request !49 (merged)