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

Updated analysis: medulloblastoma consensus subtypes #747

Closed
jharenza opened this issue Aug 27, 2020 · 8 comments
Closed

Updated analysis: medulloblastoma consensus subtypes #747

jharenza opened this issue Aug 27, 2020 · 8 comments
Assignees
Labels
blocking release molecular subtyping Related to molecular subtyping of tumors updated analysis

Comments

@jharenza
Copy link
Collaborator

jharenza commented Aug 27, 2020

What analysis module should be updated and why?

#742 via #738 #743

What changes need to be made? Please provide enough detail for another participant to make the update.

Will need to be updated once #746 goes in

What input data should be used? Which data were used in the version being updated?

RNA-Seq collapsed tables
File to be released in #746

When do you expect the revised analysis will be completed?

unsure

Who will complete the updated analysis?

probably @komalsrathi

@jharenza jharenza changed the title Updated analysis: medulloblastoma subtypes Updated analysis: medulloblastoma consensus subtypes Aug 27, 2020
@komalsrathi
Copy link
Collaborator

@jharenza I think it should be PR #743 instead of #738. That's where I am using the expected subtypes to compare the classifiers with the pathology report.

@jharenza
Copy link
Collaborator Author

@jharenza I think it should be PR #743 instead of #738. That's where I am using the expected subtypes to compare the classifiers with the pathology report.

updated, thanks!

@jharenza
Copy link
Collaborator Author

@komalsrathi - will you use the new file below as input for the classifier accuracy assessment? Thank you!

openPBTA-mb-pathology-subtypes.csv

@komalsrathi
Copy link
Collaborator

@jharenza can you assign this to me? I never see these tickets unless they are assigned (only get notified and then I have to search for it).

@jharenza
Copy link
Collaborator Author

@jharenza can you assign this to me? I never see these tickets unless they are assigned (only get notified and then I have to search for it).

done!

@komalsrathi
Copy link
Collaborator

komalsrathi commented Apr 19, 2021

@komalsrathi - will you use the new file below as input for the classifier accuracy assessment? Thank you!

openPBTA-mb-pathology-subtypes.csv

@jharenza This file does not contain two columns that we have in the current input file:
Kids_First_Biospecimen_ID and other_molecular_findings. These are also added in the output files. We don't necessarily need the other_molecular_findings field I think but BS id is required to map the output of classifier (which uses expression data and hence the BS ids) to this file. If you can update the csv file, it would be easy to incorporate. I don't want to make any local changes to this file on my end as I am seeing this was referenced in other tickets as well. Thanks!

This is the input file for ref: https://github.com/AlexsLemonade/OpenPBTA-analysis/blob/master/analyses/molecular-subtyping-MB/input/expected_class_v17.rds

So you can also either write it out as an rds (along with the version) or I can do that on my end.

@jharenza
Copy link
Collaborator Author

Added BS_ID but removed other_molecular_findings to keep this lite for the release.
openPBTA-mb-pathology-subtypes.csv

@jharenza
Copy link
Collaborator Author

Closed with #1019

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
blocking release molecular subtyping Related to molecular subtyping of tumors updated analysis
Projects
None yet
Development

No branches or pull requests

3 participants