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

After running post_splatam_opt.py for densification, the newly generated npz file has worse performance compared to splatam.py #93

Open
LemonSoda-RPG opened this issue Mar 12, 2024 · 4 comments
Assignees
Labels
bug Something isn't working

Comments

@LemonSoda-RPG
Copy link

First of all, thank you very much for your work.
I followed the steps in the readme file and ran the following code step by step, and I used the replica dataset.

  1. python scripts/splatam.py configs/replica/splatam.py
  2. python scripts/post_splatam_opt.py configs/replica/post_splatam_opt.py

This image is the result after running only splatam.py.
image.png
This image is the result after running post_splatam_opt.py following the execution of splatam.py.
image.png

I attempted to disable densification in the configuration file of post_splatam_opt, but the results remained the same.

@Nik-V9
Copy link
Contributor

Nik-V9 commented Mar 14, 2024

Hi, definitely something seems wrong here. @JayKarhade will get back to you.

@Nik-V9 Nik-V9 added the bug Something isn't working label Mar 14, 2024
@JayKarhade
Copy link
Collaborator

Hi @LemonSoda-RPG i got some time to check this - it seems to work fine on my end.

Could you share your config files and post_splatam_opt.py file? I'd like to check if there is any mismatch.

@LemonSoda-RPG
Copy link
Author

Hi @LemonSoda-RPG i got some time to check this - it seems to work fine on my end.

Could you share your config files and post_splatam_opt.py file? I'd like to check if there is any mismatch.

Thank you very much for your reply!
To verify again, I used the latest version of the code, the replica dataset, and executed the following command again.

Compared with the latest version of the code in the github repository, I only made two modifications. When running splatam, I set in the configuration file to use the real pose, not track, and closed wandb. Other than that, there is no Make any changes.

  1. python scripts/splatam.py configs/replica/splatam.py
  2. python scripts/post_splatam_opt.py configs/replica/post_splatam_opt.py

But after running post_splatam_opt.py, I still got worse results.

@Nik-V9
Copy link
Contributor

Nik-V9 commented Jun 8, 2024

Hi, can you please share the complete config files so that we can try running this on our end?

I'm not sure why using ground truth poses would cause this bug. Is the use_gt_poses config set consistently across both files? Do you still see the bug when you try the default configs?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants