Skip to content
This repository has been archived by the owner on Jun 20, 2024. It is now read-only.

Expose INCLUDE_SECONDARY_ALIGNMENTS for MergeBamAlignment task. #381

Merged
merged 1 commit into from
Jul 29, 2020

Conversation

jacarey
Copy link
Contributor

@jacarey jacarey commented Jul 29, 2020

This currently defaults to true, but because of the following bug:
broadinstitute/picard#932

Exposing this flag allows the secondary alignments to not be output and not fail while trying to calculate MD and NM flags.

@codecov-commenter
Copy link

codecov-commenter commented Jul 29, 2020

Codecov Report

Merging #381 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #381   +/-   ##
=======================================
  Coverage   91.47%   91.47%           
=======================================
  Files          31       31           
  Lines        1185     1185           
  Branches       80       80           
=======================================
  Hits         1084     1084           
  Misses        101      101           

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 a329df7...2cd82fa. Read the comment docs.

@jacarey jacarey merged commit 7c31adf into master Jul 29, 2020
@jacarey jacarey deleted the jc_remove_secondary_alignments branch July 29, 2020 18:28
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants