Eric Banks eitanbanks

  • Joined on
eitanbanks commented on pull request broadinstitute/hellbender#380
@eitanbanks

BadCigarFilter was made a default Walker filter and functionality was added to it so that it now also filters out reads whose cigar length does not…

@eitanbanks

The starred classes are critical. I wrote them so can help describe functionality.

@eitanbanks

FYI Tim Fennell recently implemented a similar tool in Picard. Only one needs to be ported - I don't care much either way.

@eitanbanks

I would like to submit that we should connect this issue to the development of the Likelihood Store. Once the LS is live no one (including external…

eitanbanks commented on pull request broadinstitute/picard#171
@eitanbanks

+1 on this being an awesome tool. @vdauwera @akiezun I would definitely recommend using this over the various GATK-based walkers! @yfarjoun same fo…

@eitanbanks

Yes, great. Yossi and I just talked through this and think we've come up with a solution that should work (I'll let him describe at his own leisure).

@eitanbanks

Interesting. So what I think you're saying is that it's going to be hard to remove this class of duplicates given the current implementation (which…