-
Notifications
You must be signed in to change notification settings - Fork 152
Native GS #1576
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
Native GS #1576
Conversation
…nto NativeGS # Conflicts: # Apps/package-lock.json # Apps/package.json
…nto NativeGS # Conflicts: # Apps/package-lock.json
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 adds native support for Gaussian Splatting (GS) rendering by introducing a sorting function for splats, shader attribute mappings for splat data, and texture format support for RG16F.
Changes:
- Added
_SortGSfunction to sort Gaussian splats by depth for proper rendering order - Added shader attribute mappings for
splatIndex0-3vertex attributes - Added RG16F texture format support for splat data textures
Reviewed changes
Copilot reviewed 4 out of 5 changed files in this pull request and generated 7 comments.
| File | Description |
|---|---|
| Plugins/NativeOptimizations/Source/NativeOptimizations.cpp | Adds _SortGS function and sortSplatQSort comparator to sort splats by depth based on view matrix |
| Plugins/NativeEngine/Source/ShaderCompilerTraversers.cpp | Adds attribute mappings for splatIndex0-3 to support splat vertex data |
| Plugins/NativeEngine/Source/NativeEngine.cpp | Adds RG16F texture format support and applies sampler flip processing to vertex shader coordinates |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: Copilot <[email protected]>
SergioRZMasson
left a comment
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.
LGTM
No description provided.