Skip to content

Commit

Permalink
Merge pull request #654 from populationgenomics/reduce_seqr_load_stor…
Browse files Browse the repository at this point in the history
…age_use

Reduce requested storage for seqr_loader jobs based on workflow sequencing type
  • Loading branch information
EddieLF authored Mar 18, 2024
2 parents 99d4c39 + 29c8bc6 commit 4dc7bd6
Show file tree
Hide file tree
Showing 4 changed files with 11 additions and 7 deletions.
2 changes: 1 addition & 1 deletion .bumpversion.cfg
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
[bumpversion]
current_version = 1.21.1
current_version = 1.21.2
commit = True
tag = False

Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/docker.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,7 @@ permissions:
contents: read

env:
VERSION: 1.21.1
VERSION: 1.21.2

jobs:
docker:
Expand Down
12 changes: 8 additions & 4 deletions cpg_workflows/resources.py
Original file line number Diff line number Diff line change
Expand Up @@ -338,13 +338,17 @@ def joint_calling_scatter_count(sequencing_group_count: int) -> int:

def storage_for_joint_vcf(
sequencing_group_count: int | None, site_only: bool = True
) -> int | None:
) -> float | None:
"""
Storage enough to fit and process a joint-called VCF
"""
if not sequencing_group_count:
return None
gb_per_sequencing_group = 1
if not site_only:
gb_per_sequencing_group = 4
if get_config()['workflow']['sequencing_type'] == 'exome':
gb_per_sequencing_group = 0.1
else:
gb_per_sequencing_group = 1.0
if not site_only:
gb_per_sequencing_group = 1.5

return gb_per_sequencing_group * sequencing_group_count
2 changes: 1 addition & 1 deletion setup.py
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@
setup(
name='cpg-workflows',
# This tag is automatically updated by bumpversion
version='1.21.1',
version='1.21.2',
description='CPG workflows for Hail Batch',
long_description=open('README.md').read(),
long_description_content_type='text/markdown',
Expand Down

0 comments on commit 4dc7bd6

Please sign in to comment.