Resolve "Image-Klasse"

Closes #31 (closed)

added 💻 Doing + 1 deleted label

added 1 commit

• 7598c8c2 - [#31 (closed)] Begin of Image class and ImageManager class, mostly taken from Buffer...

Compare with previous version

Ich habe die Image und ImageManager Klassen schonmal angefangen. Das meiste ist aus den Buffer und BufferManager Klassen übernommen.

Ich weiß nicht, was in dem create für die Images genau passieren soll, aufgrund der vielen Parameter.

Soll auch eine explizite Funktion reinkommen um "Standard"-Textur-Images zu erstellen?

added 1 commit

• 7dcc1f7b - [#31 (closed)] Added simple create function

Compare with previous version

added 1 commit

• 16d6d04a - [#31 (closed)] Added ImageLayoutSwitch function

Compare with previous version

added 1 commit

• 9b6bacff - [#31 (closed)] Changed searchMemoryType()-functions to avoid multiple declarations in...

Compare with previous version

Habe im letzten Commit die searchMemoryType(...)-Funktion in ImageManager und BufferManager ausgetauscht durch die aus der Vulkan Spec, wegen der Lizenzbedenken

added 1 commit

• a9686658 - [#31 (closed)] Added function to copy buffer to image

Compare with previous version

added 1 commit

• 0e4cee6a - [#31 (closed)] Set ImageManager in Core.cpp

Compare with previous version

Mapping von Images kannst du vermutlich weglassen (bin auch garnicht sicher, ob das ging...). Was wichtiger ist, wäre ein Staging-Buffer beim Image befüllen, damit wir nicht erst einen Buffer mittels Staging befüllen müssen, um ein Image zu befüllen.

Für den Staging-Buffer kannst du vermutlich ähnlich wie beim BufferManager eine init()-Funktion einfügen, welche den BufferManager übergeben bekommt. Dann kannst du einen Staging-Buffer erzeugen und dieser wird automatisch später wieder freigegeben.

added 1 commit

• eca73235 - [#31 (closed)] WIP: Begin of implementaion of fill method, currently not working

Compare with previous version

added 1 commit

• 45740df0 - [#31 (closed)] Added getters, marked todos, formatted code

Compare with previous version

added 118 commits

• 45740df0...fe9d5779 - 116 commits from branch develop
• 27eed21f - Merge branch 'develop' into 31-image-klasse
• d0cdcb2e - [#31 (closed)] Filled image with just one buffer because it was easier

Compare with previous version

Ich habe den einen Staging-Buffer erstmal rausgenommen und erzeuge jetzt einen pro Image. Wir brauchen eigentlich noch eine Funktion, die pro Image-Format die Pixelsize zurückgibt und die Channel-Anzahl. Am besten wir nehmen später ein eigenes Enum für das Format, statt alle Vulkan-Formate zu supporten... mal gucken.

Mir ist auch noch aufgefallen, dass wir auch ImageViews brauchen und wir müssen vom PhysicalDevice das passende/optimale Format für den Depth-Buffer abfragen. Wird wohl wieder ein produktiver Sonntag. ^^'

added 1 commit

• e167746b - [#31 (closed)] Texture loads into image, format should be supported, fingers crossed

Compare with previous version

@thollmann Bitte später bei den Materialien nicht nur die Anzahl der Channel mitgeben, sondern auch das Format der Daten. Die Daten waren von stb bereits in 4 Channel aligned, aber ich hatte angenommen es wäre nur RGB, da nur ch/ channels als 3 gegeben war.

Naja, Textur lädt schonmal. Es dürften nurnoch die ImageViews fehlen.

added 1 commit

• 024ca480 - [#31 (closed)] Added ImageView to images in ImageManager

Compare with previous version

added 1 commit

• a9db86d4 - [#31 (closed)] Allowed using depth formats for images

Compare with previous version

added 1 commit

• aebb02c4 - [#31 (closed)] Added depth buffer to pipeline if passes are configured correctly

Compare with previous version

mentioned in issue #51 (closed)

marked this merge request as ready

resolved all threads

merged

mentioned in commit 692cabd5