addressing more comments.

Author: gpx1000

attachments/simple_engine/engine.cpp

@@ -411,16 +411,8 @@ void Engine::Render() {
         return;
     }
 
-    // Get all active entities
-    std::vector<Entity*> activeEntities;
-    for (auto& entity : entities) {
-        if (entity->IsActive()) {
-            activeEntities.push_back(entity.get());
-        }
-    }
-
     // Render the scene (ImGui will be rendered within the render pass)
-    renderer->Render(activeEntities, activeCamera, imguiSystem.get());
+    renderer->Render(entities, activeCamera, imguiSystem.get());
 }
 
 float Engine::CalculateDeltaTime() {

attachments/simple_engine/renderer.h

@@ -9,6 +9,7 @@
 #include <unordered_map>
 #include <mutex>
 #include <algorithm>
+#include <memory>
 
 #include "platform.h"
 #include "entity.h"
@@ -135,7 +136,7 @@ class Renderer {
      * @param camera The camera to use for rendering.
      * @param imguiSystem The ImGui system for UI rendering (optional).
      */
-    void Render(const std::vector<Entity*>& entities, CameraComponent* camera, ImGuiSystem* imguiSystem = nullptr);
+    void Render(const std::vector<std::unique_ptr<Entity>>& entities, CameraComponent* camera, ImGuiSystem* imguiSystem = nullptr);
 
     /**
      * @brief Wait for the device to be idle.

attachments/simple_engine/renderer_rendering.cpp

@@ -504,7 +504,7 @@ void Renderer::updateUniformBufferInternal(uint32_t currentImage, Entity* entity
 }
 
 // Render the scene
-void Renderer::Render(const std::vector<Entity*>& entities, CameraComponent* camera, ImGuiSystem* imguiSystem) {
+void Renderer::Render(const std::vector<std::unique_ptr<Entity>>& entities, CameraComponent* camera, ImGuiSystem* imguiSystem) {
     // Set rendering active to prevent memory pool growth during rendering
     if (memoryPool) {
         memoryPool->setRenderingActive(true);
@@ -608,7 +608,11 @@ void Renderer::Render(const std::vector<Entity*>& entities, CameraComponent* cam
     vk::raii::PipelineLayout* currentLayout = nullptr;
 
     // Render each entity
-    for (auto entity : entities) {
+    for (auto const& uptr : entities) {
+        Entity* entity = uptr.get();
+        if (!entity || !entity->IsActive()) {
+            continue;
+        }
         // Check if ball-only rendering is enabled and filter entities accordingly
         if (imguiSystem && imguiSystem->IsBallOnlyRenderingEnabled()) {
             // Only render entities whose names contain "Ball_"