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

Allow proper addition of real-valued noise to autocorrelations #213

Open
jsdillon opened this issue Feb 18, 2022 · 2 comments
Open

Allow proper addition of real-valued noise to autocorrelations #213

jsdillon opened this issue Feb 18, 2022 · 2 comments
Assignees

Comments

@jsdillon
Copy link
Member

As of #212, adding thermal noise will increase autocorrelations to account for the noise bias from the receiver temperature. However, it does not add a random component on top of that, since the SNR on autocorrelations is so high that this rarely matters. That said, we should also add thermal noise to the autocorrelations (in addition to the receiver noise bias).

@jsdillon
Copy link
Member Author

Adding a note that @aewallwi shared some math relevant to this problem over in #212.

@aewallwi
Copy link
Collaborator

aewallwi commented Feb 21, 2022

While we are on the topic of noise in autocorrelations, I noticed that gen_white_noise method, which generates gaussian noise, is used to get noise fluctuations regardless of whether we are dealing with cross or autocorrelations.

def gen_white_noise(size: Union[int, Tuple[int]] = 1) -> np.ndarray:

The thermal noise on autocorrelation should technically be drawn from a Gamma distribution rather than a normal distribution. Is this done in hera_sim?

image

I guess this Gamma distribution converges to a normal through the central limit theorem (N = 1 million) so a gaussian is an excellent approximation but maybe we should note this somewhere in a docstring.

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

3 participants