broadinstitute / ichorCNA

Estimating tumor fraction in cell-free DNA from ultra-low-pass whole genome sequencing.
GNU General Public License v3.0
160 stars 87 forks source link

mappability threshold for correctReadCounts #70

Open mr-james-ireland opened 4 years ago

mr-james-ireland commented 4 years ago

Hi,

In utils.R at line 150, I'm wondering if a "mappability" parameter should be passed to the correctReadCounts function? Since the correctReadCounts default is 0.9, I find this step crashes for some chroms with low mappability even when I set minMapThresh to a lower value.

Thanks! -James

if (applyCorrection){

correct read counts

    **counts <- correctReadCounts(counts, chrNormalize = chrNormalize)**
    if (!is.null(map)) {
      ## filter bins by mappability
      counts <- filterByMappabilityScore(counts, map=map, mapScoreThres = mapScoreThres)
    }