-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Refactor splatfacto gaussian params with a ParameterDict #2946
Conversation
The change looks good to me. Some suggestions: (1) test PLY exporter end-to-end to safeguard the workflow is fine. |
One more thing: when rebasing on top of #2945 We need to change the parameter names for splatfacto-big. |
This should be ready to merge, fixed all the mentioned changes and also added loading logic to make it backwards-compatible with old checkpoints. |
LGTM! Can we merge this? |
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.
looks good + worked for me!
…project#2946) * refactor splatfacto with parameterdict to simplify subclassing, defining new gaussian features * stray comment * fix exp_avg key error * backwards compatibility for checkpoint * backwards comp fix --------- Co-authored-by: Matias Turkulainen <30566358+maturk@users.noreply.github.com>
This simplifies subclassing, and simplifies some of the logic for culling/splitting.