Skip to content

Increase time allocation for pipeline attempts - #55

Closed
tbrown91 wants to merge 1 commit into
sanger-tol:devfrom
tbrown91:conf_upate
Closed

Increase time allocation for pipeline attempts#55
tbrown91 wants to merge 1 commit into
sanger-tol:devfrom
tbrown91:conf_upate

Conversation

@tbrown91

@tbrown91 tbrown91 commented Aug 3, 2026

Copy link
Copy Markdown

For large genomes (>20 GB), this step was failing due to timeout error

PR checklist

  • This comment contains a description of changes (with reason).
  • If you've fixed a bug or added code that should be tested, add tests!
  • If you've added a new tool - have you followed the pipeline conventions in the contribution docs
  • Make sure your code lints (nf-core pipelines lint).
  • Ensure the test suite passes (nextflow run . -profile test,docker --outdir <OUTDIR>).
  • Check for unexpected warnings in debug mode (nextflow run . -profile debug,test,docker --outdir <OUTDIR>).
  • Usage Documentation in docs/usage.md is updated.
  • Output Documentation in docs/output.md is updated.
  • CHANGELOG.md is updated.
  • README.md is updated (including new tool citations and authors/contributors).

For large genomes, this step was failing due to timeout error

@sainsachiko sainsachiko left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

LGTM!

@tbrown91

tbrown91 commented Aug 3, 2026

Copy link
Copy Markdown
Author

Thanks @sainsachiko can you guide me through pushing this to a release, or is this someone else's responsibility?

@sainsachiko

sainsachiko commented Aug 3, 2026

Copy link
Copy Markdown

Thanks @sainsachiko can you guide me through pushing this to a release, or is this someone else's responsibility?

Hi @tbrown91 , if you want to release this as a patch (skip features under dev branch), please merge this change directly to main branch. This can be done by renaming your branch into patch, rebase, and create a PR target main branch.
On that patch branch, please ensure that CHANGELOG, pipeline versions (this might need to be changed in nf-test.snap file also, as the snapshots capture pipeline version), release date, ro-crate-metadata.json have been updated. Once that patch merged to main, a release can be created from main.

Guideline for a pipeline release can be found here. (Zenodo will be auto updated, so we basically just need to focus on what I have listed).

@tbrown91

tbrown91 commented Aug 3, 2026

Copy link
Copy Markdown
Author

Opened a patch PR here: #56

@tbrown91 tbrown91 closed this Aug 3, 2026
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