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.

Sign up for GitHub

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

Jump to bottom

Unify M2 likelihoods #5775

Closed
davidbenjamin opened this issue Mar 8, 2019 · 0 comments · Fixed by #5814
Closed

Unify M2 likelihoods #5775

davidbenjamin opened this issue Mar 8, 2019 · 0 comments · Fixed by #5814
Assignees
@davidbenjamin
Labels
Mutect refactoring
Milestone
Mutect 3

Comments

@davidbenjamin
Copy link
Contributor

The quick somatic likelihoods model in Mutect2Engine.isActive is really just a version of the full somatic likelihoods model that stops after a single iteration. The code and mathematical notes should be amended to reflect this.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@davidbenjamin davidbenjamin

Labels
Mutect refactoring
Projects
None yet
Milestone
Mutect 3
Development

Successfully merging a pull request may close this issue.

Rewrote M2 isActive method as special case of somatic likelihoods model broadinstitute/gatk
1 participant
@davidbenjamin