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

report: Clarify loci filter precedence in QC_Report.docx (issue #288) #320

Merged
merged 1 commit into from
Sep 13, 2024

Conversation

jaamarks
Copy link
Collaborator

@jaamarks jaamarks commented Sep 6, 2024

PR #305 (issue #288) changed the order of call rate filters to apply loci filters first.
This PR adds context to those changes in the QC_Report.docx.

Old

A two-stage filter by completion rate threshold of 0.80 for samples and 0.80 for loci, followed by 0.95 for samples and 0.95 for loci was performed. The first stage excluded 9 samples, and 6,498 loci. The subsequent filter excluded 40 samples and 23,799 loci, resulting in 451 samples at 708,734 loci (Table 2).

New

A two-stage filter was performed, applying loci filters followed by sample filters at each stage. The initial stage utilized a completion rate threshold of 0.80 for loci and 0.80 for samples. This resulted in the exclusion of 6,498 loci and 9 samples. The subsequent filter excluded 23,799 loci and 40 samples resulting in 451 samples at 708,734 loci (Table 2).

@jaamarks jaamarks merged commit 3cfb8b3 into default Sep 13, 2024
2 checks passed
@jaamarks jaamarks deleted the issue-288-update-qc-report branch September 13, 2024 18:23
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