Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Enable gc correction by default in somatic cnv panel workflow #5966

Merged
merged 2 commits into from Aug 6, 2019

Conversation

mwalker174
Copy link
Contributor

  • Substantially improves results on FFPE samples

@mwalker174 mwalker174 requested a review from LeeTL1220 May 24, 2019 20:42
@codecov
Copy link

codecov bot commented May 24, 2019

Codecov Report

❗ No coverage uploaded for pull request base (master@1a290ae). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             master     #5966   +/-   ##
==========================================
  Coverage          ?   86.934%           
  Complexity        ?     32787           
==========================================
  Files             ?      2014           
  Lines             ?    151468           
  Branches          ?     16642           
==========================================
  Hits              ?    131677           
  Misses            ?     13729           
  Partials          ?      6062

@samuelklee
Copy link
Contributor

Thanks for making this change! I might include more detail with the note ("Substantially improves results on FFPE samples"), for posterity---it's probably true, but we can't really say anything definitive with only N=1 and without the cross-validation procedure I mentioned on Slack. That is, the higher degree of denoising might just be an artifact of effectively removing more PCs with GC-bias correction (since I'm assuming the same number of PCs were explicitly removed in both cases), but it's possible that removing the optimal number of PCs without GC-bias correction could achieve a better result. Since our correction procedure is relatively naive, there may also be some dependence on bin size. However, I think it's probably not worth a detailed analysis, and that it's generally safe to enable correction by default.

@samuelklee
Copy link
Contributor

@mwalker174 Here's that branch. Let's just go ahead and enable this on the gCNV side as well?

@mwalker174 mwalker174 merged commit 596bc9c into master Aug 6, 2019
@mwalker174 mwalker174 deleted the mw_cnv_gc branch August 6, 2019 20:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants