Skip to content

add freebayes functionality#2

Merged
lnblum merged 1 commit into
mainfrom
freebayes
Apr 18, 2025
Merged

add freebayes functionality#2
lnblum merged 1 commit into
mainfrom
freebayes

Conversation

@lnblum
Copy link
Copy Markdown
Collaborator

@lnblum lnblum commented Apr 17, 2025

No description provided.

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

a future update could include running freebayes on only the target regions but if I recall we had some issues scaling that up and ended up running it on everything then filtering after.

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

going to default to outputting to the process name, so strelka and freebayes go to separate folders

Comment thread nextflow.config
tracedir = "${params.outdir}/pipeline_info"
run_strelka = true
run_freebayes = true
freebayes_qual_filter = 15 // default quality filter for freebayes output.
Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

freebayes didn't have a standard quality filtering like strelka, so we used a basic quality filter to discard lots of false positive variants before running the concordance analysis.

@lnblum lnblum merged commit c26139a into main Apr 18, 2025
1 check passed
@lnblum lnblum deleted the freebayes branch April 18, 2025 15:19
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