Skip to content

Fixes missing BAM channel creation when trimBam is on but PMD is off#330

Merged
apeltzer merged 15 commits intonf-core:devfrom
jfy133:fix_trimbamlogic
Jan 26, 2020
Merged

Fixes missing BAM channel creation when trimBam is on but PMD is off#330
apeltzer merged 15 commits intonf-core:devfrom
jfy133:fix_trimbamlogic

Conversation

@jfy133
Copy link
Copy Markdown
Member

@jfy133 jfy133 commented Jan 10, 2020

This bugfix adds a missing logic for creating a BAM channel for genotyping, when turning on trimbam but not pmdtools at the same time.

Otherwise, when genotyping was not requested, but trimbam was requested a missing ch_XXX_for_unifiedgenotyper error was reported,

PR checklist

  • This comment contains a description of changes (with reason)

Learn more about contributing: https://github.com/nf-core/eager/tree/master/.github/CONTRIBUTING.md

@jfy133 jfy133 self-assigned this Jan 10, 2020
@jfy133 jfy133 added this to the V2.1 "Ulm" milestone Jan 15, 2020
@jfy133 jfy133 requested a review from apeltzer January 15, 2020 16:30
@jfy133
Copy link
Copy Markdown
Member Author

jfy133 commented Jan 21, 2020

@apeltzer for a review!

@jfy133 jfy133 requested a review from maxibor January 21, 2020 07:21
@apeltzer apeltzer merged commit 87da681 into nf-core:dev Jan 26, 2020
@jfy133 jfy133 deleted the fix_trimbamlogic branch February 10, 2020 14:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants