Skip to content

adding nextflow w batch sample - #159

Merged
thomasnyc merged 4 commits into
mainfrom
nextflow-batch-sample
Aug 31, 2026
Merged

adding nextflow w batch sample#159
thomasnyc merged 4 commits into
mainfrom
nextflow-batch-sample

Conversation

@thomasnyc

Copy link
Copy Markdown
Collaborator

adding the nextflow batch sample .

Comment thread hcls/nextflow-batch/nextflow.config Outdated
Comment thread hcls/nextflow-batch/nextflow.config Outdated
Comment thread hcls/nextflow-batch/nextflow.config Outdated
Comment thread hcls/nextflow-batch/nextflow.config Outdated
*/
workDir = 'gs://<bucketname>/scratch'
google.location = 'us-central1'
google.project = '<GCP project name>'

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is there a better way to pass these variables? Manually editing the config file is a bit cumbersome. How about creating a template compatible with envsubst?

// nextflow.config.template
profiles {
'google-batch' {
params.transcriptome = 'gs://rnaseq-nf/data/ggal/transcript.fa'
params.reads = 'gs://rnaseq-nf/data/ggal/gut_{1,2}.fq'
params.multiqc = 'gs://rnaseq-nf/multiqc'
process.executor = 'google-batch'
process.container = 'docker.io/nextflow/rnaseq-nf:v1.3.1'

    workDir = 'gs://${GCP_BUCKET_NAME}/scratch'
    google.location = '${GCP_REGION}'
    google.project = '${GCP_PROJECT_ID}'
    google.batch.allowedLocations = ['regions/${GCP_REGION}']

    // Network configuration
    google.batch.network = 'global/networks/${GCP_NETWORK_NAME}'
    google.batch.subnetwork = 'regions/${GCP_REGION}/subnetworks/${GCP_SUBNET_NAME}'
}

}

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I updated the nextflow.config and also README.md Please review.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You don't want to do the envsubst thing?

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not planning to add since there is only one file to modify.

@jrossthomson jrossthomson self-assigned this Aug 28, 2026
@thomasnyc
thomasnyc merged commit a6673e9 into main Aug 31, 2026
8 checks passed
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.

2 participants