phac-nml / staramrnf

nf-core StarAMR
MIT License
0 stars 0 forks source link

Modify input parameters to integrate with IRIDA-Next UI #15

Closed sgsutcliffe closed 2 months ago

sgsutcliffe commented 3 months ago

One of the main uses of staramrnf will be to be integrated with IRIDA-Next and so I am reformatting the input formatting to be properly integrated with IRIDA-Next user-interface. Changes should not impact the function of the pipeline.

PR checklist

sgsutcliffe commented 3 months ago

The first change I made was to change the input column contigs to fastq_1 for the assembled genome that is input for staramr. This will break some nf-tests that point to an input file from dev branch but this will allow the IRIDA-Next UI to autopopulate the genome with whatever genome was uploaded.

sgsutcliffe commented 3 months ago

Changing contigs to fastq_1 worked but wasn't the right solution. See IRIDA Next Nextflow Pipeline Documentation, rather than using a fastq_1 file type I should create one for contigs

sgsutcliffe commented 3 months ago

The CLI arguments added to match the Galaxy staramr UI

sgsutcliffe commented 3 months ago

In addition to changing the MLST default name from "None" to "Automatic" I modified the function convert to be more flexible and accept species metadata from Kraken2.

sgsutcliffe commented 3 months ago

Removing the genomes param, is a default from nf-core template, and needs some additional files modified to allow it to be removed. See the iridanextexample