feature: NVRHI

[sheazywi]

No description provided.

[Copilot]

Pull Request Overview

Integrate the NVRHI rendering abstraction into the build system and update the ImGui layer to target a newer GLSL version.

• Add NVRHI as a submodule and include path.
• Update premake scripts to include and link NVRHI for both engine and editor.
• Bump ImGui OpenGL backend shader version from 410 to 450.

Reviewed Changes

Copilot reviewed 8 out of 8 changed files in this pull request and generated 1 comment.

Show a summary per file

File	Description
premake5.lua	Register new “Dependencies – NVRHI” group
StarEngine/src/StarEngine/ImGui/ImGuiLayer.cpp	Updated ImGui OpenGL GLSL version to 450
StarEngine/premake5.lua	Added NVRHI include dir and linked libraries
StarEditor/premake5.lua	Added NVRHI include directory for the editor
Dependencies.lua	Added IncludeDir["nvrhi"] path
.gitmodules	Added NVRHI submodule entry

Comments suppressed due to low confidence (2)

StarEditor/premake5.lua:26

• The editor project includes the NVRHI header path but does not link the NVRHI library. You should add "nvrhi" to the links block so the editor can resolve its symbols.

%{IncludeDir.nvrhi},

premake5.lua:37

• [nitpick] The indentation for this include line doesn’t match the single-tab style used elsewhere. Align it with the other dependency entries for consistency.

include "StarEngine/vendor/nvrhi"

[Copilot]

[nitpick] Hardcoding "#version 450" enforces a minimum OpenGL 4.5 context and may break compatibility on older hardware. Consider making the GLSL version configurable or document this requirement.

Suggested change

	ImGui_ImplOpenGL3_Init("#version 450");
	const char* glslVersion = "#version 450"; // Default GLSL version
	// Optionally, retrieve GLSL version dynamically or make it configurable
	ImGui_ImplOpenGL3_Init(glslVersion);