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

restrict_xyz #18

Open
Yzichen opened this issue Sep 12, 2024 · 1 comment
Open

restrict_xyz #18

Yzichen opened this issue Sep 12, 2024 · 1 comment

Comments

@Yzichen
Copy link

Yzichen commented Sep 12, 2024

if restrict_xyz:
assert unit_xyz is not None
unit_prob = [unit_xyz[i] / (pc_range[i + 3] - pc_range[i]) for i in range(3)]
unit_sigmoid = [4 * unit_prob[i] for i in range(3)]
if phi_activation == 'loop':
unit_sigmoid[2] = unit_prob[2]
self.unit_sigmoid = unit_sigmoid

Is your intention to limit the offset of gs to [2.0, 2.0, 0.5]? But why calculate the unit_sigmoid in this way? Where does the 4 come from?

@huang-yh
Copy link
Owner

Sorry for the confusion.

  1. Yes, the optimization of the xyzs of Gaussians would be very unstable otherwise.
  2. That is a heuristic design given that the maximum derivative of the sigmoid function is 1/4.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants