Skip to content

Conversation

@eduard-watchmaker
Copy link
Contributor

@eduard-watchmaker eduard-watchmaker commented Dec 2, 2025

mistake in the bwamem_index schema

@sateeshperi sorry for missing this one!

@eduard-watchmaker eduard-watchmaker requested a review from a team as a code owner December 2, 2025 18:13
@github-actions
Copy link

github-actions bot commented Dec 2, 2025

nf-core pipelines lint overall result: Passed ✅

Posted for pipeline commit a717f8d

+| ✅ 234 tests passed       |+
#| ❔   7 tests were ignored |#
Details

❔ Tests ignored:

  • files_exist - File is ignored: lib/nfcore_external_java_deps.jar
  • files_exist - File is ignored: conf/modules.config
  • files_exist - File is ignored: .github/workflows/ci.yml
  • files_unchanged - File ignored due to lint config: assets/nf-core-methylseq_logo_light.png
  • files_unchanged - File ignored due to lint config: docs/images/nf-core-methylseq_logo_light.png
  • files_unchanged - File ignored due to lint config: docs/images/nf-core-methylseq_logo_dark.png
  • modules_config - modules_config

✅ Tests passed:

Run details

  • nf-core/tools version 3.5.1
  • Run at 2025-12-06 18:33:29

- add BWA-MEM alignment configuration
- Update GitHub link for Eduard Casas
- replace logo image in assets
@sateeshperi
Copy link
Contributor

hold till upstream module changes: nf-core/modules#9496

sateeshperi and others added 4 commits December 3, 2025 21:54
- Refactor version output to include process name and command used for version retrieval.
- Remove legacy versions.yml file generation in favor of structured output.
@eduard-watchmaker
Copy link
Contributor Author

I can't figure out what's going on with the MULTIQC (it hangs and doesnt finish) so I'd suggest rolling back that update, merge this branch and then separately figuring out what's going on with this update and add a hot fix

- Implemented a mechanism to merge topic channel versions into the main versions file upon workflow completion.
- Updated the software version output to include both main and topic versions in separate YAML files.
- Bump NFT_VER from 0.9.2 to 0.9.3
- Bump NXF_VER from 24.10.5 to 25.04.0
@eduard-watchmaker
Copy link
Contributor Author

Should we ignore the 25.04 tests and merge @sateeshperi ?

@sateeshperi
Copy link
Contributor

Cannot ignore the tests. A fix for multiqc is in works. will be resolved soon

@sateeshperi
Copy link
Contributor

fixed tests. just need to update the gpu snapshot now

@eduard-watchmaker
Copy link
Contributor Author

@nf-core-bot update gpu snapshot path: tests/bwameth_use_gpu.nf.test

@eduard-watchmaker
Copy link
Contributor Author

fixed tests. just need to update the gpu snapshot now

I just updated them, let me know if there is anything else I can do to help! Thanks @sateeshperi

@eduard-watchmaker eduard-watchmaker merged commit 7f30dbf into dev Dec 7, 2025
99 checks passed
@sateeshperi sateeshperi deleted the bwamem_fix branch December 7, 2025 02:35
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.

--aligner bwameth samtools sort bam incorrectly named with deduplicated prefix

3 participants