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

Add station and elevation dependence for coherence time #21

Open
dpesce opened this issue Feb 26, 2024 · 0 comments
Open

Add station and elevation dependence for coherence time #21

dpesce opened this issue Feb 26, 2024 · 0 comments

Comments

@dpesce
Copy link
Collaborator

dpesce commented Feb 26, 2024

Currently the atmospheric coherence time is assumed to be constant across the entire array, and it also does not vary with source elevation. This issue stems at least in part from an ambiguity between "coherence time" and "SNR threshold integration time," which are essentially conflated right now in the code.

An elevation dependence along the lines of that used by the ARIS (Asaki et al. 2007) or MeqSilhouette (Natarajan et al. 2022) softwares would be straightforward to implement, but would involve re-interpreting the user-specified timescale as some sort of coherence time. Because that would break backwards compatibility, a full addressing of this issue will probably require incrementing the major version number of the codebase.

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