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

fix bug in M2 engine active region code #6908

Merged
merged 2 commits into from
Jan 28, 2021
Merged

fix bug in M2 engine active region code #6908

merged 2 commits into from
Jan 28, 2021

Conversation

davidbenjamin
Copy link
Contributor

Closes #6901. @fleharty

@gatk-bot
Copy link

gatk-bot commented Oct 23, 2020 •
edited
Loading

Travis reported job failures from build 31910
Failures in the following jobs:

Test Type JDK Job ID Logs
cloud openjdk8 31910.1 logs
cloud openjdk11 31910.14 logs

@fleharty
Copy link
Contributor

@davidbenjamin Can we get a regression test for this?

@fleharty
Copy link
Contributor

@davidbenjamin Pinging on this.

@droazen droazen changed the title fix typo in M2 engine active region code fix bug in M2 engine active region code Nov 13, 2020
@davidbenjamin
Copy link
Contributor Author

@fleharty test added.

@gatk-bot
Copy link

Travis reported job failures from build 32192
Failures in the following jobs:

Test Type JDK Job ID Logs
cloud openjdk8 32192.1 logs

fleharty
fleharty approved these changes Jan 28, 2021
View reviewed changes
Copy link
Contributor

@fleharty fleharty left a comment

Choose a reason for hiding this comment

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

Looks great, thanks for adding the test!
Merge away.

@davidbenjamin davidbenjamin merged commit a29e8a0 into master Jan 28, 2021
@davidbenjamin davidbenjamin deleted the db_m2_engine_typo branch January 28, 2021 19:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@fleharty fleharty fleharty approved these changes

Assignees

@fleharty fleharty

Labels
Mutect
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Possible bug in Mutect2Engine.java
3 participants
@davidbenjamin @gatk-bot @fleharty