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

Incorrect mutation data for gCSI_2017 #10

Open
ChristopherEeles opened this issue Oct 6, 2020 · 4 comments
Open

Incorrect mutation data for gCSI_2017 #10

ChristopherEeles opened this issue Oct 6, 2020 · 4 comments

Comments

@ChristopherEeles
Copy link

While investigating an issue opened by a user on GitHub I have discovered a problem with the gCSI_2017 mutation data. It appears that the matrix contains expression values instead of the normal strings needed for summarizeMolecularProfiles to work:

> molecularProfiles(gCSI_2017, 'mutation')[1:5, 1:5]
         NCI-H358   NCI-H292   NCI-H522   NCI-H650    NCI-H23
ARID1A -0.4137651 -0.4137651 -0.4137651 -0.4137651 -0.4137651
JAK1   -0.2579865 -0.2579865 -0.2579865 -0.2579865 -0.2579865
MSH2   -0.2113687 -0.2113687 -0.2113687 -0.2113687 -0.2113687
MSH6   -0.3179615 -0.3179615 -0.3179615 -0.3179615 -0.3179615
NFE2L2 -0.1223739 -0.1223739 -0.1223739 -0.1223739 -0.1223739

As such the results returned for summarizeMolecularProfiles are nonsensical:

> assay(summarizeMolecularProfiles(gCSI_2017, 'mutation', summary.stat='and'), 1)[1:5, 1:5]
       NCI-H358 NCI-H292 NCI-H522 NCI-H650 NCI-H23
ARID1A "1"      "1"      "1"      "1"      "1"    
JAK1   "1"      "1"      "1"      "1"      "1"    
MSH2   "1"      "1"      "1"      "1"      "1"    
MSH6   "1"      "1"      "1"      "1"      "1"    
NFE2L2 "1"      "1"      "1"      "1"      "1" 

Please see PharmacoGx issue #71 for more information on the users code and the sessionInfo for their R environment.

@anthfm
Copy link
Contributor

anthfm commented Oct 6, 2020

Currently working on this. The compareDrugScreens R package used for this data seems to be incorrect (http://research-pub.gene.com/gCSI-cellline-data/).

@ChristopherEeles
Copy link
Author

@imanthonny Any updates on this?

@ChristopherEeles
Copy link
Author

ChristopherEeles commented Mar 26, 2021

@anthfm,

Any updates on this? I would like to close the issue in PharmacoGx.

Best,
Chris

@anthfm
Copy link
Contributor

anthfm commented Apr 4, 2021

@ChristopherEeles Unfortunately we are still working on this.

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

No branches or pull requests

2 participants