Conversation
AlanSalcedo
left a comment
There was a problem hiding this comment.
Looks good, pending determining what n to use in the normalization of effective height.
|
Can we get some plots of the effective height at a couple different zenith angles comparing the measured to the shifted, just to sanity check that all looks as expected? |
clark2668
left a comment
There was a problem hiding this comment.
Some relatively minor comments.
There was a problem hiding this comment.
For example, a plot of the heff for this event that gets tipped over into passing might be a good one.
There was a problem hiding this comment.
Let me see if I can manage a plot at all first. This isn't easy to get out of AraSim.
|
That plot is helpful to see. So it does shift down as expected. And goes up. So this is roughly what we would have anticipated. Okay, thanks. |
|
Okay @clark2668 I'm rerunning the veff tests, but I think I've responded to all your comments. |
|
Okie dokes! Thanks for suffering my review! |
|
Okay all tests have passed. Merging now. |

This PR adds better handling of the translation of antenna gain measurements taken in one medium into another (probably something icy??).
If an antenna gain file does not have an index of refraction, no scaling of the gain argument occurs.
This PR also removes dependence on a hardcoded
freq_step_maxwhich sets the frequency resolution. Resolution is now determined dynamically (and so a lot of variables have been updated from arrays to vectors).This change resulted in a change to the Veff tests, since the normalization of the effective height changed (due to the index in its calculation changed from
n_localton_eff) and because the frequency resolution of the antenna models changed.Testing summary (as of commit 313ad80)
A3 veff test: passed ✅
birefringence veff test: passed ✅
pa veff test: passed ✅
pulser veff test: passed ✅