adding icons to repo

editor/include/editor.hpp

@@ -1,8 +1,15 @@
 #ifndef EDITOR_HPP
 #define EDITOR_HPP
 
+#include <filesystem>
 #include <open_engine.hpp>
 
+#include "open_engine/application.hpp"
+#include "open_engine/logging.hpp"
+#include "open_engine/renderer/texture.hpp"
+#include "open_engine/scene/entity.hpp"
+#include "open_engine/scene/scene_serializer.hpp"
+#include "panels/content_browser.hpp"
 #include "panels/scene_hierarchy.hpp"
 
 #include <glm/ext/matrix_transform.hpp>
@@ -19,6 +26,12 @@
 
 namespace OpenEngine {
 
+    enum class PlayState
+    {
+        Edit = 0,
+        Play = 1
+    };
+
     class CameraController : public NativeScriptableEntity
     {
         public:
@@ -71,8 +84,19 @@ namespace OpenEngine {
 
                 scene = CreateRef<Scene>();
 
+                auto command_line_args = Application::Get().GetCommandLineArgs();
+                if (command_line_args.count > 1) {
+                    auto scene_path = command_line_args[1];
+                    SceneSerializer serializer(scene);
+                    serializer.Deserialize(scene_path);
+                }
+
                 editor_camera = EditorCamera(30.0f, 1920.0f/1080.0f, 0.1f, 1000.0f);
 
+                icons["play"] = Texture2D::Create("resources/textures/icons/play.png");
+                icons["stop"] = Texture2D::Create("resources/textures/icons/stop.png");
+                // TODO: Add file texture. Get free icons and add license
+                //icons["folder"] = Texture2D::Create("resources/textures/icons/folder.png");
                 /*
                 for (float i = 0; i < 200; i++) {
                     for (float y = 0; y < 200; y++) {
@@ -90,6 +114,13 @@ namespace OpenEngine {
                 scene_hierarchy.Init(scene);
             }
 
+            Ref<Texture2D> GetIcon(const char* name)
+            {
+                if (icons.contains(name))
+                    return icons[name];
+                return nullptr;
+            }
+
             void OnDetach() override
             {
             }
@@ -114,9 +145,19 @@ namespace OpenEngine {
                 RenderCommand::Clear();
                 framebuffer->ClearBufferI(1, -1);
 
-                editor_camera.OnUpdate();
 
+                switch (state) {
+                    case PlayState::Play: {
+                        scene->OnUpdateRuntime();
+                        break;
+                    }
+                    case PlayState::Edit: {
+                        if (viewport_focused)
+                            editor_camera.OnUpdate();
                         scene->OnUpdateEditor(editor_camera);
+                        break;
+                    }
+                }
 
                 auto [mx, my] = ImGui::GetMousePos();
 
@@ -132,8 +173,10 @@ namespace OpenEngine {
                 // Mouse Picking
                 if (Input::IsMouseButtonPressed(MouseCode::ButtonLeft)
                         && mouse_x >= 0 && mouse_y >= 0
-                        && mouse_x < (int)viewport_size.x && (int)viewport_size.y &&
+                        && mouse_x < (int)viewport_size.x
-                        !ImGuizmo::IsOver() && !Input::IsKeyPressed(KeyCode::LeftAlt)) {
+                        && mouse_y < (int)viewport_size.y
+                        && !ImGuizmo::IsOver()
+                        && !Input::IsKeyPressed(KeyCode::LeftAlt)) {
                     if (!clicked) {
                         int id = framebuffer->ReadPixel(1, mouse_x, mouse_y);
 
@@ -143,7 +186,6 @@ namespace OpenEngine {
                         scene_hierarchy.SetSelectedEntity(selected_entity);
 
                     }
-
                     clicked = true;
                 } else {
                     clicked = false;
@@ -242,9 +284,9 @@ namespace OpenEngine {
                 ImGui::PushStyleVar(ImGuiStyleVar_WindowPadding, ImVec2{ 0, 0 });
 
                 ImGui::Begin("Viewport");
-                bool focused = ImGui::IsWindowFocused();
+                viewport_focused = ImGui::IsWindowFocused(ImGuiFocusedFlags_RootWindow);
-                bool hovered = ImGui::IsWindowHovered();
+                viewport_hovered = ImGui::IsWindowHovered(ImGuiHoveredFlags_RootWindow);
-                Application::Get().GetImGuiLayer()->SetBlockEvents((!focused && !hovered) || ImGui::IsAnyItemActive());
+                Application::Get().GetImGuiLayer()->SetBlockEvents((!viewport_focused && !viewport_hovered) || ImGui::IsAnyItemActive());
 
                 ImVec2 viewport_panel_size = ImGui::GetContentRegionAvail();
 
@@ -255,6 +297,15 @@ namespace OpenEngine {
                 uint32_t texture_id = framebuffer->GetColorAttachmentRendererID();
                 ImGui::Image((void*)(uint64_t)texture_id, ImVec2{ viewport_size.x, viewport_size.y }, ImVec2{ 0, 1 }, ImVec2{ 1, 0 });
 
+                if (ImGui::BeginDragDropTarget()) {
+                   if (const ImGuiPayload* payload = ImGui::AcceptDragDropPayload("CONTENT_BROWSER_PAYLOAD")) {
+                       std::filesystem::path path = (const char*)payload->Data;
+                       if (path.extension() == ".oes")
+                           OpenScene(path);
+                   }
+                    ImGui::EndDragDropTarget();
+                }
+
                 viewport_bounds[0] = { imgui_cursor_position.x, imgui_cursor_position.y };
                 viewport_bounds[1] = { imgui_cursor_position.x + viewport_size.x, 
                         imgui_cursor_position.y + viewport_size.y };
@@ -337,11 +388,62 @@ namespace OpenEngine {
                     guizmo_operation = -1;
             };
 
+            void OpenScene(const std::string& file)
+            {
+                if (!file.empty()) {
+                    scene = CreateRef<Scene>();
+                    scene->OnViewportResize((uint32_t)viewport_size.x, (uint32_t)viewport_size.y);
+                    scene_hierarchy.Init(scene);
+
+                    SceneSerializer serializer(scene);
+                    serializer.Deserialize(file);
+                }
+            }
+
+            void DrawPlayBar()
+            {
+                ImGui::PushStyleVar(ImGuiStyleVar_WindowPadding, { 0, 2 });
+                ImGui::PushStyleVar(ImGuiStyleVar_ItemInnerSpacing, { 0, 0 });
+                ImGui::PushStyleVar(ImGuiStyleVar_FramePadding, { 0, 0 });
+                ImGui::PushStyleColor(ImGuiCol_Button, { 0, 0, 0, 0 });
+                ImGui::PushStyleColor(ImGuiCol_ButtonHovered, { 0.200f, 0.207f, 0.286f, 1.0f });
+                ImGui::PushStyleColor(ImGuiCol_ButtonActive, { 0, 0, 0, 0 });
+
+                ImGui::Begin("##play_state_bar", nullptr, ImGuiWindowFlags_NoDecoration
+                        | ImGuiWindowFlags_NoScrollWithMouse
+                        | ImGuiWindowFlags_NoScrollbar
+                        | ImGuiWindowFlags_NoTitleBar);
+
+                float size = ImGui::GetContentRegionAvail().y;
+
+                auto icon = state == PlayState::Play ? icons["stop"] : icons["play"];
+
+                ImGui::SetCursorPosX((ImGui::GetContentRegionAvail().x * 0.5) - (size * 0.5) );
+                if (ImGui::ImageButton("##play_button",
+                        (ImTextureID)icon->GetID(), { size, size })) {
+                    switch (state) {
+                        case PlayState::Play : {
+                            OnSceneStop();
+                            break;
+                        }
+                        case PlayState::Edit : {
+                            OnScenePlay();
+                            break;
+                        }
+                    }
+                }
+
+                ImGui::End();
+
+                ImGui::PopStyleColor(3);
+                ImGui::PopStyleVar(3);
+            }
+
             void OnImGuiRender() override
             {
                 OE_PROFILE_FUNCTION();
 
-                ImGui::PushStyleVar(ImGuiStyleVar_WindowMinSize, ImVec2(450, 200));
+                ImGui::PushStyleVar(ImGuiStyleVar_WindowMinSize, ImVec2(450, 35));
                 {
                     if (ImGui::BeginMainMenuBar()) {
                         if (ImGui::BeginMenu("File")) {
@@ -355,15 +457,8 @@ namespace OpenEngine {
                             }
                             if (ImGui::MenuItem("Open Scene", "Ctrl+O")) {
                                 std::string file = FileDialogs::OpenFile("useless");
-                                OE_DEBUG("loading filename: {}", file);
+                                OE_DEBUG("loading scene: {}", file);
-                                if (!file.empty()) {
+                                OpenScene(file);
-                                    scene = CreateRef<Scene>();
-                                    scene->OnViewportResize((uint32_t)viewport_size.x, (uint32_t)viewport_size.y);
-                                    scene_hierarchy.Init(scene);
-
-                                    SceneSerializer serializer(scene);
-                                    serializer.Deserialize(file);
-                                }
                             }
                             if (ImGui::MenuItem("New Scene", "Ctrl+N")) {
                                 scene = CreateRef<Scene>();
@@ -383,13 +478,25 @@ namespace OpenEngine {
 
                 DrawStats();
                 scene_hierarchy.OnImGuiRender();
+                browser.OnImGuiRender();
                 DrawViewport();
+                DrawPlayBar();
 
                 ImGui::ShowDemoWindow();
                 ImGui::PopStyleVar();
             };
 
+            void OnScenePlay()
+            {
+                state = PlayState::Play;
+            }
+            void OnSceneStop()
+            {
+                state = PlayState::Edit;
+            }
+
         private:
+            ContentBrowserPanel browser;
             std::vector<Time::ProfilingResult> profiling_results;
 
             glm::vec2 world_pos_cursor = { 0.0f, 0.0f };
@@ -398,21 +505,29 @@ namespace OpenEngine {
             Ref<OpenEngine::FrameBuffer> framebuffer;
             glm::vec2 viewport_bounds[2];
 
+            bool viewport_focused = false, viewport_hovered = false;
+
             SceneHierarchy scene_hierarchy;
 
             EditorCamera editor_camera;
 
             int guizmo_operation = -1;
 
+            // TODO: Rename this to GameState
+            PlayState state = PlayState::Edit;
+
+            std::unordered_map<const char*, Ref<Texture2D>> icons;
             Ref<Scene> scene;
             Entity selected_entity;
+
+            std::vector<Entity> entities;
     };
 }
 
 class EditorApp : public OpenEngine::Application
 {
     public:
-        EditorApp();
+        EditorApp(OpenEngine::ApplicationCommandLineArgs args);
         ~EditorApp();
 };
 

editor/include/panels/content_browser.hpp

@@ -0,0 +1,23 @@
+#ifndef CONTENT_BROWSER_HPP
+#define CONTENT_BROWSER_HPP
+
+#include "open_engine/renderer/texture.hpp"
+#include <filesystem>
+
+namespace OpenEngine {
+
+    class ContentBrowserPanel
+    {
+        public:
+            ContentBrowserPanel();
+
+            void OnImGuiRender();
+
+        private:
+            std::filesystem::path current_directory;
+            Ref<Texture2D> folder_icon;
+    };
+
+}
+
+#endif // CONTENT_BROWSER_HPP

editor/src/editor.cpp

@@ -1,3 +1,4 @@
+#include "open_engine/application.hpp"
 #include <open_engine/entry_point.hpp>
 
 #include <open_engine.hpp>
@@ -9,8 +10,8 @@
 #include <overlay.hpp>
 #include <modding.hpp>
 
-EditorApp::EditorApp()
+EditorApp::EditorApp(OpenEngine::ApplicationCommandLineArgs args)
-    : Application("OpenEngine Editor")
+    : Application("OpenEngine Editor", args)
 {
     OpenEngine::Ref<OpenEngine::Layer> modding_layer = OpenEngine::CreateRef<Modding>();
     OpenEngine::Ref<OpenEngine::Layer> editor_layer = OpenEngine::CreateRef<OpenEngine::EditorLayer>();
@@ -25,8 +26,8 @@ EditorApp::~EditorApp()
 {
 }
 
-OpenEngine::Application* OpenEngine::CreateApplication()
+OpenEngine::Application* OpenEngine::CreateApplication(OpenEngine::ApplicationCommandLineArgs args)
 {
     OE_INFO("Editor Starting...");
-    return new EditorApp();
+    return new EditorApp(args);
 }

editor/src/editor_component.cpp

@@ -1,3 +1,4 @@
+#include "imgui.h"
 #include <editor_component.hpp>
 #include <open_engine.hpp>
 
@@ -106,6 +107,16 @@ namespace OpenEngine {
     {
         auto& sprite = registry.get<SpriteRendererComponent>(entity);
         ImGui::ColorEdit4("color", glm::value_ptr(sprite.color));
+        ImGui::Button("Texture", { 100, 0 });
+
+        if (ImGui::BeginDragDropTarget()) {
+            if (const ImGuiPayload* payload = ImGui::AcceptDragDropPayload("CONTENT_BROWSER_PAYLOAD"))
+                sprite.texture = Texture2D::Create(((const char*)payload->Data));
+
+            ImGui::EndDragDropTarget();
+        }
+
+        ImGui::DragFloat("Tiling factor", &sprite.tiling_factor);
     };
 
     void CameraOnImGuiRender(entt::registry& registry, entt::entity entity) 

editor/src/panels/content_browser.cpp

@@ -0,0 +1,88 @@
+#include "imgui_internal.h"
+#include <cstdint>
+#include <cstring>
+#include <panels/content_browser.hpp>
+
+#include <open_engine/renderer/texture.hpp>
+#include <open_engine/logging.hpp>
+#include <filesystem>
+#include <imgui.h>
+
+namespace OpenEngine {
+
+    const std::filesystem::path assets_directory = "assets";
+
+    ContentBrowserPanel::ContentBrowserPanel()
+        : current_directory(assets_directory)
+    {
+        // TODO: Add file texture. Get free icons and add license
+        folder_icon = Texture2D::Create("resources/textures/icons/folder.png");
+    }
+
+    void ContentBrowserPanel::OnImGuiRender()
+    {
+        ImGui::Begin("Assets");
+
+        ImGui::Text("%s", current_directory.c_str());
+
+        if (current_directory != assets_directory)
+            if (ImGui::Button(".."))
+                current_directory = current_directory.parent_path();
+
+        auto directory_it = std::filesystem::directory_iterator(current_directory);
+
+        ImVec2 button_size = { 100, 100 };
+        auto panel_width = ImGui::GetContentRegionAvail().x;
+        uint32_t table_columns = (int)(panel_width / button_size.x) - 1;
+
+        if (table_columns <= 0)
+            table_columns = 1;
+
+        ImVec2 padding( 9, 9 );
+
+        ImGui::PushStyleVar(ImGuiStyleVar_CellPadding, padding);
+
+        if (ImGui::BeginTable("table1", table_columns, ImGuiTableFlags_SizingFixedSame))
+        {
+            for (auto& entry : directory_it) {
+                auto file_name = entry.path().filename().string();
+
+                ImGui::PushID(file_name.c_str());
+
+                ImGui::TableNextColumn();
+
+                ImGui::PushStyleColor(ImGuiCol_Button, { 0.0f, 0.0f, 0.0f, 0.0f });
+                ImGui::PushStyleColor(ImGuiCol_ButtonHovered, { 0.192f, 0.196f, 0.266f, 1.0f });
+                ImGui::PushStyleColor(ImGuiCol_ButtonActive, { 0.270f, 0.278f, 0.352f, 1.0f });
+
+                ImGui::BeginGroup();
+                ImGui::ImageButton("##X", (ImTextureID)folder_icon->GetID(), button_size, { 0, 1 }, { 1, 0 });
+                if (entry.is_regular_file() && ImGui::BeginDragDropSource()) {
+                    const char* source = entry.path().c_str();
+                    ImGui::SetDragDropPayload("CONTENT_BROWSER_PAYLOAD", source, (strlen(source) + 1) * sizeof(char), ImGuiCond_Once);
+
+                    ImGui::EndDragDropSource();
+                }
+
+                float columnWidth = ImGui::GetColumnWidth();
+                float textWidth = ImGui::CalcTextSize(file_name.c_str()).x;
+                ImGui::SetCursorPosX(ImGui::GetCursorPosX() + (columnWidth - textWidth) * 0.5f);
+                ImGui::TextWrapped("%s", file_name.c_str());
+
+                ImGui::EndGroup();
+
+                if (entry.is_directory()) {
+                    if (ImGui::IsItemHovered() && ImGui::IsMouseDoubleClicked(ImGuiMouseButton_Left))
+                        current_directory = entry.path().c_str();
+                }
+
+                ImGui::PopStyleColor(3);
+                ImGui::PopID();
+            }
+            ImGui::EndTable();
+        }
+        ImGui::PopStyleVar();
+
+        ImGui::End();
+    }
+}

imgui.ini

@@ -10,38 +10,39 @@ Collapsed=0
 
 [Window][Statistics]
 Pos=0,24
-Size=409,300
+Size=224,437
 Collapsed=0
 DockId=0x00000003,0
 
 [Window][Properties]
 Pos=822,24
-Size=450,739
+Size=450,805
 Collapsed=0
 DockId=0x00000007,0
 
 [Window][Viewport]
-Pos=411,24
+Pos=226,61
-Size=409,1363
+Size=594,956
 Collapsed=0
-DockId=0x00000009,0
+DockId=0x00000012,0
 
 [Window][Dear ImGui Demo]
-Pos=822,765
+Pos=822,831
-Size=450,622
+Size=450,556
 Collapsed=0
 DockId=0x00000008,0
 
 [Window][Scene]
-Pos=0,326
+Pos=0,463
-Size=409,1061
+Size=224,924
 Collapsed=0
 DockId=0x00000004,0
 
 [Window][Dear ImGui Style Editor]
-Pos=60,60
+Pos=1308,24
-Size=591,1380
+Size=800,1001
 Collapsed=0
+DockId=0x0000000E,0
 
 [Window][Example: Console]
 Pos=345,786
@@ -133,24 +134,50 @@ Collapsed=0
 Pos=343,24
 Size=1665,1371
 Collapsed=0
-DockId=0x00000009,1
+DockId=0x00000012,1
 
 [Window][Graph Editor]
 Pos=343,24
 Size=1665,1371
 Collapsed=0
-DockId=0x00000009,1
+DockId=0x00000012,1
+
+[Window][Assets]
+Pos=226,1019
+Size=594,368
+Collapsed=0
+DockId=0x0000000C,0
+
+[Window][##play]
+Pos=226,24
+Size=594,60
+Collapsed=0
+DockId=0x0000000F,0
+
+[Window][##play_state_bar]
+Pos=226,24
+Size=594,35
+Collapsed=0
+DockId=0x00000011,0
 
 [Docking][Data]
 DockSpace               ID=0x08BD597D Window=0x1BBC0F80 Pos=0,24 Size=1272,1363 Split=X
   DockNode              ID=0x00000005 Parent=0x08BD597D SizeRef=820,1386 Split=X
-    DockNode    ID=0x00000001 Parent=0x00000005 SizeRef=409,1386 Split=Y Selected=0xE601B12F
+    DockNode            ID=0x00000001 Parent=0x00000005 SizeRef=224,1386 Split=Y Selected=0xE601B12F
-      DockNode  ID=0x00000003 Parent=0x00000001 SizeRef=255,300 Selected=0x553E127E
+      DockNode          ID=0x00000003 Parent=0x00000001 SizeRef=255,417 Selected=0x553E127E
-      DockNode  ID=0x00000004 Parent=0x00000001 SizeRef=255,1061 Selected=0xE601B12F
+      DockNode          ID=0x00000004 Parent=0x00000001 SizeRef=255,883 Selected=0xE601B12F
-    DockNode    ID=0x00000002 Parent=0x00000005 SizeRef=1697,1386 Split=Y Selected=0xC450F867
+    DockNode            ID=0x00000002 Parent=0x00000005 SizeRef=594,1386 Split=Y Selected=0xC450F867
-      DockNode  ID=0x00000009 Parent=0x00000002 SizeRef=483,784 CentralNode=1 Selected=0xC450F867
+      DockNode          ID=0x00000009 Parent=0x00000002 SizeRef=483,784 Split=Y Selected=0xC450F867
+        DockNode        ID=0x0000000B Parent=0x00000009 SizeRef=409,932 Split=X Selected=0xC450F867
+          DockNode      ID=0x0000000D Parent=0x0000000B SizeRef=1080,993 Split=Y Selected=0xC450F867
+            DockNode    ID=0x0000000F Parent=0x0000000D SizeRef=594,60 Selected=0x8325EBDA
+            DockNode    ID=0x00000010 Parent=0x0000000D SizeRef=594,931 Split=Y Selected=0xC450F867
+              DockNode  ID=0x00000011 Parent=0x00000010 SizeRef=594,35 HiddenTabBar=1 Selected=0xAB37695D
+              DockNode  ID=0x00000012 Parent=0x00000010 SizeRef=594,956 CentralNode=1 Selected=0xC450F867
+          DockNode      ID=0x0000000E Parent=0x0000000B SizeRef=800,993 Selected=0x3EEA4247
+        DockNode        ID=0x0000000C Parent=0x00000009 SizeRef=409,368 Selected=0x42C24103
       DockNode          ID=0x0000000A Parent=0x00000002 SizeRef=483,600 Selected=0x1BCA3180
   DockNode              ID=0x00000006 Parent=0x08BD597D SizeRef=450,1386 Split=Y Selected=0x8C72BEA8
-    DockNode    ID=0x00000007 Parent=0x00000006 SizeRef=444,738 Selected=0x8C72BEA8
+    DockNode            ID=0x00000007 Parent=0x00000006 SizeRef=444,769 Selected=0x8C72BEA8
-    DockNode    ID=0x00000008 Parent=0x00000006 SizeRef=444,622 Selected=0x5E5F7166
+    DockNode            ID=0x00000008 Parent=0x00000006 SizeRef=444,531 Selected=0x5E5F7166
 

open_engine/include/open_engine/application.hpp

@@ -10,10 +10,25 @@
 int main(int argc, char **argv);
 
 namespace OpenEngine {
+
+    struct ApplicationCommandLineArgs
+    {
+        int count = 0;
+        char** args = nullptr;
+
+        const char* operator[](int index)
+        {
+            OE_CORE_ASSERT(index < count, "Index is higher than argument count.");
+
+            return args[index];
+        };
+    };
+
     class Application
     {
         public:
-            Application(const std::string& name = "OpenEngine Project");
+            Application(const std::string& name,
+                    ApplicationCommandLineArgs& args);
             ~Application();
 
             virtual void OnEvent(Event& event);
@@ -25,6 +40,7 @@ namespace OpenEngine {
             ImGuiLayer* GetImGuiLayer() { return imgui_layer.get(); };
 
             inline static Application& Get() { return *instance; };
+            ApplicationCommandLineArgs GetCommandLineArgs() const { return arguments; };
 
             inline Window& GetWindow() { return *window; };
 
@@ -36,6 +52,7 @@ namespace OpenEngine {
             bool OnWindowResize(WindowResizeEvent& event);
 
         private:
+            ApplicationCommandLineArgs arguments;
             const std::string name; 
             std::unique_ptr<Window> window;
 
@@ -49,7 +66,7 @@ namespace OpenEngine {
     };
 
     // Is defined by client
-    Application* CreateApplication();
+    Application* CreateApplication(ApplicationCommandLineArgs args);
 }
 
 #endif // APPLICATION_HPP

open_engine/include/open_engine/entry_point.hpp

@@ -1,17 +1,16 @@
 #ifndef ENTRY_POINT_HPP
 #define ENTRY_POINT_HPP
 
-#include "open_engine/core.hpp"
 #include "open_engine/application.hpp"
 #include "open_engine/logging.hpp"
 
-extern OpenEngine::Application* OpenEngine::CreateApplication();
+extern OpenEngine::Application* OpenEngine::CreateApplication(OpenEngine::ApplicationCommandLineArgs args);
 
 int main(int argc, char** argv)
 {
     OE_PROFILE_BEGIN_SESSION("Startup", "open_engine-startup.json");
     OpenEngine::Logger::Init();
-    auto app = OpenEngine::CreateApplication();
+    auto app = OpenEngine::CreateApplication({ argc, argv });
     OE_PROFILE_END_SESSION();
 
     OE_PROFILE_BEGIN_SESSION("Runtime", "open_engine-runtime.json");

open_engine/include/open_engine/opengl/opengl_shader.hpp

@@ -14,7 +14,7 @@ namespace OpenEngine {
         public:
             OpenGLShader(const std::string& shader_path);
             OpenGLShader(const std::string& name, const std::string& vertex_src, const std::string& frament_src);
-            virtual ~OpenGLShader() = default;
+            virtual ~OpenGLShader();
 
             virtual const std::string& GetName() const override { return name; };
 
@@ -44,11 +44,21 @@ namespace OpenEngine {
         private:
             std::string ReadFile(const std::string& shader_path);
             std::unordered_map<GLenum, std::string> PreProcess(const std::string& shader_source);
-            void Compile(const std::unordered_map<GLenum, std::string> sources);
             
-            void CheckCompileErrors(unsigned int shader, const std::string& type);
+            void CompileOrGetVulkanBinaries(const std::unordered_map<GLenum, std::string>& shader_sources);
+            void CompileOrGetOpenGLBinaries();
+            void CreateProgram();
+            void Reflect(GLenum stage, const std::vector<uint32_t>& shader_data);
+            //void CheckCompileErrors(unsigned int shader, const std::string& type);
 
+        private:
             std::string name;
+            std::string file_path;
+
+            std::unordered_map<GLenum, std::vector<uint32_t>> vulkan_spirv;
+            std::unordered_map<GLenum, std::vector<uint32_t>> opengl_spirv;
+
+            std::unordered_map<GLenum, std::string> m_OpenGLSourceCode;
 
             u_int32_t id;
     };

open_engine/include/open_engine/opengl/opengl_uniform_buffer.hpp

@@ -0,0 +1,24 @@
+#ifndef OPENGL_UNIFORM_BUFFER_HPP
+#define OPENGL_UNIFORM_BUFFER_HPP
+
+#include "open_engine/renderer/uniform_buffer.hpp"
+#include <cstdint>
+
+namespace OpenEngine {
+
+    class OpenGLUniformBuffer : public UniformBuffer
+    {
+        public:
+            OpenGLUniformBuffer(uint32_t size, uint32_t binding);
+            virtual ~OpenGLUniformBuffer();
+
+            virtual void SetData(const void* data, uint32_t size,
+                    uint32_t offset = 0) override;
+
+        private:
+            uint32_t id = 0;
+    };
+
+}
+
+#endif // OPENGL_UNIFORM_BUFFER_HPP

open_engine/include/open_engine/renderer/renderer2d.hpp

@@ -44,6 +44,9 @@ namespace OpenEngine {
             static void DrawQuad(const Transform& transform_data,
                     const Ref<Texture2D>& texture, int entity_id,
                     float tiling_factor = 1.0f);
+            static void DrawQuad(const Transform& transform_data,
+                    const Ref<Texture2D>& texture, const glm::vec4& color,
+                    int entity_id, float tiling_factor = 1.0f);
             static void DrawQuad(const Transform& transform_data,
                     const Ref<Subtexture2D>& subtexture, int entity_id,
                     float tiling_factor = 1.0f);
@@ -53,6 +56,9 @@ namespace OpenEngine {
             static void DrawQuad(const glm::mat4& transform,
                     const Ref<Texture2D>& texture, int entity_id,
                     float tiling_factor = 1.0f);
+            static void DrawQuad(const glm::mat4& transform,
+                    const Ref<Texture2D>& texture, const glm::vec4& color,
+                    int entity_id, float tiling_factor = 1.0f);
             static void DrawQuad(const glm::mat4& transform,
                     const Ref<Subtexture2D>& subtexture, int entity_id,
                     float tiling_factor = 1.0f);

open_engine/include/open_engine/renderer/uniform_buffer.cpp

@@ -0,0 +1,21 @@
+#include <pch.hpp>
+
+#include <renderer/uniform_buffer.hpp>
+#include <opengl/opengl_uniform_buffer.hpp>
+#include <renderer/renderer.hpp>
+
+namespace OpenEngine {
+
+    Ref<UniformBuffer> UniformBuffer::Create(uint32_t size, uint32_t binding)
+    {
+        switch (Renderer::GetAPI())
+        {
+            case RendererAPI::API::None:    OE_CORE_ASSERT(false, "RendererAPI::None is currently not supported!"); return nullptr;
+            case RendererAPI::API::OpenGL:  return CreateRef<OpenGLUniformBuffer>(size, binding);
+        }
+
+        OE_CORE_ASSERT(false, "Unknown RendererAPI!");
+        return nullptr;
+    }
+
+}

open_engine/include/open_engine/renderer/uniform_buffer.hpp

@@ -0,0 +1,20 @@
+#ifndef UNIFORM_BUFFER_HPP
+#define UNIFORM_BUFFER_HPP
+
+#include <cstdint>
+#include <glm/fwd.hpp>
+namespace OpenEngine {
+    
+    class UniformBuffer
+    {
+        public:
+            virtual ~UniformBuffer() {}
+            virtual void SetData(const void* data, uint32_t size,
+                    uint32_t offset = 0) = 0;
+
+            static Ref<UniformBuffer> Create(uint32_t size, uint32_t binding);
+        
+    };
+}
+
+#endif // UNIFORM_BUFFER_HPP

open_engine/include/open_engine/scene/components.hpp

@@ -3,6 +3,7 @@
 
 #include "open_engine/scene/native_scriptable_entity.hpp"
 #include "open_engine/scene/scene_camera.hpp"
+#include "open_engine/renderer/texture.hpp"
 
 #include <glm/ext/matrix_transform.hpp>
 #include <glm/gtc/type_ptr.hpp>
@@ -45,6 +46,8 @@ namespace OpenEngine {
     struct SpriteRendererComponent 
     {
         glm::vec4 color{ 1.0f, 1.0f, 1.0f, 1.0f };
+        Ref<Texture2D> texture;
+        float tiling_factor = 1.0f;
 
         SpriteRendererComponent() = default;
         SpriteRendererComponent(const SpriteRendererComponent&) = default;

open_engine/include/open_engine/scene/entity.hpp

@@ -22,7 +22,10 @@ namespace OpenEngine {
                 T& AddComponents(Args&&... args)
                 {
                     OE_ASSERT(!HasComponent<T>(), "Entity already has component.");
-                    return scene->registry.emplace<T>(handle, std::forward<Args>(args)...);
+
+                    T& component = scene->registry.emplace<T>(handle, std::forward<Args>(args)...);
+                    scene->OnComponentAdded(*this, component);
+                    return component;
                 };
 
             template<typename T, typename ... Args>

open_engine/include/open_engine/scene/scene.hpp

@@ -27,6 +27,10 @@ namespace OpenEngine {
 
             Entity GetPrimaryCamera();
 
+        private:
+            template<typename T>
+                void OnComponentAdded(Entity entity, T& component);
+
         private:
             entt::registry registry;
 

open_engine/src/open_engine/application.cpp

@@ -13,8 +13,9 @@
 #include <imgui.h>
 
 namespace OpenEngine {
-    Application::Application(const std::string& name)
+    Application::Application(const std::string& name,
-        : name(name)
+            ApplicationCommandLineArgs& args)
+        : name(name), arguments(args)
     {
         OE_PROFILE_FUNCTION();
 

open_engine/src/open_engine/opengl/opengl_shader.cpp

@@ -1,18 +1,24 @@
 #include <pch.hpp>
 
-#include <opengl/opengl_shader.hpp>
-
 #include <core.hpp>
-#include <glm/gtc/type_ptr.hpp>
+#include <opengl/opengl_shader.hpp>
+#include <instrumentor.hpp>
+#include <core/time.hpp>
 #include <logging.hpp>
-#include <glm/glm.hpp>
 
+#include <shaderc/shaderc.hpp>
+#include <spirv_cross/spirv_cross.hpp>
+#include <spirv_cross/spirv_glsl.hpp>
+#include <glm/gtc/type_ptr.hpp>
+#include <glm/glm.hpp>
+#include <filesystem>
 #include <glad/glad.h>
-#include <alloca.h>
+//#include <alloca.h>
-#include <cstddef>
-#include <fstream>
 
 namespace OpenEngine {
+    
+    namespace Utils {
+
         static GLenum ShaderTypeFromString(const std::string& type)
         {
             if (type == "vertex")
@@ -20,11 +26,94 @@ namespace OpenEngine {
             if (type == "fragment" || type == "pixel")
                 return GL_FRAGMENT_SHADER;
 
-        OE_CORE_ERROR("Could not compile shader! Unkown shader type: {}", type);
+            OE_CORE_ASSERT(false, "Unknown shader type!");
-
             return 0;
         }
 
+        static shaderc_shader_kind GLShaderStageToShaderC(GLenum stage)
+        {
+            switch (stage)
+            {
+                case GL_VERTEX_SHADER:   return shaderc_glsl_vertex_shader;
+                case GL_FRAGMENT_SHADER: return shaderc_glsl_fragment_shader;
+            }
+            OE_CORE_ASSERT(false, "No");
+            return (shaderc_shader_kind)0;
+        }
+
+        static const char* GLShaderStageToString(GLenum stage)
+        {
+            switch (stage)
+            {
+                case GL_VERTEX_SHADER:   return "GL_VERTEX_SHADER";
+                case GL_FRAGMENT_SHADER: return "GL_FRAGMENT_SHADER";
+            }
+            OE_CORE_ASSERT(false, "No");
+            return nullptr;
+        }
+
+        static const char* GetCacheDirectory()
+        {
+            // TODO: make sure the assets directory is valid
+            return "assets/cache/shader/opengl";
+        }
+
+        static void CreateCacheDirectoryIfNeeded()
+        {
+            std::string cache_directory = GetCacheDirectory();
+            if (!std::filesystem::exists(cache_directory))
+                std::filesystem::create_directories(cache_directory);
+        }
+
+        static const char* GLShaderStageCachedOpenGLFileExtension(uint32_t stage)
+        {
+            switch (stage)
+            {
+                case GL_VERTEX_SHADER:    return ".cached_opengl.vert";
+                case GL_FRAGMENT_SHADER:  return ".cached_opengl.frag";
+            }
+            OE_CORE_ASSERT(false, "No");
+            return "";
+        }
+
+        static const char* GLShaderStageCachedVulkanFileExtension(uint32_t stage)
+        {
+            switch (stage)
+            {
+                case GL_VERTEX_SHADER:    return ".cached_vulkan.vert";
+                case GL_FRAGMENT_SHADER:  return ".cached_vulkan.frag";
+            }
+            OE_CORE_ASSERT(false, "No");
+            return "";
+        }
+
+
+    }
+
+    OpenGLShader::OpenGLShader(const std::string& filepath)
+        : file_path(filepath)
+    {
+        OE_PROFILE_FUNCTION();
+
+        Utils::CreateCacheDirectoryIfNeeded();
+
+        std::string source = ReadFile(filepath);
+        auto shader_sources = PreProcess(source);
+
+        {
+            CompileOrGetVulkanBinaries(shader_sources);
+            CompileOrGetOpenGLBinaries();
+            CreateProgram();
+        }
+
+        // Extract name from filepath
+        auto last_slash = filepath.find_last_of("/\\");
+        last_slash = last_slash == std::string::npos ? 0 : last_slash + 1;
+        auto lastDot = filepath.rfind('.');
+        auto count = lastDot == std::string::npos ? filepath.size() - last_slash : lastDot - last_slash;
+        name = filepath.substr(last_slash, count);
+    }
+
     OpenGLShader::OpenGLShader(const std::string& name, const std::string& vertex_src, const std::string& fragment_src)
         : name(name)
     {
@@ -34,163 +123,263 @@ namespace OpenEngine {
         sources[GL_VERTEX_SHADER] = vertex_src;
         sources[GL_FRAGMENT_SHADER] = fragment_src;
 
-        Compile(sources);
+        CompileOrGetVulkanBinaries(sources);
+        CompileOrGetOpenGLBinaries();
+        CreateProgram();
     }
 
-    OpenGLShader::OpenGLShader(const std::string& shader_path)
+    OpenGLShader::~OpenGLShader()
     {
         OE_PROFILE_FUNCTION();
 
-        {
+        glDeleteProgram(id);
-            OE_PROFILE_SCOPE("Compiling Shader");
-            std::string source = ReadFile(shader_path);
-            auto shader_sources = PreProcess(source);
-            Compile(shader_sources);
     }
 
-        // Getting file name
+    std::string OpenGLShader::ReadFile(const std::string& filepath)
-        auto last_slash = shader_path.find_last_of("/\\");
-        last_slash = last_slash == std::string::npos ? 0 : last_slash + 1;
-        auto last_dot = shader_path.rfind('.');
-
-        auto count = last_dot == std::string::npos ? shader_path.size() - last_slash : last_dot - last_slash;
-        name = shader_path.substr(last_slash, count);
-    }
-
-    void OpenGLShader::Compile(const std::unordered_map<GLenum, std::string> sources)
-    {
-        OE_PROFILE_FUNCTION();
-
-        GLuint program = glCreateProgram();
-        GLenum *gl_shader_ids = (GLenum*)alloca(sources.size());
-
-        size_t shader_index = 0;
-		for (auto& kv_pair : sources)
-		{
-			GLenum type = kv_pair.first;
-			const std::string& source = kv_pair.second;
-
-			GLuint shader = glCreateShader(type);
-
-			const GLchar* source_cstr = source.c_str();
-			glShaderSource(shader, 1, &source_cstr, 0);
-
-			glCompileShader(shader);
-
-			GLint is_compiled = 0;
-			glGetShaderiv(shader, GL_COMPILE_STATUS, &is_compiled);
-			if (is_compiled == GL_FALSE)
-			{
-				GLint max_length = 0;
-				glGetShaderiv(shader, GL_INFO_LOG_LENGTH, &max_length);
-
-				std::vector<GLchar> info_log(max_length);
-				glGetShaderInfoLog(shader, max_length, &max_length, &info_log[0]);
-
-				glDeleteShader(shader);
-
-				OE_CORE_ERROR("{0}", info_log.data());
-				OE_CORE_ERROR("Shader compilation failure!");
-				break;
-			}
-
-			glAttachShader(program, shader);
-			gl_shader_ids[shader_index++] = shader;
-		}
-		
-		id = program;
-
-		glLinkProgram(program);
-
-		GLint is_linked = 0;
-		glGetProgramiv(program, GL_LINK_STATUS, (int*)&is_linked);
-		if (is_linked == GL_FALSE)
-		{
-			GLint max_length = 0;
-			glGetProgramiv(program, GL_INFO_LOG_LENGTH, &max_length);
-
-			// The maxLength includes the NULL character
-			std::vector<GLchar> infoLog(max_length);
-			glGetProgramInfoLog(program, max_length, &max_length, &infoLog[0]);
-
-			// We don't need the program anymore.
-			glDeleteProgram(program);
-			
-		    for (int i = 0; i < sources.size(); i++)
-				glDeleteShader(gl_shader_ids[i]);
-
-			OE_CORE_ERROR("{0}", infoLog.data());
-			OE_CORE_ERROR("Shader link failure!");
-			return;
-		}
-
-		for (int i = 0; i < sources.size(); i++) {
-			glDetachShader(program, gl_shader_ids[i]);
-            glDeleteShader(gl_shader_ids[i]);
-        }
-    }
-
-    std::unordered_map<GLenum, std::string> OpenGLShader::PreProcess(const std::string& shader_source)
-    {
-        OE_PROFILE_FUNCTION();
-
-        // TODO: Perhaps consider using getline to get the position of errors. or even regexes
-        std::unordered_map<GLenum, std::string> sources;
-        const char* type_token = "#type";
-
-        size_t type_token_lenght = strlen(type_token);
-        size_t pos = shader_source.find(type_token, 0);
-        while (pos != std::string::npos) {
-            size_t eol = shader_source.find("\n", pos);
-            if (eol == std::string::npos)
-                OE_CORE_ERROR("Syntax error.");
-            size_t begin = pos + type_token_lenght + 1;
-            std::string type = shader_source.substr(begin, eol - begin);
-            if (type != "vertex" && type != "fragment" && type != "pixel")
-                OE_CORE_ERROR("Invalid shader type.");
-
-            size_t next_line_pos = shader_source.find_first_not_of("\n", eol);
-            pos = shader_source.find(type_token, next_line_pos);
-            sources[ShaderTypeFromString(type)] =
-                shader_source.substr(next_line_pos, pos - (next_line_pos == std::string::npos ? shader_source.size() - 1 : next_line_pos));
-        }
-
-        return sources;
-    }
-
-    std::string OpenGLShader::ReadFile(const std::string& shader_path)
     {
         OE_PROFILE_FUNCTION();
 
         std::string result;
-
+        std::ifstream in(filepath, std::ios::in | std::ios::binary); // ifstream closes itself due to RAII
-        std::ifstream input(shader_path);
+        if (in)
-        if (input) {
+        {
-            input.seekg(0, std::ios::end);
+            in.seekg(0, std::ios::end);
-
+            size_t size = in.tellg();
-            size_t size = input.tellg();
+            if (size != -1)
-            if (size != -1) {
+            {
                 result.resize(size);
-                input.seekg(0, std::ios::beg);
+                in.seekg(0, std::ios::beg);
-                input.read(&result[0], result.size());
+                in.read(&result[0], size);
-                input.close();
-            } else {
-                OE_CORE_ERROR("Could not read from file {}", shader_path);
             }
-        } else {
+            else
-            OE_CORE_ERROR("Shader file could not be open or does not exist: {}", shader_path);
+            {
+                OE_CORE_ERROR("Could not read from file '{0}'", filepath);
+            }
+        }
+        else
+        {
+            OE_CORE_ERROR("Could not open file '{0}'", filepath);
         }
 
         return result;
     }
 
+    std::unordered_map<GLenum, std::string> OpenGLShader::PreProcess(const std::string& source)
+    {
+        OE_PROFILE_FUNCTION();
+
+        std::unordered_map<GLenum, std::string> shader_sources;
+
+        const char* type_token = "#type";
+        size_t type_token_length = strlen(type_token);
+        size_t pos = source.find(type_token, 0); //Start of shader type declaration line
+        while (pos != std::string::npos)
+        {
+            size_t eol = source.find_first_of("\r\n", pos); //End of shader type declaration line
+            OE_CORE_ASSERT(eol != std::string::npos, "Syntax error");
+            size_t begin = pos + type_token_length + 1; //Start of shader type name (after "#type " keyword)
+            std::string type = source.substr(begin, eol - begin);
+            OE_CORE_ASSERT(Utils::ShaderTypeFromString(type), "Invalid shader type specified");
+
+            size_t next_line_pos = source.find_first_not_of("\r\n", eol); //Start of shader code after shader type declaration line
+            OE_CORE_ASSERT(next_line_pos != std::string::npos, "Syntax error");
+            pos = source.find(type_token, next_line_pos); //Start of next shader type declaration line
+
+            shader_sources[Utils::ShaderTypeFromString(type)] = (pos == std::string::npos) ? source.substr(next_line_pos) : source.substr(next_line_pos, pos - next_line_pos);
+        }
+
+        return shader_sources;
+    }
+
+    void OpenGLShader::CompileOrGetVulkanBinaries(const std::unordered_map<GLenum, std::string>& shaderSources)
+    {
+        GLuint program = glCreateProgram();
+
+        shaderc::Compiler compiler;
+        shaderc::CompileOptions options;
+        options.SetTargetEnvironment(shaderc_target_env_vulkan, shaderc_env_version_vulkan_1_2);
+        const bool optimize = true;
+        if (optimize)
+            options.SetOptimizationLevel(shaderc_optimization_level_performance);
+
+        std::filesystem::path cache_directory = Utils::GetCacheDirectory();
+
+        auto& shader_data = vulkan_spirv;
+        shader_data.clear();
+        for (auto&& [stage, source] : shaderSources)
+        {
+            std::filesystem::path shader_file_path = file_path;
+            std::filesystem::path cached_path = cache_directory / (shader_file_path.filename().string() + Utils::GLShaderStageCachedVulkanFileExtension(stage));
+
+            std::ifstream in(cached_path, std::ios::in | std::ios::binary);
+            if (in.is_open())
+            {
+                in.seekg(0, std::ios::end);
+                auto size = in.tellg();
+                in.seekg(0, std::ios::beg);
+
+                auto& data = shader_data[stage];
+                data.resize(size / sizeof(uint32_t));
+                in.read((char*)data.data(), size);
+            }
+            else
+            {
+                shaderc::SpvCompilationResult module = compiler.CompileGlslToSpv(source, Utils::GLShaderStageToShaderC(stage), file_path.c_str(), options);
+                if (module.GetCompilationStatus() != shaderc_compilation_status_success)
+                {
+                    OE_CORE_ERROR(module.GetErrorMessage());
+                    OE_CORE_ASSERT(false, "No");
+                }
+
+                shader_data[stage] = std::vector<uint32_t>(module.cbegin(), module.cend());
+
+                std::ofstream out(cached_path, std::ios::out | std::ios::binary);
+                if (out.is_open())
+                {
+                    auto& data = shader_data[stage];
+                    out.write((char*)data.data(), data.size() * sizeof(uint32_t));
+                    out.flush();
+                    out.close();
+                }
+            }
+        }
+
+        for (auto&& [stage, data]: shader_data)
+            Reflect(stage, data);
+    }
+
+    void OpenGLShader::CompileOrGetOpenGLBinaries()
+    {
+        auto& shader_data = opengl_spirv;
+
+        shaderc::Compiler compiler;
+        shaderc::CompileOptions options;
+        options.SetTargetEnvironment(shaderc_target_env_opengl, shaderc_env_version_opengl_4_5);
+        const bool optimize = false;
+        if (optimize)
+            options.SetOptimizationLevel(shaderc_optimization_level_performance);
+
+        std::filesystem::path cache_directory = Utils::GetCacheDirectory();
+
+        shader_data.clear();
+        m_OpenGLSourceCode.clear();
+        for (auto&& [stage, spirv] : vulkan_spirv)
+        {
+            std::filesystem::path shader_file_path = file_path;
+            std::filesystem::path cached_path = cache_directory / (shader_file_path.filename().string() + Utils::GLShaderStageCachedOpenGLFileExtension(stage));
+
+            std::ifstream in(cached_path, std::ios::in | std::ios::binary);
+            if (in.is_open())
+            {
+                in.seekg(0, std::ios::end);
+                auto size = in.tellg();
+                in.seekg(0, std::ios::beg);
+
+                auto& data = shader_data[stage];
+                data.resize(size / sizeof(uint32_t));
+                in.read((char*)data.data(), size);
+            }
+            else
+            {
+                spirv_cross::CompilerGLSL glsl_compiler(spirv);
+                m_OpenGLSourceCode[stage] = glsl_compiler.compile();
+                auto& source = m_OpenGLSourceCode[stage];
+
+                shaderc::SpvCompilationResult module = compiler.CompileGlslToSpv(source, Utils::GLShaderStageToShaderC(stage), file_path.c_str());
+                if (module.GetCompilationStatus() != shaderc_compilation_status_success)
+                {
+                    OE_CORE_ERROR(module.GetErrorMessage());
+                    OE_CORE_ASSERT(false, "No");
+                }
+
+                shader_data[stage] = std::vector<uint32_t>(module.cbegin(), module.cend());
+
+                std::ofstream out(cached_path, std::ios::out | std::ios::binary);
+                if (out.is_open())
+                {
+                    auto& data = shader_data[stage];
+                    out.write((char*)data.data(), data.size() * sizeof(uint32_t));
+                    out.flush();
+                    out.close();
+                }
+            }
+        }
+    }
+
+    void OpenGLShader::CreateProgram()
+    {
+        GLuint program = glCreateProgram();
+
+        std::vector<GLuint> shader_ids;
+        for (auto&& [stage, spirv] : opengl_spirv)
+        {
+            GLuint shaderID = shader_ids.emplace_back(glCreateShader(stage));
+            glShaderBinary(1, &shaderID, GL_SHADER_BINARY_FORMAT_SPIR_V, spirv.data(), spirv.size() * sizeof(uint32_t));
+            glSpecializeShader(shaderID, "main", 0, nullptr, nullptr);
+            glAttachShader(program, shaderID);
+        }
+
+        glLinkProgram(program);
+
+        GLint is_linked;
+        glGetProgramiv(program, GL_LINK_STATUS, &is_linked);
+        if (is_linked == GL_FALSE)
+        {
+            GLint max_length;
+            glGetProgramiv(program, GL_INFO_LOG_LENGTH, &max_length);
+
+            std::vector<GLchar> info_log(max_length);
+            glGetProgramInfoLog(program, max_length, &max_length, info_log.data());
+            OE_CORE_ERROR("Shader linking failed ({0}):\n{1}", file_path, info_log.data());
+
+            glDeleteProgram(program);
+
+            for (auto id : shader_ids)
+                glDeleteShader(id);
+        }
+
+        for (auto id : shader_ids)
+        {
+            glDetachShader(program, id);
+            glDeleteShader(id);
+        }
+
+        id = program;
+    }
+
+    void OpenGLShader::Reflect(GLenum stage, const std::vector<uint32_t>& shader_data)
+    {
+        spirv_cross::Compiler compiler(shader_data);
+        spirv_cross::ShaderResources resources = compiler.get_shader_resources();
+
+        OE_CORE_TRACE("OpenGLShader::Reflect - {0} {1}", Utils::GLShaderStageToString(stage), file_path);
+        OE_CORE_TRACE("    {0} uniform buffers", resources.uniform_buffers.size());
+        OE_CORE_TRACE("    {0} resources", resources.sampled_images.size());
+
+        OE_CORE_TRACE("Uniform buffers:");
+        for (const auto& resource : resources.uniform_buffers)
+        {
+            const auto& buffer_type = compiler.get_type(resource.base_type_id);
+            uint32_t buffer_size = compiler.get_declared_struct_size(buffer_type);
+            uint32_t binding = compiler.get_decoration(resource.id, spv::DecorationBinding);
+            int member_count = buffer_type.member_types.size();
+
+            OE_CORE_TRACE("  {0}", resource.name);
+            OE_CORE_TRACE("    Size = {0}", buffer_size);
+            OE_CORE_TRACE("    Binding = {0}", binding);
+            OE_CORE_TRACE("    Members = {0}", member_count);
+        }
+    }
+
     void OpenGLShader::Bind() const
     { 
+        OE_PROFILE_FUNCTION();
         glUseProgram(id); 
     }
 
     void OpenGLShader::Unbind() const
     {
+        OE_PROFILE_FUNCTION();
         glUseProgram(0);
     }
 
@@ -275,30 +464,4 @@ namespace OpenEngine {
     {
         glUniform4fv(glGetUniformLocation(id, name.c_str()), 1, glm::value_ptr(value)); 
     }
-
-    void OpenGLShader::CheckCompileErrors(unsigned int shader, const std::string& type)
-    {
-        int success = GL_FALSE;
-        char infoLog[1024];
-        if (type != "PROGRAM") {
-            glGetShaderiv(shader, GL_COMPILE_STATUS, &success);
-            if (success == GL_FALSE) {
-                glGetShaderInfoLog(shader, 1024, NULL, infoLog);
-                OE_CORE_ERROR("Shader of type {}, couldn't be compiled:", type);
-                OE_CORE_ERROR("\t{}", infoLog);
-            }
-        } else {
-            if (glGetProgramiv == nullptr) {
-                OE_CORE_ERROR("OpenGL functions are not loaded!");
-                return;
-            }
-            glGetProgramiv(shader, GL_LINK_STATUS, &success);
-            if (success == GL_FALSE)
-            {
-                glGetProgramInfoLog(shader, 1024, NULL, infoLog);
-                OE_CORE_ERROR("Program of type {}, couldn't be compiled:", type);
-                OE_CORE_ERROR("\t{}", infoLog);
-            }
-        }
-    }
 }

open_engine/src/open_engine/opengl/opengl_texture.cpp

@@ -22,7 +22,7 @@ namespace OpenEngine {
 		glTextureStorage2D(id, 1, internal_format, width, height);
 
 		glTextureParameteri(id, GL_TEXTURE_MIN_FILTER, GL_LINEAR);
-		glTextureParameteri(id, GL_TEXTURE_MAG_FILTER, GL_NEAREST);
+		glTextureParameteri(id, GL_TEXTURE_MAG_FILTER, GL_LINEAR);
 
 		glTextureParameteri(id, GL_TEXTURE_WRAP_S, GL_REPEAT);
         glTextureParameteri(id, GL_TEXTURE_WRAP_T, GL_REPEAT);
@@ -59,7 +59,7 @@ namespace OpenEngine {
         glTextureStorage2D(id, 1, _internal_format, width, height);
 
         glTextureParameteri(id, GL_TEXTURE_MIN_FILTER, GL_LINEAR);
-        glTextureParameteri(id, GL_TEXTURE_MAG_FILTER, GL_NEAREST);
+        glTextureParameteri(id, GL_TEXTURE_MAG_FILTER, GL_LINEAR);
 
         glTextureSubImage2D(id, 0, 0, 0, width, height, _data_format, GL_UNSIGNED_BYTE, data);
 

open_engine/src/open_engine/opengl/opengl_uniform_buffer.cpp

@@ -0,0 +1,27 @@
+#include <pch.hpp>
+
+#include <opengl/opengl_uniform_buffer.hpp>
+
+#include <glad/glad.h>
+
+namespace OpenEngine {
+
+    OpenGLUniformBuffer::OpenGLUniformBuffer(uint32_t size, uint32_t binding)
+	{
+		glCreateBuffers(1, &id);
+		glNamedBufferData(id, size, nullptr, GL_DYNAMIC_DRAW);
+		glBindBufferBase(GL_UNIFORM_BUFFER, binding, id);
+	}
+
+	OpenGLUniformBuffer::~OpenGLUniformBuffer()
+	{
+		glDeleteBuffers(1, &id);
+	}
+
+
+	void OpenGLUniformBuffer::SetData(const void* data, uint32_t size, uint32_t offset)
+	{
+		glNamedBufferSubData(id, offset, size, data);
+	}
+
+}

open_engine/src/open_engine/renderer/editor_camera.cpp

@@ -63,8 +63,7 @@ namespace OpenEngine {
 
 	void EditorCamera::OnUpdate()
 	{
-		if (Input::IsKeyPressed(Key::LeftAlt))
+		if (Input::IsKeyPressed(Key::LeftAlt)) {
-		{
 			const glm::vec2& mouse{ Input::GetMouseX(), Input::GetMouseY() };
 			glm::vec2 delta = (mouse - initial_mouse_position) * 0.003f;
 			initial_mouse_position = mouse;

open_engine/src/open_engine/renderer/renderer2d.cpp

@@ -1,4 +1,5 @@
 #include "renderer/editor_camera.hpp"
+#include "renderer/uniform_buffer.hpp"
 #include <pch.hpp>
 
 #include <renderer/render_command.hpp>
@@ -27,7 +28,7 @@ namespace OpenEngine {
 
     struct Renderer2DData
     {
-        static const uint32_t max_quads = 10000;
+        static const uint32_t max_quads = 20000;
         static const uint32_t max_vertices = max_quads * 4;
         static const uint32_t max_indices = max_quads * 6;
         static const uint32_t max_texture_slots = 32;
@@ -47,6 +48,13 @@ namespace OpenEngine {
         glm::vec4 quad_vertex_positioning[4];
 
         Statistics stats;
+
+        struct CameraData
+        {
+            glm::mat4 view_projection;
+        };
+        CameraData camera_buffer;
+        Ref<UniformBuffer> camera_uniform_buffer;
     };
 
     static Renderer2DData renderer_data;
@@ -100,8 +108,6 @@ namespace OpenEngine {
             samplers[i] = i;
 
         renderer_data.texture_shader = Shader::Create("assets/shaders/texture.glsl");
-        renderer_data.texture_shader->Bind();
-        renderer_data.texture_shader->SetIntArray("u_Texture", samplers, renderer_data.max_texture_slots);
 
         renderer_data.texture_slots[0] = renderer_data.white_texture;
 
@@ -109,6 +115,8 @@ namespace OpenEngine {
         renderer_data.quad_vertex_positioning[1] = {0.5, -0.5, 0.0f, 1.0f};
         renderer_data.quad_vertex_positioning[2] = {0.5, 0.5, 0.0f, 1.0f};
         renderer_data.quad_vertex_positioning[3] = {-0.5, 0.5, 0.0f, 1.0f};
+
+        renderer_data.camera_uniform_buffer = UniformBuffer::Create(sizeof(Renderer2DData), 0);
     }
 
     void Renderer2D::Shutdown()
@@ -130,8 +138,8 @@ namespace OpenEngine {
     {
         OE_PROFILE_FUNCTION();
 
-        renderer_data.texture_shader->Bind();
+        renderer_data.camera_buffer.view_projection = camera.GetViewProjectionMatrix();
-        renderer_data.texture_shader->SetMat4("u_ViewProjection", camera.GetViewProjectionMatrix());
+        renderer_data.camera_uniform_buffer->SetData(&renderer_data.camera_buffer, sizeof(Renderer2DData::CameraData));
 
         StartBatch();
     }
@@ -140,10 +148,8 @@ namespace OpenEngine {
     {
         OE_PROFILE_FUNCTION();
 
-        glm::mat4 view_projection = camera.GetProjection() * glm::inverse(transform);
+        renderer_data.camera_buffer.view_projection = camera.GetProjection() * glm::inverse(transform);
-
+		renderer_data.camera_uniform_buffer->SetData(&renderer_data.camera_buffer, sizeof(Renderer2DData::CameraData));
-        renderer_data.texture_shader->Bind();
-        renderer_data.texture_shader->SetMat4("u_ViewProjection", view_projection);
 
         StartBatch();
     }
@@ -152,10 +158,8 @@ namespace OpenEngine {
     {
         OE_PROFILE_FUNCTION();
 
-        glm::mat4 view_projection = camera.GetViewProjection();
+        renderer_data.camera_buffer.view_projection = camera.GetViewProjection();
-
+		renderer_data.camera_uniform_buffer->SetData(&renderer_data.camera_buffer, sizeof(Renderer2DData::CameraData));
-        renderer_data.texture_shader->Bind();
-        renderer_data.texture_shader->SetMat4("u_ViewProjection", view_projection);
 
         StartBatch();
     }
@@ -180,6 +184,7 @@ namespace OpenEngine {
         for (int i = 0; i < renderer_data.texture_slot_index; i++)
             renderer_data.texture_slots[i]->Bind(i);
 
+        renderer_data.texture_shader->Bind();
         RenderCommand::DrawIndexed(renderer_data.vertex_array, renderer_data.quad_index_count);
         
         renderer_data.stats.draw_calls++;
@@ -225,6 +230,43 @@ namespace OpenEngine {
         DrawQuad(transform, texture, entity_id);
     }
 
+    void Renderer2D::DrawQuad(const Transform& transform_data,
+            const Ref<Texture2D>& texture, const glm::vec4& color,
+            int entity_id, float tiling_factor)
+    {
+        OE_PROFILE_FUNCTION();
+
+        constexpr size_t quad_vertex_count = 4;
+        constexpr glm::vec2 texture_coords[] = { { 0.0f, 0.0f }, { 1.0f, 0.0f }, { 1.0f, 1.0f }, { 0.0f, 1.0f }};
+
+        if (renderer_data.quad_index_count >= Renderer2DData::max_indices)
+            FlushAndReset();
+
+        glm::vec3 position = transform_data.position;
+        glm::vec3 size = transform_data.size;
+
+        float texture_index = 0;
+        for (uint32_t i = 1; i < renderer_data.texture_slot_index; i++) {
+            if (*renderer_data.texture_slots[i].get() == *texture.get()) {
+                texture_index = (float)i;
+                break;
+            }
+        }
+
+        if (texture_index == 0) {
+            if (renderer_data.texture_slot_index >= Renderer2DData::max_texture_slots)
+                FlushAndReset();
+
+            texture_index = (float)renderer_data.texture_slot_index;
+            renderer_data.texture_slots[renderer_data.texture_slot_index] = texture;
+            renderer_data.texture_slot_index++;
+        }
+
+        glm::mat4 transform = glm::translate(glm::mat4(1.0f), transform_data.position)
+            * glm::rotate(glm::mat4(1.0f), glm::radians(transform_data.rotation), { 0.0f, 0.0f, 1.0f })
+            * glm::scale(glm::mat4(1.0f), transform_data.size);
+    }
+
     void Renderer2D::DrawQuad(const Transform& transform_data,
             const Ref<Subtexture2D>& subtexture, int entity_id,
             float tiling_factor)
@@ -333,6 +375,49 @@ namespace OpenEngine {
         renderer_data.stats.quad_count++;
     }
 
+    void Renderer2D::DrawQuad(const glm::mat4& transform,
+            const Ref<Texture2D>& texture, const glm::vec4& color,
+            int entity_id, float tiling_factor)
+    {
+        OE_PROFILE_FUNCTION();
+
+        constexpr size_t quad_vertex_count = 4;
+        constexpr glm::vec2 texture_coords[] = { { 0.0f, 0.0f }, { 1.0f, 0.0f }, { 1.0f, 1.0f }, { 0.0f, 1.0f }};
+
+        if (renderer_data.quad_index_count >= Renderer2DData::max_indices)
+            FlushAndReset();
+
+        float texture_index = 0;
+        for (uint32_t i = 1; i < renderer_data.texture_slot_index; i++) {
+            if (*renderer_data.texture_slots[i].get() == *texture.get()) {
+                texture_index = (float)i;
+                break;
+            }
+        }
+
+        if (texture_index == 0) {
+            if (renderer_data.texture_slot_index >= Renderer2DData::max_texture_slots)
+                FlushAndReset();
+
+            texture_index = (float)renderer_data.texture_slot_index;
+            renderer_data.texture_slots[renderer_data.texture_slot_index] = texture;
+            renderer_data.texture_slot_index++;
+        }
+
+        for (size_t i = 0; i < quad_vertex_count; i++) {
+            renderer_data.quad_vertex_ptr->position = transform * renderer_data.quad_vertex_positioning[i];
+            renderer_data.quad_vertex_ptr->color = color;
+            renderer_data.quad_vertex_ptr->tex_coord = texture_coords[i];
+            renderer_data.quad_vertex_ptr->tex_index = texture_index;
+            renderer_data.quad_vertex_ptr->tiling_factor = tiling_factor;
+            renderer_data.quad_vertex_ptr->id = entity_id;
+            renderer_data.quad_vertex_ptr++;
+        }
+
+        renderer_data.quad_index_count += 6;
+        renderer_data.stats.quad_count++;
+    }
+
     void Renderer2D::DrawQuad(const glm::mat4& transform,
             const Ref<Subtexture2D>& subtexture, int entity_id,
             float tiling_factor)

open_engine/src/open_engine/renderer/uniform_buffer.cpp

@@ -0,0 +1,21 @@
+#include <pch.hpp>
+
+#include <renderer/uniform_buffer.hpp>
+#include <opengl/opengl_uniform_buffer.hpp>
+#include <renderer/renderer.hpp>
+
+namespace OpenEngine {
+
+    Ref<UniformBuffer> UniformBuffer::Create(uint32_t size, uint32_t binding)
+    {
+        switch (Renderer::GetAPI())
+        {
+            case RendererAPI::API::None:    OE_CORE_ASSERT(false, "RendererAPI::None is currently not supported!"); return nullptr;
+            case RendererAPI::API::OpenGL:  return CreateRef<OpenGLUniformBuffer>(size, binding);
+        }
+
+        OE_CORE_ASSERT(false, "Unknown RendererAPI!");
+        return nullptr;
+    }
+
+}

open_engine/src/open_engine/scene/camera.cpp

@@ -1,3 +1,4 @@
+#include "core.hpp"
 #include <glm/trigonometric.hpp>
 #include <pch.hpp>
 
@@ -32,6 +33,7 @@ namespace OpenEngine {
 
     void SceneCamera::SetViewportSize(uint32_t width, uint32_t height)
     {
+        OE_CORE_ASSERT(width > 0 && height > 0, "Viewport size invalid.");
         aspect_ratio = (float)width / (float)height;
 
         RecalculateProjection();

open_engine/src/open_engine/scene/scene.cpp

@@ -59,7 +59,13 @@ namespace OpenEngine {
 
                 for (const auto& entity : view) {
                     auto [transform, sprite] = view.get<TransformComponent, SpriteRendererComponent>(entity);
-                    Renderer2D::DrawQuad(GetTransformFromComp(transform), sprite.color, (int)entity);
+
+                    if (sprite.texture)
+                        Renderer2D::DrawQuad(GetTransformFromComp(transform),
+                                sprite.texture, sprite.color, (int)entity, sprite.tiling_factor);
+                    else
+                        Renderer2D::DrawQuad(GetTransformFromComp(transform),
+                                sprite.color, (int)entity);
                 }
 
                 Renderer2D::EndScene();
@@ -76,7 +82,12 @@ namespace OpenEngine {
 		{
 			auto [transform, sprite] = group.get<TransformComponent, SpriteRendererComponent>(entity);
 
-			Renderer2D::DrawQuad(GetTransformFromComp(transform), sprite.color, (int)entity);
+            if (sprite.texture)
+                Renderer2D::DrawQuad(GetTransformFromComp(transform),
+                        sprite.texture, sprite.color, (int)entity, sprite.tiling_factor);
+            else
+                Renderer2D::DrawQuad(GetTransformFromComp(transform),
+                        sprite.color, (int)entity);
 		}
 
 		Renderer2D::EndScene();
@@ -107,4 +118,37 @@ namespace OpenEngine {
 
         return {};
     }
+
+    template<typename T>
+        void Scene::OnComponentAdded(Entity entity, T& component)
+        {
+            static_assert(false);
+        }
+
+    template<>
+	void Scene::OnComponentAdded<TransformComponent>(Entity entity, TransformComponent& component)
+	{
+	}
+
+    template<>
+	void Scene::OnComponentAdded<CameraComponent>(Entity entity, CameraComponent& component)
+	{
+		if (viewport_width > 0 && viewport_height > 0)
+			component.camera.SetViewportSize(viewport_width, viewport_height);
+	}
+
+    template<>
+	void Scene::OnComponentAdded<SpriteRendererComponent>(Entity entity, SpriteRendererComponent& component)
+	{
+	}
+
+	template<>
+	void Scene::OnComponentAdded<TagComponent>(Entity entity, TagComponent& component)
+	{
+	}
+
+	template<>
+	void Scene::OnComponentAdded<NativeScriptComponent>(Entity entity, NativeScriptComponent& component)
+	{
+	}
 }