Skip to content
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

Gaussian Splats 0.1 #4991

Closed
wants to merge 7 commits into from
Closed

Gaussian Splats 0.1 #4991

wants to merge 7 commits into from

Conversation

emilk
Copy link
Member

@emilk emilk commented Jan 31, 2024

What

MVP for gaussian splats.

Can load .ply files as found on https://poly.cam/tools/gaussian-splatting and display them without the gaussian transparency:

Screenshot 2024-01-31 at 16 32 48

Implementation

This adds a scale and rotation to points, which are then used to generate splats in the vertex shader (where the 3x3 covariance matrix is calculated)

Blockers for merging

  • Use the same Points archetype
  • HalfSize is not the right component for splat scale
    • Maybe encode splat size as a GaussianStdDev3 component or something?
  • Picking for splats
  • Separate PointSplatBuilder
  • Separate point_splat.wgsl
  • Decide on point_cloud.rs - split, or reuse?

Shortcomings

  • No transparency sorting
  • No actual gaussian blur
  • No support for view-dependent color using Spherical Harmonics
  • Gaussians must be decomposed in rotation+scale (no 3x3 covariance support)
  • Uses a lot of VRAM
  • The .ply loader is really slow

Checklist

  • I have read and agree to Contributor Guide and the Code of Conduct
  • I've included a screenshot or gif (if applicable)
  • I have tested the web demo (if applicable):
  • The PR title and labels are set such as to maximize their usefulness for the next release's CHANGELOG

@hardikdava
Copy link

Interesting work! looking forward to it!

@emilk
Copy link
Member Author

emilk commented Aug 8, 2024

Sadly this won't happen anytime soon

@emilk emilk closed this Aug 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants