Skip to content

Update nextflow script to new strict syntax and workflow ouput#3

Draft
dgoebel97 wants to merge 3 commits intogi-bielefeld:clowm-integrationfrom
dgoebel97:update-nextflow-syntax
Draft

Update nextflow script to new strict syntax and workflow ouput#3
dgoebel97 wants to merge 3 commits intogi-bielefeld:clowm-integrationfrom
dgoebel97:update-nextflow-syntax

Conversation

@dgoebel97
Copy link

  1. Update nextflow script to new strict syntax: https://www.nextflow.io/docs/stable/strict-syntax.html
  2. Update nextflow script to new workflow output definition: https://www.nextflow.io/docs/stable/tutorials/workflow-outputs.html
  3. Update python docker containers to 3.14-slim
  4. Update minimum Nextflow version to 25.10
  5. Add meta-data of SANS to nextflow.config

@rwittler Please check if the new information in nextflow.config are correct and if any major contributors are missing

@dgoebel97 dgoebel97 marked this pull request as draft January 12, 2026 12:33
@dgoebel97
Copy link
Author

Apparently, if I use a S3 path as param outdir, the results are NOT uploaded

@dgoebel97
Copy link
Author

Apparently, if I use a S3 path as param outdir, the results are NOT uploaded

The problem is the NFS. If the work directory is on the NFS, then the files will be not uploaded. Creating an Index files solves this issue, since the Nextflow process has to read all output files before uploading them.

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