Skip to content

Rework Asset Loader API

Offene Aufgaben aus Ticket #63 (closed) müssen noch nachgearbeitet werden. Dabei handelt es sich hauptsächlich um das Anlegen und Befüllen des Sampler-Structs. Desweiteren muss der Asset Loader nach den Entscheidungen aus #74 (closed) überarbeitet werden.

  1. sampler struct erstellen
  2. Code aufräumen
    • Problematische Stellen im Code markieren und dokumentieren
    • große loadScene-Funktion in Unterfunktionen Zerlegen
    • markierte Stellen im Code nochmal überarbeiten
  3. API Rework:
    • public structs zu Klassen mit private sections + getter (setter brauchen wir nicht, oder?)
    • Einzelne loadScene-Funktion ergänzen entsprechend der Diskussion in #74 (closed): probeScene, loadScene und loadMesh
    • Zu loadMesh(): Es muss irgendwie ein Mesh innerhalb einer Szene identifiziert werden können (nach einem Aufruf von probeScene); was sagt der glTF-Standard zu der Eindeutigkeit von Namen der Meshes? Sind Namen optional?
  4. Kleinere Sachen
    • glTF-Datei finden, die ein image nicht als URI angibt sondern als mime-type+bufferView
    • loadTexture in unsere API integrieren und das extra-struct rausschmeißen
  5. Randfall beim vertex buffer stride: Wenn in der glTF ein stride von 0 steht (no interleaving), müssen wir selbst rechnen: https://www.khronos.org/opengl/wiki/Vertex_Specification#Vertex_buffer_offset_and_stride
  6. Randfall: Mehr als einen (vertex)buffer
  7. loadMesh() lädt derzeit alle Texturen, nicht nur die Texturen, die für das Mesh relevant sind
  8. probeScene() lädt derzeit nicht die Metadaten der Texturen und Sampler
  9. Manche Funktionen wie createTextures müssen angepasst werden um in verschiedenen Szenarios zu funktionieren (z.B. ob Texturdaten geladen und dekodiert werden sollen). Derzeit sind einige Funktionen nur auf den Fall von loadScene() ausgelegt, also direkt alles zu laden.
  10. Warnung in probeScene(), loadScene() etc ausgeben falls Mesh-Namen nicht eindeutig sind
  11. loadScene() umschreiben, sodass sie auf einen Aufruf von probeScene() und eine Schleife über loadMesh() reduziert wird.
  12. loadMesh() ändern: Statt einem Pfad kriegt es eine bereits geprobte Scene übergeben und muss die glTF-Datei nicht selbst laden; außerdem wird das Mesh über den index statt den Namen identifiziert.
  13. URIs im Scene struct speichern
    • URIs von Buffern und Texturen in Scene speichern und in Texture und VertexGroup darauf verweisen.
    • Sonderfall: Texturen müssen nicht zwingend eine URI haben sondern können stattdessen auf eine bufferView verweisen...
  14. Randfall testen: Texturen, die statt einer URI eine BufferView haben (ist bereits implementiert, aber ungetestet) Wird nicht getestet, ist entsprechend im API header dokumentiert.
    • In diesem Randfall ist das Laden und Decodieren der Daten derzeit aufgeteilt in zwei Schritte: loadTextureData() decodiert die Daten, die probeScene() vorher aus dem verlinkten Buffer kopiert hat... aber probeScene() sollte eigentlich keine Binärdaten laden und dieser Buffer noch nicht gefüllt sein zu dem Zeitpunkt. Nevermind

CC: @thollmann @tfrisch @larshoerttrich @susannedoetsch @mmints @mvogt

Edited by Trevor Hollmann