Skip to content
This repository has been archived by the owner on Jun 21, 2023. It is now read-only.

Updated manuscript: glioma brain region method #106

Closed
jharenza opened this issue Jul 1, 2020 · 6 comments
Closed

Updated manuscript: glioma brain region method #106

jharenza opened this issue Jul 1, 2020 · 6 comments

Comments

@jharenza
Copy link
Collaborator

jharenza commented Jul 1, 2020

What section of the manuscript should be updated?

https://alexslemonade.github.io/OpenPBTA-manuscript/#clinical-data-harmonization

Add the section (and subsection) needing an update.
Table describing clinical attributes.

Proposed changes

What changes need to be made and/or what details are missing?
Table is missing some information around how glioma brain region was defined.

Explain your rationale for the update.
Missing information - needs to be filled in.

Please include a level of detail that will allow another participant to make the update.
Below is the logic that was used for this (informed by Dr. Angela Waanders):

gliomas$glioma_brain_region <- ifelse(grepl("Thalamus|Pons|Spinal Cord|Brain Stem|Midbrain|Hippocampus|Medulla", gliomas$primary_site), "midline", 
                             ifelse(grepl("Lobe|Cerebellum", gliomas$primary_site), "hemispheric", 
                                    ifelse(grepl("Blood", gliomas$primary_site), "blood", "other")))
gliomas$glioma_brain_region <- ifelse(grepl("Basal Ganglia|Posterior Fossa|Hippocampus;Temporal|Temporal Lobe;Thalamus", gliomas$primary_site), "mixed", paste0(gliomas$glioma_brain_region))

Relevant literature

If there is relevant scientific literature, put links to those items here.
NA

@allisonheath
Copy link

I believe these were further reviewed by the CCI group, just wanted to make sure the same mappings were kept upon that review @jainpayal022 @aadamk @baileyckelly ?

@aadamk
Copy link

aadamk commented Aug 25, 2020

@baileyckelly - during an HGG meeting, you mentioned brain region as resolved. Has the mapping above remained the same or have there been any alterations based on discussion with Cassie?

@jharenza
Copy link
Collaborator Author

@baileyckelly - during an HGG meeting, you mentioned brain region as resolved. Has the mapping above remained the same or have there been any alterations based on discussion with Cassie?

The mappings have expanded from hemispheric, midline, and mixed to many other, and additionally are being named CNS_region in V17.

@baileyckelly or @aadamk - would one of you want to update/add a table of the new mappings (probably easiest) to the manuscript methods?

@aadamk
Copy link

aadamk commented Sep 24, 2020

@jharenza
Updated mappings are attached based on latest v17 histologies candidate file here.

OpenPBTAv17_CNSmapping.txt

@jharenza
Copy link
Collaborator Author

I am working on this today

@jharenza jharenza mentioned this issue Sep 24, 2020
5 tasks
@jharenza
Copy link
Collaborator Author

close with #126

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants