Move igenomes config into pipeline code about tools HOT 2 OPEN

@ewels mentioned that users like to modify this genomes config. In that case, it should be refactored into an index file (e.g. JSON, YAML). Then you can load it in the config, although it still might be easier to have a param for the file name (e.g. params.igenomes_index) and load it in the pipeline code:

def getGenomesIndex(filename) {
  new groovy.json.JsonSlurper().parseText(file(filename).text)
}

from tools.

My summary is that we have two problems here:

1. The includeConfig is within an if statement in the config. This won't work with the new config parser, and needs resolving ASAP.
2. The config itself uses nested parameters, support for which may be dropped in the slightly longer term.

So the most urgent thing is a short term fix for (1). Instead of this:

// Load igenomes.config if required
if (!params.igenomes_ignore) {
    includeConfig 'conf/igenomes.config'
} else {
    params.genomes = [:]
}

Suggestion is to use a ternary expression, which should work:

// Load igenomes.config if required
includeConfig !params.igenomes_ignore ? 'conf/igenomes.config' : 'conf/igenomes_ignored.config'

Where a new igenomes_ignored.config file would simply contain:

params.genomes = [:]

Can maybe think of more elegant syntax, but that's the gist of it.

Regarding (2) in the longer term - it'd be nice to rewrite how all of the iGenomes configuration works. This will break how existing user's config files work, but I think it's ok if we're overhauling the entire references system anyway. So hopefully we can incorporate this syntax change along with the new references back end.

from tools.