search

HERA-Team / hera-validation

Archive of formal software pipeline validation tests
http://hera.pbworks.com/w/page/130621356/Validation
0 stars 3 forks source link

Step 0.1.1: Imaginary power #17

Closed r-pascua closed 5 years ago

r-pascua commented 5 years ago

0.1.1 -- Flat P(k) Analysis: Imaginary Power from Time-Offset Visibilities

When computing power spectra from time-offset visibilities, imaginary power is introduced to the spectrum due to the breaking of baseline-redundancy from Earth rotation. This notebook includes a derivation of the expected imaginary power and shows that the output from the hera_pspec.pspecdata pipeline agrees to about 1.5%. The distribution of imaginary power is substantially different from the real power and its expected form remains an open question.

Contributors:

Some questions

Steps Forward

A suggested list of condensed automatic tests arising from this PR, and relevant repos to include them in:

A suggested list of follow-up validation tests which may not already appear on the validation path:

r-pascua commented 5 years ago

Do the equation numbers work for you? I've found that GIthub doesn't mesh well with the equation numbering plugin, and it just shows every equation as (1) when I view it (even though the equations are correctly numbered in the original notebook).

steven-murray commented 5 years ago

For me all the equations are showing up as (1) as well (firefox). I've noticed several weird things about viewing notebooks on github over the years, I guess this is just another. Not sure we can do much about it. Perhaps I'll make an issue to remind us of it, if anyone has time to solve it!

r-pascua commented 5 years ago

@acliu do the equations work for you? I can hardcode the equation number instead of referencing it the way I would in a document I've typeset (~\ref{...}), but it doesn't seem like there's much point to do so if the equation numbers aren't even correct.

acliu commented 5 years ago

It's fine; just leave things as they are. I just thought it was trivial to fix, but it's not a big deal. I'd go ahead and merge when you're ready.