Skip to content

✨Migrating Core Renderer to using vulkan-cpp V6#205

Merged
SpinnerX merged 106 commits into
mainfrom
vulkan_v6
Jun 15, 2026
Merged

✨Migrating Core Renderer to using vulkan-cpp V6#205
SpinnerX merged 106 commits into
mainfrom
vulkan_v6

Conversation

@SpinnerX

@SpinnerX SpinnerX commented Jun 14, 2026

Copy link
Copy Markdown
Member

Brief Overview

This PR migrates the core renderer to vulkan-cpp V6. Full changes can be viewed here.

Alongside with this rewrite, this includes major refactoring and architectural cleanups related to how modules are being imported in the codebase.

Changelogs

  • Design: Rewrite the core renderer to use vulkan-cpp V6 APIs.
  • Bottleneck: Eliminate descriptor-per-object bottlenecks and replaced individual descriptor buffer objects to using buffer device addresses.
  • Improvement: Optimize for large texture arrays with index as the accessing points to handle large storage image resources.
  • Refactor: to better utilize C++20 module partitions rather then performing individual module imports.
  • Refactor: Decoupled editor-level logic in level scene to editor_world.cppm.
  • Bug Fixes: Prevent crashes when not loading an HDRI environment; to load a default pixel color.
  • Bonus: Feature to perform Image-Based Lighting (IBL) works due to the flexibility for preparation for the PBR system.

SpinnerX added 30 commits May 28, 2026 01:26
@SpinnerX
More maintenance and logic cleanup for better flow for refactoring
109e332
@SpinnerX
Did some more maintenance in the editor code
4b09b0f
@SpinnerX
WIP in migrating renderer to fully using vulkan-cpp
8171cb9
@SpinnerX
WIP: removed altas::vk::swapchain in replacing with vk::swapchain
81d8924
@SpinnerX
Refactor: Dyanmic rendering to change background color. Continue maki…
67b3286 
…ng more progress
@SpinnerX
Moved location for calling logical_device->wait() call
8ac88b1
@SpinnerX
Removed old files that are not needed + removing redundant import dep…
43fabb9 
…endencies from codebase
@SpinnerX
Created listeners for event::bus in application to remove warnings
9561260
@SpinnerX
Removed old outdated shader files
967f8c6
@SpinnerX
Added the render_context class for preparing rendering operations aut…
43b91a3 
…omation
@SpinnerX
refactor: reworking render to use dynamic rendering and get triangle …
60c4b92 
…working once again
@SpinnerX
refactor: Called destruct call for render_context to ensure proper cl…
6d3ecac 
…eanup
@SpinnerX
refactor: Added backt the level scene code
080ba6c
@SpinnerX
Reworked some of the API to start integrating into the renderer
6d5e3ac
@SpinnerX
refactor: Getting viking demo using variety of modern Vulkan features…
73f9d1b 
… to work
@SpinnerX
refactor: Added back the working camera system
fd795fd
@SpinnerX
refactor: Re-added the mesh loading to the render_context
127ecbd
@SpinnerX
refactor: Did some code cleanup in render_context removing old code n…
5d6d133 
…ot in use
@SpinnerX
refactor: Added mesh_source to viking room demo
18bfe1b
@SpinnerX
Fixed some buffer parameters specifications
d622d55
@SpinnerX
refactor: Added testing objects for demonstration
fe3a5e0
@SpinnerX
refactor: Added testing example code for scene transitions, not part …
b2f6e23 
…of this change. For render context, I did some minor code cleanup and fixed naming conventions
@SpinnerX
refactor: Re-added texture loading and did some linter cleanup for sp…
8cf8305 
…ecific API parameters
@SpinnerX
refactor: wip in getting material table lookup logic to work furthermore
71a5f90
@SpinnerX
Fixed texture size for descriptor allocations
a5d79a5
@SpinnerX
refactor: Redid the core renderer to better handle model matrices and…
8f78443 
… scene related uniforms using the power of indexing to large storage buffers on the GPU
@SpinnerX
refactor: update to using vulkan-cpp/6.1
4c8eceb
@SpinnerX
refactor: code cleanup and removing any dead code that were present a…
3c5f1bd 
…nd redid some of the logic better
@SpinnerX
refactor: remove dead code commented in entry_point.cpp
a74b2dc
@SpinnerX
refactor: update project CMakeLists.txt
2748401
SpinnerX added 19 commits June 12, 2026 18:04
@SpinnerX
refactor: removed old unused glsl shaders from builtin.shaders
5076f03
@SpinnerX
refactor: disable window resizing temporarily
d758251
@SpinnerX
refactor: moved gpu-specific data to gpu_uniforms.cppm
8ba2d8f
@SpinnerX
refactor: update project CMakeLists.txt to include gpu_uniforms.cppm
1fd6d24
@SpinnerX
refactor: fixed logger imports to be a module partition rather then r…
14d6562 
…epresenting as an indivudal primary module
@SpinnerX
refactor: removed unused logger imports as it is replaced with the ut…
096cbbd 
…ilities import instead
@SpinnerX
refactor: changed import atlas.common to making common.cppm as a modu…
e7ff24e 
…le partition
@SpinnerX
refactor: remove graphics_api enum as there is no reason for an RHI a…
f4b6834 
…bstraction
@SpinnerX
refactor: move state.cppm and poll_state.cppm as module partitions
1b4a2d5
@SpinnerX
refactor: added ui.cppm for partitioning the moudle units for widgets…
1399033 
… and imgui_stdlib.cppm
@SpinnerX
refactor: minor fixes in ui.cppm
44f5e3f
@SpinnerX
refactor: scenes to represent imports as module partitions rather the…
cc7d422 
…n individual partition units
@SpinnerX
refactor: made jolt_cpp implementation imports be represented as modu…
6211cfa 
…le partitions
@SpinnerX
refactor: update the project CMakeLists.txt
051a114
@SpinnerX
refactor: migrated some module units for math to be imported as modul…
39356bc 
…e units
@SpinnerX
refactor: added core editor module units to be represented as module …
ff6c6ce 
…partitions
@SpinnerX
refactor: removed some unused vulkan module imports
4b067dc
@SpinnerX
refactor: added extensive documentation comments and fixed some redun…
922b152 
…dant parameter specification types; include some additional module partition organization
@SpinnerX
refactor: removed vulkan::shader_resource_group
e1b1952
@SpinnerX SpinnerX self-assigned this Jun 14, 2026
@SpinnerX SpinnerX added the 📐design Define public API interfaces label Jun 14, 2026
@SpinnerX SpinnerX linked an issue Jun 14, 2026 that may be closed by this pull request
Migrate core renderer to using V6 of vulkan-cpp #202
Closed
@SpinnerX SpinnerX added the 💠core Core architectural task with system-wide impacts. label Jun 14, 2026
@SpinnerX SpinnerX changed the title Migrating Core Renderer to using vulkan-cpp V6 ✨ ✨Migrating Core Renderer to using vulkan-cpp V6 Jun 14, 2026
SpinnerX
SpinnerX commented Jun 15, 2026
View reviewed changes

@SpinnerX SpinnerX left a comment

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Merging this PR.

@SpinnerX SpinnerX merged commit e052f18 into main Jun 15, 2026
4 checks passed
@SpinnerX SpinnerX deleted the vulkan_v6 branch June 15, 2026 00:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

@SpinnerX SpinnerX

Labels

💠core Core architectural task with system-wide impacts. 📐design Define public API interfaces

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Migrate core renderer to using V6 of vulkan-cpp

1 participant

@SpinnerX