search

HERA-Team / hera_qm

HERA Data Quality Metrics
MIT License
2 stars 2 forks source link

Add option to ignore cross-polarized visibilities in XRFI #406

Closed jsdillon closed 2 years ago

jsdillon commented 2 years ago

I've been investigating how to make XRFI run more quickly on site.

I was under the impression for a while that this was actually how XRFI worked when looking at only autos, though I just figured out that that's not the case.

This PR leaves the status quo as the default. Whether we want to use this option in future stage 2 XRFI is an open question.

codecov[bot] commented 2 years ago

Codecov Report

Merging #406 (092f647) into master (b559443) will increase coverage by 0.00%. The diff coverage is 100.00%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #406   +/-   ##
=======================================
  Coverage   97.01%   97.01%           
=======================================
  Files          10       10           
  Lines        3313     3317    +4     
=======================================
+ Hits         3214     3218    +4     
  Misses         99       99
Impacted Files Coverage Δ
hera_qm/utils.py 97.29% <100.00%> (+0.01%) :arrow_up:
hera_qm/xrfi.py 99.79% <100.00%> (+<0.01%) :arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update b559443...092f647. Read the comment docs.