-
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
Get rid of redundant projection matrix computations in Splatfacto (needs a gsplat update) #3019
Get rid of redundant projection matrix computations in Splatfacto (needs a gsplat update) #3019
Conversation
0761483
to
6c68012
Compare
nerfstudio/models/splatfacto.py
Outdated
@@ -738,7 +714,7 @@ def get_outputs(self, camera: Cameras) -> Dict[str, Union[torch.Tensor, List]]: | |||
1, | |||
quats_crop / quats_crop.norm(dim=-1, keepdim=True), | |||
viewmat.squeeze()[:3, :], | |||
projmat.squeeze() @ viewmat.squeeze(), | |||
None, # Deprecated projmat |
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.
If nerfstudio-project/gsplat#149 gets merged then this needs to be updated, I think just changing the API all at once one time is better than 2 incremental changes
One more thing, the gsplat version should be bumped to |
Needs gsplat >= 0.1.9
6c68012
to
d9c2282
Compare
Done! |
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! will merge once gsplat is bumped
…eds a gsplat update) (nerfstudio-project#3019) Get rid of projection matrix in ProjectGaussians with gsplat >= 0.1.9
Requires nerfstudio-project/gsplat#97