search

hammerlab / biokepi

Bioinformatics Ketrew Pipelines
Apache License 2.0
27 stars 4 forks source link

Name library properly in BWA-MEM #401

Open ihodes opened 7 years ago

ihodes commented 7 years ago

We currently give them all the same name in the @RG header, same for platform (we should at least have the option to name these correctly, and expose it in the EDSL).

smondet commented 7 years ago

we use the sample-name from th FASTQ pairs which we control already right? https://github.com/hammerlab/biokepi/blob/master/src/bfx_tools/bwa.ml#L106

ihodes commented 7 years ago

It's the "LB" part of the @RG that's the issue in this case: https://github.com/hammerlab/biokepi/blob/master/src/bfx_tools/bwa.ml#L75