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

Clarification on the computation of m_{bind} #1

Open
niqodea opened this issue Oct 30, 2024 · 0 comments
Open

Clarification on the computation of m_{bind} #1

niqodea opened this issue Oct 30, 2024 · 0 comments

Comments

@niqodea
Copy link

niqodea commented Oct 30, 2024

Hello! Thank you for releasing the code for this fascinating paper. After reviewing the code, I have a question regarding the computation of m_{bind}.

In the paper on arXiv (link to paper), m_{bind} is described as the average of two terms—one with an added bias term (\psi) and one without (\chi). This struck me as unusual, so I checked the code and noticed that neither of the averaged terms appears to include any bias.

Specifically, I observed that the variable magnitude_psi is computed from psi before adding the rotational bias to produce z (which I believe corresponds to \phi in the paper). This suggests a potential inconsistency between the code and the paper unless it’s confirmed that adding the rotational bias does not impact the magnitude of \psi (which seems unlikely).

Could it be that the paper might need to redefine \psi to exclude the rotational bias term, or is there another interpretation I'm missing? Thanks for any clarification!

@niqodea niqodea changed the title Clarification on the computation of m_bind Clarification on the computation of m_{bind} Oct 30, 2024
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

1 participant