Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Allow auto and/or cross only files #368

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

aewallwi
Copy link
Contributor

@aewallwi aewallwi commented Aug 26, 2020

provide a data file with only cross or only autos currently breaks xrfi_run. This PR fixes this.

@aewallwi aewallwi requested a review from jsdillon August 26, 2020 04:55
@codecov
Copy link

codecov bot commented Aug 26, 2020

Codecov Report

Merging #368 (cea6f70) into master (5a92245) will increase coverage by 0.00%.
The diff coverage is 100.00%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #368   +/-   ##
=======================================
  Coverage   97.12%   97.13%           
=======================================
  Files          10       10           
  Lines        3274     3277    +3     
=======================================
+ Hits         3180     3183    +3     
  Misses         94       94           
Impacted Files Coverage Δ
hera_qm/xrfi.py 99.79% <100.00%> (+<0.01%) ⬆️

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 5a92245...cea6f70. Read the comment docs.

@aewallwi aewallwi force-pushed the allow_auto_cross_only_files branch from f771554 to aad10e9 Compare October 3, 2020 03:07
@aewallwi aewallwi force-pushed the allow_auto_cross_only_files branch from aad10e9 to cea6f70 Compare February 28, 2021 04:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant