Resizing-Event

Das Resizing des Fensters benötigt eine neuerstellte SwapChain.

• Eventanbindung (abhängig von #14 (closed) -> ist in dem Ticket schon angebunden worden)
• Swapchain recreation

laut Vulkan-Tutorial werden die folgenden Schritte benötigt:

createSwapChain();
createImageViews();
createRenderPass();
createGraphicsPipeline();
createFramebuffers();
createCommandBuffers();

CC: @kkraemer4 @vaka1997 @jfmorgenstern @gaida

added 🍪 ToDo 👍 Feature 🖥️ I/O labels

mentioned in issue #14 (closed)

marked this issue as related to #14 (closed)

changed the description

changed the description

assigned to @gaida

removed 🍪 ToDo label

added 💻 Doing label

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

mentioned in merge request !43 (merged)

marked the checklist item Eventanbindung (abhängig von #14 (closed) -> ist in dem Ticket schon angebunden worden) as completed

mentioned in commit df58bf24

Sachen die Recreated werden müssen:
-Swapchain
-Swapchain-Image-Views
-Images die abhängig von Fenstergröße sind (z.B. Depth Buffer)
-Renderpasses die entsprechende Images benutzen

Vorschlag: Resize callback für User (bereits verfügbar)
Image und Renderpass brauchen resize calls

core.resizeImage(imageHandle, width height);
core.resizePipeline(pipelineHandle, width height);

@awasmut @tfrisch Ihr seid ja beide die Leads der Gruppen die das machen und da wollte ich Fragen was ihr zu dem Vorschlag von @agauggel meint und ob ihr mir da helfen würdet.

Ich müsste mal nachgucken, aber ein Image lässt sich garnicht so leicht resizen. Das muss man voraussichtlich komplett deallokieren und neu erzeugen, da sonst Speicher fehlt.

Das kann ich einbauen... ich fände zwar irgendwie angenehmer, wenn man so Images wie den Depth Buffer nicht selbst resizen muss, auf der Anwendungsebene, aber erstmal sollte das okay sein.

Ein Resize-Event ist dann auf jeden Fall notwendig und am besten wird erst komplett die Swapchain aufgeräumt und neu erzeugt, bevor das aufgerufen wird. Damit keine Abhängigkeit darin irgendwie Probleme macht.

Ich habe nochmal in den Code geschaut und aktuell wird der Depth Buffer pro Frame neu erzeugt in der passenden Größe. ^^'

Das ist nicht effizient, aber löst euer Problem bereits. In folgender MR habe ich das mal angepasst, so dass der Depth Buffer nur bei Größenänderungen neu erzeugt wird. Können wir später trotzdem noch anders handhaben. ^^'

Das Resizen lässt jetzt auch nicht mehr abstürzen, denke es fehlt dann nur noch der Renderpass wenn ich nix vergessen habe.

Es ist tatsächlich die Pipeline betroffen, die letzten Endes präsentiert und den Viewport und Scissor dazu bestimmt. Die Swapchain images resizen müsste man eigentlich nicht manuell, da ja die presentation engine in Kontrolle dieser ist (weshalb man da auch zB keinen extra memory allokieren muss, etc).

Ich bin leider noch nicht dazu gekommen, die dynamic states einer pipeline auszuprobieren, aber das hier sollte auf jeden Fall von Interesse sein: VkDynamicState Genauer: VK_DYNAMIC_STATE_VIEWPORT und dann entsprechend vkCmdSetViewport

Wollen wir uns dafür einen Tag hinsetzen und das gemeinsam ausprobieren?

Wäre ich dafür, da ich aber heute und morgen keine Zeit habe schlag ich Freitag vor. Bei dir war es wegen der Arbeit glaub ich ähnlich. Ansonsten sag mir nen Tag (außer Freitag Abend) und ich versuch es möglich zu machen.

mentioned in commit a10fd28d

mentioned in commit 8f3645df

marked the checklist item Swapchain recreation as completed

mentioned in commit 55f9c51c

mentioned in commit 2406d39c

mentioned in commit 38098a4a

mentioned in commit fcf7cd40

mentioned in commit 89bc0fea

mentioned in commit 973d1da2

mentioned in commit c5ea5767

closed with merge request !43 (merged)