nf-core / modules

Repository to host tool-specific module files for the Nextflow DSL2 community!
https://nf-co.re/modules
MIT License
280 stars 716 forks source link

new module: custom/awk_fai_to_bed #5132

Closed maxulysse closed 7 months ago

maxulysse commented 7 months ago

Is there an existing module for this?

Is there an open PR for this?

Is there an open issue for this?

Are you going to work on this?

maxulysse commented 7 months ago

cf https://github.com/nf-core/sarek/blob/master/modules/local/build_intervals/main.nf https://github.com/nf-core/raredisease/blob/master/modules/local/create_bed_from_fai.nf

asp8200 commented 7 months ago

@maxulysse : Maybe I can do this one? It seems the main thing is to write some nf-test tests for the "new" module, right?

asp8200 commented 7 months ago

@maxulysse : Concerning the proposed name of the new module - the nf-core tool doesn't like underscores in the name of modules. How about we go with custom/faitobed?

asp8200 commented 7 months ago

Made this draft, but couldn't the nf-core module gawk be used instead?

tdanhorn commented 7 months ago

Since this is a stand-alone process just for awk, it looks to me like a very good use case for the gawk module. (If it were a line in some other process, a module would be overkill, but for this it seems like a perfect fit.)

asp8200 commented 7 months ago

@maxulysse : How about I make a PR to Sarek where I replace local/buid_intervals with nf-core/gawk?

maxulysse commented 7 months ago

Yeah, let's see if you can make it work

asp8200 commented 7 months ago

@maxulysse @tdanhorn @ramprasadn : What do you think about this?

maxulysse commented 7 months ago

@maxulysse @tdanhorn @ramprasadn : What do you think about this?

PURE GENIUS