Skip to content

Commit

Permalink
No need to run gsutil inside the job (#770)
Browse files Browse the repository at this point in the history
  • Loading branch information
MattWellie committed May 25, 2024
1 parent ce03f06 commit 6ce1d28
Show file tree
Hide file tree
Showing 4 changed files with 7 additions and 5 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.24.11
current_version = 1.24.12
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.24.11
VERSION: 1.24.12

jobs:
docker:
Expand Down
6 changes: 4 additions & 2 deletions cpg_workflows/jobs/gcnv.py
Original file line number Diff line number Diff line change
Expand Up @@ -322,9 +322,11 @@ def postprocess_calls(
model_shard_args = ''
calls_shard_args = ''

# forced ordering here just in case
# import with hail batch instead, then unpack in the container. No need to depend on a gcloud/gsutil install
# https://batch.hail.populationgenomics.org.au/batches/454143/jobs/1
for name, path in [(shard, shard_paths[shard]) for shard in shard_basenames()]:
gcp_related_commands.append(f'gsutil cat {path} | tar -xz -C $BATCH_TMPDIR/inputs')
shard_tar = get_batch().read_input(str(path))
gcp_related_commands.append(f'tar -xzf {shard_tar} -C $BATCH_TMPDIR/inputs')
model_shard_args += f' --model-shard-path $BATCH_TMPDIR/inputs/{name}-model'
calls_shard_args += f' --calls-shard-path $BATCH_TMPDIR/inputs/{name}-calls'

Expand Down
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.24.11',
version='1.24.12',
description='CPG workflows for Hail Batch',
long_description=open('README.md').read(),
long_description_content_type='text/markdown',
Expand Down

0 comments on commit 6ce1d28

Please sign in to comment.