sanger-tol / genomenote

This Nextflow DSL2 pipeline takes aligned HiC reads, creates contact maps and a table of statistics.
https://pipelines.tol.sanger.ac.uk/genomenote
MIT License
23 stars 6 forks source link

Unnecessary sort #45

Closed muffato closed 1 year ago

muffato commented 1 year ago

Description of the bug

Found whilst reviewing #44

In the contact_maps subworkflow, bedtools/bamtobed already sorts the bed file, cf https://github.com/sanger-tol/nf-core-modules/blob/main/modules/bedtools/bamtobed/main.nf#L28 There is no need to sort it again at https://github.com/sanger-tol/genomenote/blob/dev/subworkflows/local/contact_maps.nf#L41

(noticed that because bedtools/bamtobed takes a ton of memory)

Command used and terminal output

No response

Relevant files

No response

System information

No response