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.

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

Develop #12

Open
wants to merge 7 commits into
base: master
Choose a base branch
from
Open

Develop #12

wants to merge 7 commits into from

Conversation

Redmar-van-den-Berg
Copy link
Contributor

Checklist

  • Pull request details were added to CHANGELOG.md.
  • New tests have been added to the matrix section of the
    .github/workflows/ci.yml file.

Redmar-van-den-Berg and others added 7 commits March 17, 2021 14:01
Previously, multi sample calling was done on the merged per sample g.vcf
files. However, in rare cases, VCF files that are merged by bcftools can
trigger a crash in GATK.

To resolve this, the multi sample calling now uses the scattered per
sample g.vcf files, so that GATK does not have to act on VCF files that
were modified by bcftools.

As an added advantage, using the scatters directly for the multi sample
calling allows for better parallelisation.
Docker hub has started to remove unused images from free accounts, which
means that it might remove images used by this pipeline without notice.
Therefore the pipeline now exclusively uses images from quay.io or
official repositories from docker hub, which do not have this
limitation.
* Use tmp on shared filesystem for aling rule

* Group up tests to reduce load on test runners

* Only test small-scatter on github

* Check size of created files on failure

* Reduce the number of test scatters

* Add new entry to changelog

* Pin Snakemake version
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