Closed jsdillon closed 1 year ago
Base: 97.29% // Head: 97.15% // Decreases project coverage by -0.14%
:warning:
Coverage data is based on head (
da49a16
) compared to base (8d7d090
). Patch coverage: 100.00% of modified lines in pull request are covered.:exclamation: Current head da49a16 differs from pull request most recent head c0ba658. Consider uploading reports for the commit c0ba658 to get more accurate results
:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.
This PR speeds up expand_omni_sol by avoiding (if possible) a chi^2 calculation that's unnecessary under normal redcal operation (for example, when not using a min_bl_cut or max_bl_cut). It also adds a dictionary to a RedundantCalibrator object that prevents a lot of unnecessarily repeated list comprehension.
Here's the before:
(This takes 31.4 s on my laptop when not running inside snakeviz.)
Here's the after:
(This takes 13.2 s on my laptop when not running inside snakeviz.)