search

HERA-Team / hera_cal

Library for HERA data reduction, including redundant calibration, absolute calibration, and LST-binning.
MIT License
13 stars 8 forks source link

Allow an advanced user to speed-up `build_fringe_rate_profiles` #904

Closed jsdillon closed 1 year ago

jsdillon commented 1 year ago

This PR allows an advanced user to skip several time-consuming steps in frf.build_fringe_rate_profiles(). I find this particularly useful in the case I'm working on, where I want to compute the FR profile for a single baseline over many different bands. This allows me to speed up that calculation by a factor of ~10 by skipping certain steps that are unnecessary or doing them just once.

There is no change to the default behavior of the function.

jsdillon commented 1 year ago

@aewallwi Again, I'm tagging you so that you're aware of this... not necessarily to review.

codecov[bot] commented 1 year ago

Codecov Report

Patch coverage: 100.00% and no project coverage change.

Comparison is base (d174d3f) 97.15% compared to head (efd2008) 97.15%.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #904 +/- ## ======================================= Coverage 97.15% 97.15% ======================================= Files 23 23 Lines 10262 10271 +9 ======================================= + Hits 9970 9979 +9 Misses 292 292 ``` | [Impacted Files](https://app.codecov.io/gh/HERA-Team/hera_cal/pull/904?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=HERA-Team) | Coverage Δ | | |---|---|---| | [hera\_cal/frf.py](https://app.codecov.io/gh/HERA-Team/hera_cal/pull/904?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=HERA-Team#diff-aGVyYV9jYWwvZnJmLnB5) | `97.53% <100.00%> (+0.03%)` | :arrow_up: |

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.