On Thu, Nov 20, 2014 at 10:17 AM, John Marshall <j...@sanger.ac.uk> wrote: > On 19 Nov 2014, at 17:47, Peter Cock <p.j.a.c...@googlemail.com> wrote: >> I have no idea what 'minQLen' was short for, unless the Q >> was a typo? > > It would be Minimum Query Length, where query is a synonym for read > (imagine you were an aligner being asked to map a read...). This is also > what SAM/BAM's QNAME field stands for, as both read and reference > start with R... > >> My proposed change to samtools is here: >> https://github.com/samtools/samtools/pull/321 > > Thanks, I've applied this (after modifying it a little). > > John
It looks like I miss-read the function name - I rejected "minimum query length" believing this was checking the mapped length, rather than the actual full query sequence length. The applied fix looks good :) Peter ------------------------------------------------------------------------------ Download BIRT iHub F-Type - The Free Enterprise-Grade BIRT Server from Actuate! Instantly Supercharge Your Business Reports and Dashboards with Interactivity, Sharing, Native Excel Exports, App Integration & more Get technology previously reserved for billion-dollar corporations, FREE http://pubads.g.doubleclick.net/gampad/clk?id=157005751&iu=/4140/ostg.clktrk _______________________________________________ Samtools-help mailing list Samtools-help@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/samtools-help
