Merge pull request #128 from sanger-tol/patch

Patch 1.3.1

.github/workflows/linting.yml

@@ -19,7 +19,7 @@ jobs:
       - uses: actions/setup-node@v3
 
       - name: Install editorconfig-checker
-        run: npm install -g editorconfig-checker
+        run: npm install -g editorconfig-checker@3.0.2
 
       - name: Run ECLint check
         run: editorconfig-checker -exclude README.md $(find .* -type f | grep -v '.git\|.py\|.md\|json\|yml\|yaml\|html\|css\|work\|.nextflow\|build\|nf_core.egg-info\|log.txt\|Makefile')

CHANGELOG.md

@@ -3,6 +3,18 @@
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/)
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+## [[1.3.1](https://github.com/sanger-tol/readmapping/releases/tag/1.3.0)] - Antipodean Opaleye (patch 1) - [2024-09-24]
+
+### Enhancements & fixes
+
+- Fixed bug in handling CRAM HiC inputs introduced in 1.1.0
+- Fixed bug in handling PacBio FASTQ inputs introduced in 1.3.0
+
+| Dependency | Old version | New version |
+| ---------- | ----------- | ----------- |
+| `bbtools`  |             | 39.01       |
+| `seqtk`    | 1.4         |             |
+
 ## [[1.3.0](https://github.com/sanger-tol/readmapping/releases/tag/1.3.0)] - Antipodean Opaleye - [2024-08-23]
 
 ### Enhancements & fixes

CITATIONS.md

@@ -10,6 +10,10 @@
 
 ## Pipeline tools
 
+- [BBTools](http://sourceforge.net/projects/bbmap/)
+
+  > Bushnell B. BBTools software package. 2014. http://sourceforge.net/projects/bbmap/
+
 - [Blast](https://pubmed.ncbi.nlm.nih.gov/20003500/)
 
   > Camacho C, Coulouris G, Avagyan V, Ma N, Papadopoulos J, Bealer K, Madden TL. BLAST+: architecture and applications. BMC Bioinformatics. 2009 Dec 15;10:421. doi: 10.1186/1471-2105-10-421. PMID: 20003500; PMCID: PMC2803857.
@@ -18,7 +22,7 @@
 
   > Vasimuddin Md, Misra S, Li H, Aluru S. Efficient Architecture-Aware Acceleration of BWA-MEM for Multicore Systems. 2019 IEEE International Parallel and Distributed Processing Symposium. 2019 May;314–24. doi: 10.1109/IPDPS.2019.00041.
 
-- [CRUMBLE]
+- [CRUMBLE](https://www.ncbi.nlm.nih.gov/pmc/articles/PMC6330002/)
 
   > Bonfield JK, McCarthy SA, Durbin R. Crumble: reference free lossy compression of sequence quality values. Bioinformatics. 2019 Jan;35(2):337-339. doi: 10.1093/bioinformatics/bty608. PubMed PMID: 29992288; PMCID: PMC6330002.
 
@@ -30,14 +34,10 @@
 
   > Danecek P, Bonfield JK, Liddle J, Marshall J, Ohan V, Pollard MO, Whitwham A, Keane T, McCarthy SA, Davies RM, Li H. Twelve years of SAMtools and BCFtools. Gigascience. 2021 Feb 16;10(2):giab008. doi: 10.1093/gigascience/giab008. PMID: 33590861; PMCID: PMC7931819.
 
-- [SeqKit]
+- [SeqKit](https://www.ncbi.nlm.nih.gov/pmc/articles/PMC5051824/)
 
   > Shen W, Le S, Li Y, Hu F. SeqKit: A cross-platform and ultrafast toolkit for FASTA/Q file manipulation. PLoS One. 2016 Oct 5;11(10):e0163962. doi: 10.1371/journal.pone.0163962. PubMed PMID: 27706213; PMCID: PMC5051824.
 
-- [Seqtk]
-
-  > Li H. Toolkit for processing sequences in FASTA/Q formats. GitHub Repository. 2012. https://github.com/lh3/seqtk. Accessed August 2024.
-
 ## Software packaging/containerisation tools
 
 - [Anaconda](https://anaconda.com)

LICENSE

@@ -1,6 +1,6 @@
 MIT License
 
-Copyright (c) @priyanka-surana
+Copyright (c) 2022-2024 Genome Research Ltd.
 
 Permission is hereby granted, free of charge, to any person obtaining a copy
 of this software and associated documentation files (the "Software"), to deal

conf/base.config

@@ -41,6 +41,11 @@ process {
         memory = { check_max( ((meta.datatype == "pacbio_clr" || meta.datatype == "ont") ? 2.GB : 1.GB) * task.attempt, 'memory'  ) }
     }
 
+    // minimum 1GB memory
+    withName: 'BBMAP_FILTERBYNAME' {
+        memory = { check_max( 1.GB * task.attempt, 'memory' ) }
+    }
+
     withName: 'SAMTOOLS_COLLATETOFASTA' {
         cpus   = { log_increase_cpus(4, 2*task.attempt, 1, 2) }
         memory = { check_max( 1.GB  * Math.ceil( meta.read_count / 1000000 ) * task.attempt, 'memory' ) }

conf/modules.config

@@ -15,12 +15,16 @@ process {
         ext.args = '-F 0x200 -nt'
     }
 
+    withName: BBMAP_FILTERBYNAME {
+        ext.args = 'include=f'
+    }
+
     withName: '.*:.*:ALIGN_HIC:BWAMEM2_MEM' {
         ext.args = { "-5SPCp -R ${meta.read_group}" }
     }
 
     withName: '.*:.*:ALIGN_ILLUMINA:BWAMEM2_MEM' {
-        ext.args = { "-R ${meta.read_group}" }
+        ext.args = { "-p -R ${meta.read_group}" }
     }
 
     withName: SAMTOOLS_MERGE {

modules.json

@@ -5,6 +5,11 @@
         "https://github.com/nf-core/modules.git": {
             "modules": {
                 "nf-core": {
+                    "bbmap/filterbyname": {
+                        "branch": "master",
+                        "git_sha": "666652151335353eef2fcd58880bcef5bc2928e1",
+                        "installed_by": ["modules"]
+                    },
                     "blast/blastn": {
                         "branch": "master",
                         "git_sha": "583edaf97c9373a20df05a3b7be5a6677f9cd719",
@@ -81,11 +86,6 @@
                         "git_sha": "03fbf6c89e551bd8d77f3b751fb5c955f75b34c5",
                         "installed_by": ["modules"]
                     },
-                    "seqtk/subseq": {
-                        "branch": "master",
-                        "git_sha": "730f3aee80d5f8d0b5fc532202ac59361414d006",
-                        "installed_by": ["modules"]
-                    },
                     "untar": {
                         "branch": "master",
                         "git_sha": "4e5f4687318f24ba944a13609d3ea6ebd890737d",