berman-lab / ymap

YMAP - Yeast Mapping Analysis Pipeline : An online pipeline for the analysis of yeast genomic datasets.
MIT License
6 stars 6 forks source link

Wrong segmentation in ChARM #56

Closed ghost closed 7 years ago

ghost commented 7 years ago

Currently the ChARM algorithm does another smoothing after the differentiation filter, this results in wrong segmentation that appeared while running 195L_R_CCGTCC_L002_R1 with the hapmap CBS_vs_BG2.
When running with the second smoothing filter (after differentiation) we get the following result: fig cnv-snp-map 1
And without the second smoothing filter we get: fig cnv-snp-map 1

The segmentation has changed for chromosomes: A,C,G,I and now it's more accurate and since this is not part of the original ChARM article it will be removed in commit 644c15a4dab4166af30ffa79496ba47572cb6b5e (the commit will be merged to Integration and master once the matlab on lovelace will be updated)

darrenabbey commented 7 years ago

The CNV segmentation module could use being replaced with a more modern algorithm. It would be worthwhile to look around and see what the best method out there is right now. There may be software that can be integrated without having to rewrite.

Another note: the copy number data in these figures looks very strange, so it does not surprise me the existing algorithm may not be ideal.

On Oct 10, 2016 6:13 AM, "GuyKha" notifications@github.com wrote:

Currently the ChARM algorithm https://github.com/berman-lab/ymap/wiki/ChARM-algorithm does another smoothing after the differentiation filter, this results in wrong segmentation that appeared while running 195L_R_CCGTCC_L002_R1 with the hapmap CBS_vs_BG2.

When running with the second smoothing filter (after differentiation) we get the following result: [image: fig cnv-snp-map 1] https://cloud.githubusercontent.com/assets/20476730/19234163/7e2f41f2-8ef2-11e6-86e2-269f74ecced0.png

And without the second smoothing filter we get: [image: fig cnv-snp-map 1] https://cloud.githubusercontent.com/assets/20476730/19234179/993402e4-8ef2-11e6-9d29-a57624357530.png

The segmentation has changed for chromosomes: A,C,G,I and now it's more accurate and since this is not part of the original ChARM article it will be removed in commit 644c15a https://github.com/berman-lab/ymap/commit/644c15a4dab4166af30ffa79496ba47572cb6b5e

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/berman-lab/ymap/issues/56, or mute the thread https://github.com/notifications/unsubscribe-auth/AKPuRL7HXdV8I2aYI8bxYObk_22Xx2_bks5qyh3wgaJpZM4KSdGe .