-
Notifications
You must be signed in to change notification settings - Fork 872
Dev -> master for nf-core/rnaseq 3.23.0 #1717
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
Merged
Changes from all commits
Commits
Show all changes
394 commits
Select commit
Hold shift + click to select a range
3d55e3b
Adding tag to ignore GPU workflows on the CPU runner
gburnett-nvidia cb2a979
Update snaps
pinin4fjords ca2c728
Adding cpu tags to all other tests
gburnett-nvidia e00a783
docs: Add changelog entry for topic-based version reporting migration
pinin4fjords 49dfa8f
fix: Remove ifEmpty(null) from PREPARE_GENOME versions emit
pinin4fjords adfc874
update pipeline snaps
pinin4fjords 9421596
fix: Update tests and snapshots for topic-based version reporting
pinin4fjords 7a18fb4
fix: Remove trailing null from sentieon snapshots, add missing proces…
pinin4fjords c7bc3ea
fix: Update sentieon version 202503.01 -> 202503.02
pinin4fjords 278f9df
chore: Update picard ARM container to 3.4.0
pinin4fjords 70f9856
fix: Address PR review feedback for prokaryotic support
pinin4fjords c01d5f8
chore: Add Matthias Zepper to credits for PR review
pinin4fjords 715816b
style: Fix table formatting in usage.md
pinin4fjords 34599fc
Fix snap
pinin4fjords 5d7ba82
fix: Remove broken .out.versions references for topic-based modules
pinin4fjords 90f32d3
Merge pull request #1689 from nf-core/feature/modules-topics-update
pinin4fjords 2429747
Merge origin/dev into feature/prokaryotic-support
pinin4fjords fe3a04e
fix: Remove .out.versions calls for topic-based modules
pinin4fjords cf42ebd
fix: Add read group to Bowtie2 output for Picard MarkDuplicates
pinin4fjords 00d141e
test: Update prepare_genome snapshot for topic-based versions
pinin4fjords 28f1481
Update snap
pinin4fjords d20a221
Merge pull request #1688 from nf-core/feature/prokaryotic-support
pinin4fjords 091b1c1
refactor: Replace tag-based GPU/CPU test filtering with nf-test.confi…
pinin4fjords 5ed29f7
Merge remote-tracking branch 'origin/dev' into parabricks-unblock
pinin4fjords 1a68b3c
Centralise STAR alignment config with Parabricks support
pinin4fjords 63324bf
Rework Parabricks tests with smaller eukaryotic test data
pinin4fjords fb6cfe6
Add Parabricks GPU acceleration docs
pinin4fjords 3e4be3f
Update a couple of snaps
pinin4fjords 415bc94
fix: Inline variable declarations in nf-test.config for linter
pinin4fjords d36b186
Revert "fix: Inline variable declarations in nf-test.config for linter"
pinin4fjords d30f74f
fix: Exclude nf-test.config from Nextflow linter
pinin4fjords ae8a2f5
fix: Use correct -exclude flag for nextflow lint
pinin4fjords 17d7c7d
test: Revert pipeline-level parabricks test to eukaryotic data
pinin4fjords 728b24b
fix: Restore withName:'.*' resource limits in test config
pinin4fjords 26acdbb
fix: Add missing inputs to ALIGN_STAR non-GPU tests
pinin4fjords 8492b0f
Update snap: parabricks with-markdups stub from CI
pinin4fjords c292ed4
ci: Skip parabricks tests on ARM runners
pinin4fjords 306312a
Merge pull request #1 from nf-core/resolve-parabricks
gburnett-nvidia 6ad50ef
Addressing unstable log_progress output in align_star
gburnett-nvidia 27950b7
Updating Parabricks default tests to use prokaryotic input (smaller d…
gburnett-nvidia 19a7809
Updating rnafq2bam module
gburnett-nvidia 8015391
Updating nf-test requirements to 30GB (will fail default CI runner)
gburnett-nvidia 66cbbf3
ci: Upgrade GPU runner to g4dn.2xlarge (32 GB RAM)
pinin4fjords 9077c47
ci: Switch GPU runner to g5.2xlarge (A10G, 24 GB VRAM)
pinin4fjords f7b04f7
ci: Limit PARABRICKS_RNA_FQ2BAM to single concurrent task
pinin4fjords 38147d5
ci: Increase resourceLimits in gpu profile to 30 GB
pinin4fjords 66786e6
ci: Re-trigger GPU tests
pinin4fjords ff54319
fix: Move gpu profile after test profiles so resourceLimits override …
pinin4fjords 40e7c60
fix: Add test_gpu profile with dedicated resourceLimits for GPU CI
pinin4fjords 25c20bd
style: Restore original profile definition order in nextflow.config
pinin4fjords 581d829
fix: Add --out-prefix to parabricks rnafq2bam for consistent STAR log…
pinin4fjords 5bb695b
fix: Update align_star parabricks snapshot for prefixed STAR output f…
pinin4fjords a19925b
fix: adding fa_icon for Parabricks option to nextflow_schema.json
gburnett-nvidia 52e2aef
fix: Update pipeline parabricks snapshots for --out-prefix changes
pinin4fjords 8d400f8
Migrating version outputs to topics
gburnett-nvidia d3a0f22
Merge branch 'add_parabricks_star' of github.com:gburnett-nvidia/rnas…
gburnett-nvidia 77d89dc
fix: remove Parabricks stargenomegenerate by using existing module wi…
gburnett-nvidia 95e7be8
chore: updating snapshot
gburnett-nvidia 87a071c
fix: swapping out parabricks stargenomegenerate module in align_star …
gburnett-nvidia 8f5a93d
chore: update snapshots
gburnett-nvidia 95b9c3a
refactor: Replace local iGenomes STAR modules with nf-core aliases
pinin4fjords 937ebf9
feat: Update cat/fastq module to handle uncompressed FASTQ inputs
pinin4fjords aa93e86
feat: Accept uncompressed FASTQ in input schema and validate consistency
pinin4fjords b63ef00
Merge pull request #1686 from nf-core/feature/uncompressed-fastq-input
pinin4fjords 583d448
fix: Skip validation of igenomes_base to fix offline mode
pinin4fjords a6de221
docs: Add changelog entry for offline validation fix
pinin4fjords 569414c
refactor: Make legacy STAR flag logic more explicit per review feedback
pinin4fjords 59739d4
fix: Only skip igenomes_base validation when NXF_OFFLINE=true
pinin4fjords e764568
Merge pull request #1696 from nf-core/fix/igenomes-offline-validation
pinin4fjords eacf6b4
fix: Add ARM-aware container selection for iGenomes STAR tests
pinin4fjords 8c1a8e4
docs: Add sync warning to iGenomes test config
pinin4fjords fcba504
Merge branch 'dev' into remove-igenomes-local-modules
pinin4fjords d9e2022
docs: Fix arm param help text to reflect current behaviour
pinin4fjords 915bd47
Merge pull request #1694 from nf-core/remove-igenomes-local-modules
pinin4fjords 87e825f
Merge origin/dev into add_parabricks_star
pinin4fjords 139b4a3
fix: Add mutual exclusion validation for Parabricks and Sentieon
pinin4fjords d1222f8
fix: Drop mandatory colon from ALIGN_STAR process selectors
pinin4fjords 5f2d86d
chore: Update parabricks/rnafq2bam module from nf-core/modules
pinin4fjords f4b8371
ci: Adopt multi-tier GPU runner pattern from nf-core/modules
pinin4fjords 8732dfc
Revert "ci: Adopt multi-tier GPU runner pattern from nf-core/modules"
pinin4fjords 89303f1
fix: Address review findings for Parabricks PR
pinin4fjords 9326d20
style: Fix modules.json formatting for prettier
pinin4fjords bdcd4c6
fix: Add samtools stats to with-markdups Parabricks snapshot
pinin4fjords b6c822d
ci: Re-trigger GPU tests (previous run had zombie spot instances)
pinin4fjords ad74ea4
fix: Update multiqc_citations md5 in with-markdups snapshot
pinin4fjords b2fc8ab
fix: Update ambig_info.tsv md5 in parabricks snapshots
pinin4fjords f0fb974
fix: Revert ambig_info.tsv md5 to original value in both snapshots
pinin4fjords 65e5138
fix: Exclude non-deterministic ambig_info.tsv from Parabricks snapshots
pinin4fjords 530c22f
Merge pull request #1685 from gburnett-nvidia/add_parabricks_star
gburnett-nvidia 2b81d09
Add support for seq_platform parameter in the alignment step
c-mertes 3ba5d0f
Merge remote-tracking branch 'upstream/dev' into add-seq-platform
c-mertes bec71c1
Move seq_platform into sample sheet
c-mertes c5272f0
update meta data
c-mertes 6e8796d
feat: Add FastQC step after BBSplit/rRNA filtering
pinin4fjords fe36671
test: Update snapshots for FastQC filtered outputs
pinin4fjords dff959d
Merge pull request #1700 from nf-core/fastqc-filtered
pinin4fjords a3a2870
feat: Add tximport processing for RSEM outputs (#1320)
pinin4fjords 9784d80
refactor: Replace local RSEM modules with nf-core equivalents
pinin4fjords dd843ae
refactor: Extract shared tximport logic into quant_tximport_summarize…
pinin4fjords 85c5f56
refactor: Replace local quantify_rsem with nf-core subworkflow
pinin4fjords d4a8283
fix: Sync quantify_pseudo_alignment with nf-core/modules
pinin4fjords 47676ef
test: Update pipeline-level snapshots for RSEM tximport changes
pinin4fjords 1b66130
update modules
pinin4fjords 07fb60f
docs: Fix RSEM SummarizedExperiment filenames and add assay details
pinin4fjords 7a10e47
Re-fix snapshots
pinin4fjords d33307c
fix: Include single-library samples in merged fastq output
pinin4fjords b391dbc
docs: Add changelog entry for PR #1704
pinin4fjords 78dd50e
Update subworkflow
pinin4fjords c1d9f82
fix: Wire save_merged_fastq param to subworkflow input
pinin4fjords 7c60a8b
fix: Exclude .rds files from snapshot md5 comparison (ARM compatibility)
pinin4fjords f6f990b
Merge pull request #1704 from nf-core/fix/save-merged-fastq-single-sa…
pinin4fjords 61f26fa
fix: Remove .rds entries from md5 assertion sections in snapshots
pinin4fjords f83c774
Merge pull request #1697 from nf-core/rsem-tximport
pinin4fjords 6dff565
feat: Enable UMI deduplication with STAR/RSEM aligner (closes #829)
pinin4fjords af83c42
Update snap
pinin4fjords 20d4f93
docs: Add changelog entry for PR #1707
pinin4fjords 34d3d99
Merge branch 'enable-umi-rsem' of github.com:nf-core/rnaseq into enab…
pinin4fjords fcaf820
Merge pull request #1707 from nf-core/enable-umi-rsem
pinin4fjords 2916928
fix: Allow pre-built pseudo-aligner index without fasta (#1706)
pinin4fjords 83eb312
docs: Add changelog entry for PR #1708
pinin4fjords 2dffdac
fix: Revert filter_gtf_needed change, GTF_FILTER handles missing fasta
pinin4fjords ed686cd
Merge pull request #1708 from nf-core/fix/salmon-index-no-fasta
pinin4fjords 6fd42ba
docs: update metro map renders with nf-metro v0.4.2
pinin4fjords 6877f68
docs: add metro map source and regeneration instructions
pinin4fjords 006f939
docs: add changelog entry for metro map update
pinin4fjords 3dec50e
fix: revert CONTRIBUTING.md, move metro map docs, add trailing newlines
pinin4fjords 08cf550
fix: match old PNG width (2265px) for metro map renders
pinin4fjords 9ee90e3
docs: use output_width in METRO_MAP.md to match old PNG size
pinin4fjords 72a9a53
docs: regenerate metro map renders with nf-metro v0.4.3
pinin4fjords 1f779f9
docs: shorten Infer Strandedness label to fix label placement
pinin4fjords 1befb0f
fix: add trailing newlines to SVGs for pre-commit
pinin4fjords 478470d
docs: regenerate metro map renders with nf-metro v0.4.4
pinin4fjords 165e19a
fix: move changelog entry to correct position (numerical order)
pinin4fjords 8df6726
docs: route RSEM path through UMI-tools Dedup in metro map
pinin4fjords 2421901
fix: scope GPU container flags to process_gpu tasks only
pinin4fjords 2385efe
docs: keep gpu profile block with comment explaining containerOptions…
pinin4fjords bb728a6
docs: add gpu profile to Parabricks usage example
pinin4fjords 8211607
refactor: remove gpu profile, apply GPU flags unconditionally to proc…
pinin4fjords 73905be
docs: add issue reporter to changelog credits
pinin4fjords 01b4b7b
refactor: restore gpu profile as opt-in for container GPU flags
pinin4fjords 834da6f
refactor: expose gpu_container_options param, decouple pbrun --num-gpus
pinin4fjords d4c9dfd
refactor: replace gpu profile with --gpu param
pinin4fjords e399097
chore: regenerate metro map images with nf-metro 0.4.5
pinin4fjords 0133d23
fix: add trailing newlines to SVGs for pre-commit
pinin4fjords 6faa531
docs: regenerate metro map renders with nf-metro v0.4.6
pinin4fjords bfc1c7a
docs: regenerate metro map with nf-metro v0.4.7, fix cat FASTQ label
pinin4fjords b3069a3
fix: move changelog entry to end of list
pinin4fjords 53913cd
fix: remove double trailing newline from SVGs
pinin4fjords fb3a952
Merge pull request #1709 from nf-core/update/metro-map-0.4.2
pinin4fjords b2e66e7
refactor: make accelerator directive executor-conditional
pinin4fjords feda840
refactor: rename --gpu param to --use_gpu
pinin4fjords a5e0e57
Merge branch 'dev' into fix/gpu-profile-scope
pinin4fjords 4810ab5
Update docs/usage.md
pinin4fjords 33024a0
refactor: auto-detect container GPU flags, default gpu_container_opti…
pinin4fjords b805034
refactor: drop --use_gpu param, set accelerator unconditionally
pinin4fjords b24e43d
fix: align gpu_container_options default with schema (null not empty …
pinin4fjords e1c6cec
fix: restore indentation on test profile block
pinin4fjords f03e851
refactor: remove executor gating from containerOptions
pinin4fjords e149812
Merge pull request #1711 from nf-core/fix/gpu-profile-scope
pinin4fjords 39eb586
Merge branch 'dev' into add-seq-platform
pinin4fjords 363db33
refactor: move seq_platform/seq_center from module inputs to ext.args…
pinin4fjords 2350694
revert: restore example samplesheet without seq_platform column
pinin4fjords cbbf6f1
feat: add seq_center samplesheet field and RG override tests
pinin4fjords 277efc2
fix: add --read-group-pl for Parabricks, document missing CN support
pinin4fjords 5ae5a9c
refactor: consolidate RG tag handling into single block
pinin4fjords 4a4cab8
docs: clarify samplesheet seq_platform/seq_center are for per-sample use
pinin4fjords 6c86398
docs: add changelog entry for seq_platform/seq_center support
pinin4fjords 09a5256
fix: move changelog entry to correct PR number order
pinin4fjords 99f2ccf
style: fix prettier formatting in usage.md
pinin4fjords 3620506
fix: override publishDir in RG test configs for strict-mode compat
pinin4fjords 035da66
fix: move RG tests to separate files to avoid config merge conflicts
pinin4fjords cdbd343
chore: install updated modules from nf-core/modules#10246
pinin4fjords 28ee672
chore: update sentieon/staralign module to remove seq_platform/seq_ce…
pinin4fjords a6bd1fd
fix: add bowtie2 RG support, remove stale hisat2 patch, validate seq_…
pinin4fjords a4f68e7
style: remove redundant docker.runOptions from RG test config
pinin4fjords 5d25d43
style: remove redundant docker.runOptions from all test configs
pinin4fjords 769d26b
style: remove unnecessary params from RG test config
pinin4fjords fbd125b
fix: add missing params to RG test config for strict-mode compat
pinin4fjords f59a5dc
docs: regenerate metro map with nf-metro v0.5.1
pinin4fjords ddbe46d
docs: fix metro map PNG width to match previous dimensions
pinin4fjords 409eb70
docs: re-render metro map with nf-metro 0.5.2
pinin4fjords 09f2e91
docs: re-render metro map with nf-metro 0.5.2
pinin4fjords fc8f3b4
Merge remote and re-render metro map with nf-metro 0.5.2
pinin4fjords cb2472d
docs: re-render metro map with --no-straight-diamonds
pinin4fjords f8f5c97
docs: update METRO_MAP.md with --no-straight-diamonds and version pin
pinin4fjords 7b4c15a
Merge pull request #1713 from nf-core/refactor/seq-platform-via-ext-args
pinin4fjords 062038a
docs: add PR #1714 to changelog
pinin4fjords bb03829
Merge branch 'dev' into update/metro-map-0.5.0
pinin4fjords 10b4bfe
Bump version to 3.23.0 ahead of release
pinin4fjords 7f4751e
Add PR #1715 to CHANGELOG
pinin4fjords 177d4ca
fix: Add informative error when Salmon fails to produce strandedness …
pinin4fjords 33297ee
docs: Add changelog entry for PR #1716
pinin4fjords 23b63b7
Merge pull request #1715 from nf-core/release/3.23.0
pinin4fjords 8e1181d
Merge branch 'dev' into fix/salmon-strandedness-error-message
pinin4fjords adf9efc
Merge branch 'dev' into update/metro-map-0.5.0
pinin4fjords 90d0e02
Merge pull request #1716 from nf-core/fix/salmon-strandedness-error-m…
pinin4fjords f748c50
feat: add --seq_platform parameter for global sequencing platform RG tag
pinin4fjords 9db365e
docs: update changelog with PR #1718
pinin4fjords ff1a4d1
Merge pull request #1718 from nf-core/feat/seq-platform-param
pinin4fjords 52e59ea
docs: move descriptions above details blocks, add missing deps to cha…
pinin4fjords f16883c
docs: add changelog entry for docs/changelog cleanup PR
pinin4fjords e0598c9
docs: update changelog with PR #1719
pinin4fjords e60691e
Update umitools modules for pins/ patch removal
pinin4fjords 686d7e2
fix: refine .nftignore patterns, update snapshots, remove redundant v…
pinin4fjords c1859bc
docs: update changelog with PR #1725
pinin4fjords 845cf1d
docs: update changelog with PR #1724
pinin4fjords 7658884
fix: add salmon.merged.tx2gene.tsv to parabricks snapshots
pinin4fjords 1820b78
fix: remove workflow.out.versions from parabricks align_star tests
pinin4fjords 17c382d
Merge pull request #1719 from nf-core/fix/pr1717-docs-changelog
pinin4fjords 27236a9
Merge branch 'dev' into fix/umitools-module-updates
pinin4fjords a1ba921
Merge branch 'dev' into fix/nftignore-snapshots
pinin4fjords cd7783f
ci: increase nf-test sharding from 14 to 20
pinin4fjords c4a380c
Merge pull request #1724 from nf-core/fix/umitools-module-updates
pinin4fjords 86f3fab
fix: remove tx2gene from parabricks stub snapshots
pinin4fjords 5e7443d
Merge branch 'dev' into fix/nftignore-snapshots
pinin4fjords e9f5958
Merge branch 'dev' into update/metro-map-0.5.0
pinin4fjords 196a5e6
Merge pull request #1725 from nf-core/fix/nftignore-snapshots
pinin4fjords 8f3916a
Merge pull request #1714 from nf-core/update/metro-map-0.5.0
pinin4fjords d903c4e
docs: re-render metro map with nf-metro v0.5.3
pinin4fjords e2d43fa
docs: add PR #1728 to changelog and fix release date
pinin4fjords 84f0792
docs: re-render metro map with nf-metro v0.5.4
pinin4fjords 02b80f0
ci: bump Nextflow to 25.04.3 to fix conda --mkdir bug
pinin4fjords 0d67b07
docs: add PR #1730 to changelog
pinin4fjords 8e379e1
ci: bump minimum Nextflow version to 25.04.3
pinin4fjords 381a3be
Merge pull request #1728 from nf-core/update/metro-map-0.5.3
pinin4fjords 0af2caa
Merge branch 'dev' into fix/nxf-version-conda-mkdir
pinin4fjords 6356ebf
refactor: replace opaque tuple indexing with destructured names
pinin4fjords 360e979
ci: align GPU workflow with other CI workflows
pinin4fjords bfc090f
ci: bump Nextflow to 25.04.3 to fix conda --mkdir bug
pinin4fjords 897bb4c
docs: add PR #1730 to changelog
pinin4fjords 618a39b
ci: bump minimum Nextflow version to 25.04.3
pinin4fjords a0fafe0
Merge pull request #1730 from nf-core/fix/nxf-version-conda-mkdir
pinin4fjords 2babe57
refactor: compose test_prokaryotic profile on prokaryotic
pinin4fjords bd06b4a
Merge branch 'dev' into fix/prokaryotic-profile-composability
pinin4fjords 288e5fd
docs: add PR #1735 to changelog
pinin4fjords c7ca5e5
docs: add PR #1734 to changelog
pinin4fjords 8a38278
merge: resolve conflicts with dev after #1730 merge
pinin4fjords b746123
ci: drop latest-everything from GPU workflow
pinin4fjords 4ce1a57
fix: align ch_transcript_fasta_placeholder assignment
pinin4fjords 15f75cf
Merge pull request #1735 from nf-core/fix/pr1717-code-quality
pinin4fjords b91f906
Merge branch 'dev' into fix/prokaryotic-profile-composability
pinin4fjords 12ca2e0
Merge pull request #1734 from nf-core/fix/prokaryotic-profile-composa…
pinin4fjords 6b76036
test: load prokaryotic config in nf-test instead of duplicating params
pinin4fjords e3d5469
docs: add PR #1736 to changelog
pinin4fjords 4e5e748
test: remove prokaryotic snapshots for regeneration
pinin4fjords f8927b3
test: update prokaryotic star_salmon snapshot for skip_bigwig
pinin4fjords 35ce3a7
Merge pull request #1736 from nf-core/fix/prokaryotic-nftest-use-profile
pinin4fjords 924015e
fix: use bioconda channel for iGenomes STAR 2.6.1d on x86_64
pinin4fjords fdffb6f
Merge branch 'dev' into fix/igenomes-star-conda
pinin4fjords e8ba853
Merge pull request #1738 from nf-core/fix/igenomes-star-conda
pinin4fjords File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
The table of contents is too big for display.
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,17 @@ | ||
| name: Nextflow strict syntax lint | ||
| on: | ||
| pull_request: | ||
| push: | ||
| branches: [master, dev] | ||
|
|
||
| jobs: | ||
| lint: | ||
| runs-on: ubuntu-latest | ||
| steps: | ||
| - uses: actions/checkout@v4 | ||
|
|
||
| - name: Install Nextflow | ||
| uses: nf-core/setup-nextflow@v2 | ||
|
|
||
| - name: Nextflow strict syntax lint | ||
| run: nextflow lint -o concise -exclude nf-test.config . |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,125 @@ | ||
| name: Run GPU nf-tests | ||
| on: | ||
| push: | ||
| branches: | ||
| - dev | ||
| pull_request: | ||
| release: | ||
| types: [published] | ||
| workflow_dispatch: | ||
| inputs: | ||
| runners: | ||
| description: "Runners to test on" | ||
| type: string | ||
| default: "gpu" | ||
|
|
||
| # Cancel if a newer run is started | ||
| concurrency: | ||
| group: ${{ github.workflow }}-${{ github.event.pull_request.number || github.ref }} | ||
| cancel-in-progress: true | ||
|
|
||
| env: | ||
| GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }} | ||
| # renovate: datasource=github-releases depName=askimed/nf-test versioning=semver | ||
| NFT_VER: "0.9.3" | ||
| NXF_ANSI_LOG: false | ||
| NXF_SINGULARITY_CACHEDIR: ${{ github.workspace }}/.singularity | ||
| NXF_SINGULARITY_LIBRARYDIR: ${{ github.workspace }}/.singularity | ||
|
|
||
| jobs: | ||
| get-shards: | ||
| runs-on: | ||
| - runs-on=${{ github.run_id }}-nf-test-get-shards-gpu | ||
| - runner=2cpu-linux-x64 | ||
| name: "Get Shards" | ||
| outputs: | ||
| shard: ${{ steps.set-shards.outputs.shard }} | ||
| total_shards: ${{ steps.set-shards.outputs.total_shards }} | ||
| steps: | ||
| - name: Check out pipeline code | ||
| uses: actions/checkout@0ad4b8fadaa221de15dcec353f45205ec38ea70b # v4 | ||
| with: | ||
| fetch-depth: 0 | ||
|
|
||
| - name: run nf-test-shard action | ||
| id: set-shards | ||
| uses: ./.github/actions/get-shards | ||
| env: | ||
| NFT_VER: ${{ env.NFT_VER }} | ||
| with: | ||
| tags: "gpu" | ||
| max_shards: 2 | ||
|
|
||
| - name: Set outputs | ||
| id: set-outputs | ||
| run: | | ||
| echo "shard=${{ steps.set-shards.outputs.shard }}" >> $GITHUB_ENV | ||
| echo "total_shards=${{ steps.set-shards.outputs.total_shards }}" >> $GITHUB_ENV | ||
|
|
||
| nf-test-gpu: | ||
| needs: [get-shards] | ||
| runs-on: "runs-on=${{ github.run_id }}/family=g5.2xlarge/image=ubuntu24-gpu-x64" | ||
| name: "GPU Test | ${{ matrix.profile }} | ${{ matrix.shard }} | ${{ matrix.NXF_VER }} | ${{ matrix.filters }}" | ||
| env: | ||
| NXF_VER: ${{ matrix.NXF_VER }} | ||
|
|
||
| # Only run on push if this is the nf-core dev branch (merged PRs) | ||
| # and number of shards is greater than 0 | ||
| if: | | ||
| needs.get-shards.outputs.total_shards > 0 && | ||
| ( | ||
| github.event_name != 'push' || | ||
| ( github.event_name == 'push' && github.repository == 'nf-core/rnaseq' ) | ||
| ) | ||
| strategy: | ||
| fail-fast: false | ||
| matrix: | ||
| profile: [docker, singularity] | ||
| shard: ${{ fromJson(needs.get-shards.outputs.shard) }} | ||
| NXF_VER: | ||
| - "25.04.3" | ||
| filters: [pipeline] | ||
|
|
||
| steps: | ||
| - name: Check out pipeline code | ||
| uses: actions/checkout@0ad4b8fadaa221de15dcec353f45205ec38ea70b # v4 | ||
| with: | ||
| fetch-depth: 0 | ||
|
|
||
| - name: Test CUDA | ||
| run: | | ||
| nvidia-smi -L | ||
|
|
||
| - name: Run nf-test Action | ||
| uses: ./.github/actions/nf-test | ||
| with: | ||
| profile: test_gpu,${{ matrix.profile }} | ||
| shard: ${{ matrix.shard }} | ||
| total_shards: ${{ needs.get-shards.outputs.total_shards }} | ||
| filters: ${{ matrix.filters }} | ||
| tags: "gpu" | ||
|
|
||
| confirm-pass-gpu: | ||
| runs-on: | ||
| - runs-on=${{ github.run_id }}-confirm-pass | ||
| - runner=2cpu-linux-x64 | ||
| needs: [nf-test-gpu] | ||
|
adamrtalbot marked this conversation as resolved.
|
||
| if: always() | ||
| steps: | ||
| - name: One or more tests failed | ||
| if: ${{ contains(needs.*.result, 'failure') }} | ||
| run: exit 1 | ||
|
|
||
| - name: One or more tests cancelled | ||
| if: ${{ contains(needs.*.result, 'cancelled') }} | ||
| run: exit 1 | ||
|
|
||
| - name: All tests ok | ||
| if: ${{ contains(needs.*.result, 'success') }} | ||
| run: exit 0 | ||
|
|
||
| - name: debug-print | ||
| if: always() | ||
| run: | | ||
| echo "toJSON(needs) = ${{ toJSON(needs) }}" | ||
| echo "toJSON(needs.*.result) = ${{ toJSON(needs.*.result) }}" | ||
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.