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.

Sign up for GitHub

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

Jump to bottom

Add "cutout objects" (ellipsoid/box) feature #31

Merged
merged 12 commits into from
Oct 13, 2023
Merged

Add "cutout objects" (ellipsoid/box) feature #31

merged 12 commits into from
Oct 13, 2023

Conversation

aras-p
Copy link
Owner

@aras-p aras-p commented Oct 13, 2023

Continuation of what @hybridherbst started in #24:

  • Ability to add "Cutout" objects in either ellipsoid or box shape to the GaussianSplatRenderer. When multiple cutouts are present, then a splat is left if it's inside any cutout.
  • When exporting the modified splat file, both your manual splat deletions and cutouts are taken into account.

Clipboard01

hybridherbst and others added 12 commits October 9, 2023 08:05
@hybridherbst
ellipsoid cutout
d6c9ad8
@hybridherbst
move ellipsoid cutout to compute shader
f57f218
@aras-p
Improve renderer editor (resources section folded by default, asset w…
aac79cb 
…arning displayed under asset field)
@aras-p
Multiple cutouts, each being either an ellipsoid or a box
a37ca4b
@aras-p
Cutouts and deleted splats can be either hidden or faded
3b87805 
Cutout splats are excluded from drag-selection
While editing splats, cutout gizmos are displayed
@aras-p
Helper buttons to create cutouts etc.
204b58b
@aras-p
Ignore cutout splats when exporting
5ba693e
@aras-p
Nullref safety
07f6715
@aras-p
Merge branch 'main' into feature/ellipsoid-cutout
9b0f2ef
@aras-p
Nullref safety
c5d56a5
@aras-p
Tweak cutout gizmo colors
3c7755d
@aras-p
Tweaking cutouts workflow:
c3dbeb3 
- Remove "faded" display mode; with always visible cutout gizmos it's not that useful
- Make export PLY button available when any cutouts are active
- Select cutout created after Add Cutout button
-
@aras-p aras-p mentioned this pull request Oct 13, 2023
Closed
@aras-p aras-p merged commit 9ab4b4f into main Oct 13, 2023
dylanebert pushed a commit to dylanebert/UnityGaussianSplatting that referenced this pull request Oct 13, 2023
@aras-p
Merge pull request aras-p#31 from aras-p/feature/ellipsoid-cutout
a2d8f9c 
Add "cutout objects" (ellipsoid/box) feature
@aras-p aras-p deleted the feature/ellipsoid-cutout branch October 17, 2023 11:02
HIHHIYAYAYOO pushed a commit to HIHHIYAYAYOO/UnityGaussianSplatting that referenced this pull request Nov 30, 2023
@aras-p
Merge pull request aras-p#31 from aras-p/feature/ellipsoid-cutout
cfdc14f 
Add "cutout objects" (ellipsoid/box) feature
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@aras-p @hybridherbst