add dedicated output channel for trimmomatic stderr log (needed for multiqc) #5501
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description & Reason
The Trimmomatic module does not appear to capture the standard error log (
2>
) that is required for parsing by MultiQC as an output. Although there is a different type of log file exposed as an output via the-trimlog
command line parameter, this trim log is quite different than the standard error log and is not parsed by MultiQC based on my own testing and the MultiQC docs on Trimmomatic.My hacky workaround right now is adding
ext.args = "2> trim_out.log"
to themodules.config
file, but it feels improper because of the limited control of file naming and how this stderr log gets captured in the same output channel as the trim log. (Are people using a different approach to capture this log for MultiQC? I am happy to reject this PR if there is a clean workaround that I am not aware of.)My proposed changes separates the stderr log (needed for MultiQC) and trim logs into their own separate output channels. If this gets incorporated, MultiQC report generation becomes cleaner: