-
Notifications
You must be signed in to change notification settings - Fork 0
End of SPRINT 01 #49
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
End of SPRINT 01 #49
Conversation
* Engine lifecycle * Window management with GLFW
…ray::get_vertex_buffers
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
This PR implements the foundational components for Sprint 01 of the LMGL graphics engine, establishing core rendering infrastructure, scene graph management, and comprehensive test coverage.
Key Changes:
- Complete implementation of the rendering pipeline (shaders, buffers, textures, framebuffers, vertex arrays)
- Scene graph system with hierarchical node transforms and mesh management
- Camera system with perspective/orthographic projection and screen-to-world unprojection
- Comprehensive test suite across all components with CI/CD improvements
Reviewed changes
Copilot reviewed 49 out of 63 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| test/scene/scene_test.cpp | Tests scene graph hierarchy and transform propagation |
| test/scene/node_test.cpp | Tests node transforms, parent-child relationships, and rotation handling |
| test/scene/mesh_test.cpp | Tests procedural mesh generation (cube, quad, sphere) and vertex data |
| test/scene/camera_test.cpp | Tests camera projection modes, view matrices, and screen unprojection |
| test/renderer/vertex_array_test.cpp | Tests VAO creation, buffer attachment, and layout configuration |
| test/renderer/texture_test.cpp | Tests texture creation, resizing, and data management |
| test/renderer/shader_test.cpp | Tests shader compilation, uniform management, and library system |
| test/renderer/renderer_test.cpp | Tests render pipeline, draw calls, and render mode switching |
| test/renderer/framebuffer_test.cpp | Tests framebuffer creation, attachment, and resizing |
| test/renderer/buffer_test.cpp | Tests vertex/index buffer layout and stride calculations |
| test/core/engine_test.cpp | Tests singleton engine initialization and delta time tracking |
| src/scene/scene.cpp | Implements scene graph update and transform propagation |
| src/scene/node.cpp | Implements hierarchical node transforms and parent-child management |
| src/scene/mesh.cpp | Implements procedural mesh generation with vertex/index data |
| src/scene/camera.cpp | Implements camera projection and view matrix calculation |
| src/renderer/vertex_array.cpp | Implements VAO with vertex attribute configuration |
| src/renderer/texture.cpp | Implements texture loading, binding, and data management with STB |
| src/renderer/shader.cpp | Implements shader compilation, linking, and uniform caching |
| src/renderer/renderer.cpp | Implements render queue sorting and batched mesh rendering |
| src/renderer/framebuffer.cpp | Implements framebuffer with color/depth attachments |
| src/renderer/buffer.cpp | Implements vertex/index buffers with layout management |
| src/core/engine.cpp | Implements singleton engine with GLFW initialization and main loop |
| shaders/*.{vert,frag,glsl} | Adds basic and flat-color shaders for rendering |
| sandbox/main.cpp | Demonstrates engine features with animated scene |
| include/lmgl/**/*.hpp | Declares all classes with comprehensive documentation |
| CMakeLists.txt | Updates build configuration with new sources and headless mode |
| README.md | Adds project documentation and badges |
Comments suppressed due to low confidence (1)
test/core/engine_test.cpp:1
- Corrected spelling of 'a Same' to 'the Same' in test name.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Implemented: