getting skybox ready

4 files changed, 62 insertions, 17 deletions

diff --git a/src/core/camera.h b/src/core/camera.h
index bacbca9..9709677 100644
--- a/src/core/camera.h
+++ b/src/core/camera.h
@@ -29,7 +29,7 @@ PUB Mat4 Camera_ViewProj(Camera* c, f32 lens_height, f32 lens_width, Mat4* out_v
 PUB Mat4 Camera_View2D(Camera* c); // TODO: 2D cameras
 
 
-// TODO: Basic reusable camera controls
+// TODO: (HIGH) Basic reusable camera controls
 /*
 Right click + move = pan
 Left click = orbit camera
diff --git a/src/new_render/pbr.c b/src/new_render/pbr.c
index 6ac5613..f793002 100644
--- a/src/new_render/pbr.c
+++ b/src/new_render/pbr.c
@@ -59,11 +59,9 @@ GPU_Pipeline* PBR_PipelineCreate(GPU_Renderpass* rpass) {
 void PBR_Execute(PBR_Storage* storage, Camera camera, TextureHandle shadowmap_tex,
                  RenderEnt* entities, size_t entity_count) {
   // 1. set up our pipeline
-
   // 2. upload constant data (camera, lights)
-
   // 3. draw each entity
-  //  - upload material data
+  //  - upload material data -> in the future we will sort & batch by material
   //  - upload model transform
   //  - emit draw call
 
@@ -84,7 +82,8 @@ void PBR_Execute(PBR_Storage* storage, Camera camera, TextureHandle shadowmap_te
   Vec3 light_color = vec3(300.0, 300.0, 300.0);
   Binding_Lights
       lights_data = { .pointLights = {
-                          // FIXME: fill out soem default lights to use
+                          // FIXME: add lights to our RenderScene structure. for now these are
+                          // hardcoded
                           (pbr_point_light){ .pos = vec3(10, 10, 10), .color = light_color },
                           (pbr_point_light){ .pos = vec3(-10, 10, 10), .color = light_color },
                           (pbr_point_light){ .pos = vec3(10, -10, 10), .color = light_color },
diff --git a/src/new_render/skybox.c b/src/new_render/skybox.c
index 6afefc3..a0e151a 100644
--- a/src/new_render/skybox.c
+++ b/src/new_render/skybox.c
@@ -1,6 +1,9 @@
 #include "skybox.h"
 #include <assert.h>
+#include "file.h"
 #include "glad/glad.h"
+#include "log.h"
+#include "primitives.h"
 #include "ral_common.h"
 #include "ral_impl.h"
 #include "ral_types.h"
@@ -8,8 +11,13 @@
 #include "render_types.h"
 
 Skybox Skybox_Create(const char** face_paths, int n) {
+  INFO("Creating a skybox");
   assert(n == 6);  // ! we're only supporting a full cubemap for now
 
+  // -- cube verts
+  Geometry geom = Geo_CreateCuboid(f32x3(1.0, 1.0, 1.0));
+  Mesh cube = Mesh_Create(&geom, true);
+
   // -- cubemap texture
   TextureHandle handle;
   GPU_Texture* tex = GPU_TextureAlloc(&handle);
@@ -33,29 +41,66 @@ Skybox Skybox_Create(const char** face_paths, int n) {
 
   // shader pipeline
 
+  
+
+  ShaderData shader_data = { .data = NULL, .get_layout = &Skybox_GetLayout };
+
+  GPU_RenderpassDesc rpass_desc = {
+    .default_framebuffer = true,
+  };
+  GPU_Renderpass* pass = GPU_Renderpass_Create(rpass_desc);
+
+  arena scratch = arena_create(malloc(1024 * 1024), 1024 * 1024);
+
+  Str8 vert_path = str8("assets/shaders/skybox.vert");
+  Str8 frag_path = str8("assets/shaders/pbr_textured.frag");
+  str8_opt vertex_shader = str8_from_file(&scratch, vert_path);
+  str8_opt fragment_shader = str8_from_file(&scratch, frag_path);
+  if (!vertex_shader.has_value || !fragment_shader.has_value) {
+    ERROR_EXIT("Failed to load shaders from disk")
+  }
+
   VertexDescription pos_only = { .debug_label = "Position only verts" };
   VertexDesc_AddAttr(&pos_only, "inPos", ATTR_F32x3);
   pos_only.use_full_vertex_size = true;
 
-  ShaderData shader_data = { .data = NULL, .get_layout = &Skybox_GetLayout };
-
   GraphicsPipelineDesc pipeline_desc = {
     .debug_name = "Skybox pipeline",
     .vertex_desc = pos_only,
     .data_layouts = { shader_data },
     .data_layouts_count = 1,
     .vs = {
-
+      .debug_name = "Skybox Vertex Shader",
+      .filepath = vert_path,
+      .code = vertex_shader.contents
     },
     .fs = {
-
+      .debug_name = "Skybox Fragment Shader",
+      .filepath = frag_path,
+      .code = fragment_shader.contents
     },
     .wireframe = false,
     .depth_test = true,
   };
 
-  return (Skybox){
-    .texture = handle,
+  GPU_Pipeline* pipeline = GPU_GraphicsPipeline_Create(pipeline_desc, pass);
 
-  };
+  return (Skybox){ .cube = cube, .texture = handle, .pipeline = pipeline };
+}
+
+void Skybox_Draw(Skybox* skybox, Camera camera) {
+  GPU_CmdEncoder* enc = GPU_GetDefaultEncoder();
+  GPU_CmdEncoder_BeginRender(enc, skybox->pipeline->renderpass);
+  GPU_EncodeBindPipeline(enc, skybox->pipeline);
+  GPU_EncodeSetDefaults(enc);
+
+  // Shader data
+  SkyboxUniforms uniforms = { .in_position = camera.position, .cubemap = skybox->texture };
+  ShaderData skybox_data = { .data = &uniforms, .get_layout = Skybox_GetLayout };
+
+  GPU_EncodeSetVertexBuffer(enc, skybox->cube.vertex_buffer);
+  GPU_EncodeSetVertexBuffer(enc, skybox->cube.index_buffer);
+  GPU_EncodeDrawIndexed(enc, skybox->cube.geometry->indices->len);
+
+  GPU_CmdEncoder_EndRender(enc);
 }
\ No newline at end of file
diff --git a/src/new_render/skybox.h b/src/new_render/skybox.h
index ec06658..5540381 100644
--- a/src/new_render/skybox.h
+++ b/src/new_render/skybox.h
@@ -3,22 +3,23 @@
  */
 
 #pragma once
-#include "backend_opengl.h"
+#include "camera.h"
 #include "defines.h"
 #include "ral_types.h"
+#include "render_types.h"
 
 typedef struct Skybox {
-    BufferHandle vertex_buffer;
+    Mesh cube;
     TextureHandle texture;
     GPU_Pipeline* pipeline; // "shader"
 } Skybox;
 
 PUB Skybox Skybox_Create(const char** face_paths, int n); // should always pass n = 6 for now
 
-PUB void Skybox_Draw(Skybox* skybox);
+PUB void Skybox_Draw(Skybox* skybox, Camera camera);
 
 typedef struct SkyboxUniforms {
-    Vec3 in_pos;
+    Vec3 in_position;
     TextureHandle cubemap;
 } SkyboxUniforms;
 
@@ -40,7 +41,7 @@ static ShaderDataLayout Skybox_GetLayout(void* data) {
     };
 
     if (has_data) {
-        b1.data.bytes.data = &d->in_pos;
+        b1.data.bytes.data = &d->in_position;
         b2.data.texture.handle = d->cubemap;
     }
     return (ShaderDataLayout) {