Add prefix to counts and multiqc files #143
Conversation
|
@PhilPalmer Can you review this? I addressed this issue as well as this one. Notice that @angarb and I decided not to change any file names for rMATS, only MultiQC and counts. |
| variable_read_length = minlen == params.readlength ? false : true | ||
| // get run name and date prefix for counts matrix and multiqc | ||
| run_name = params.run_name ? params.run_name + "_" : "" | ||
| date = new Date().format("MM-dd-yy") |
There was a problem hiding this comment.
Ah nice find, this looks ideal, I hadn't heard of new Date() before
| date = new Date().format("MM-dd-yy") | ||
| run_prefix = run_name + date | ||
| // Set star index to read length unless otherwise specified | ||
| star_index = params.star_index ? params.star_index : "/projects/anczukow-lab/reference_genomes/human/Gencode/star_overhangs/star_${params.readlength}" |
There was a problem hiding this comment.
Ah very cool, you added the selection of the star index based on the read length 👌
|
@PhilPalmer It looks like this failed the CI test. I didn't think it was failing before. Is this something to worry about? |
|
Just seen this, I suspect it's fine. I.e. a GitHub issue and not a pipeline issue If I try and rerun it now it fails but I think that is because the branch has been deleted and it can't checkout to it I really wouldn't worry though because the changes have now been merged and tests are passing on the master branch 🎉 |
No description provided.