Managing gaussians.binding_counter in the function densify_and_split
seungjun-moon opened this issue · comments
First of all, thank you for providing awesome results of your research.
I have a question about the function densify_and_clone
in scene/gaussian_model
.
Since we add the number of connections properly in line 466, I think the adding function at line 465 is redundant.
When I tried with my toy dataset, it occurs an undesirable discrepancy between the number of self._xyz
and the sum of self.binding_counter
.
When I wiped out line 465, it worked well.
If I missed something, please let me know.
Thank you.
Hi Seungjun, thanks for reporting this bug!
You are right, we should have removed that line.
We have fixed this bug in the latest commit.