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

oncoprint - open data in oncoprinter #3227

Merged

Conversation

adamabeshouse
Copy link
Contributor

@adamabeshouse adamabeshouse commented May 22, 2020

image

@adamabeshouse adamabeshouse changed the title oncoprint - download data in oncoprinter format oncoprint - open data in oncoprinter May 27, 2020
@jjgao
Copy link
Member

jjgao commented May 28, 2020

Very cool!

Would it be hard to support mutational spectrum? Maybe they can be supplied as clinical data in the same format as in download (I couldn't find the issue about it somehow).

When heatmap is added, maybe also add a warning that heatmap will not be supported. (or can we support heatmap as well?

Copy link
Member

@onursumer onursumer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, a few minor comments

@@ -339,7 +329,7 @@ function makeGeneticTrackDatum_Data_Type3(oncoprinterInputLine:OncoprinterInputL
return ret as OncoprinterGeneticTrackDatum_Data;
}*/

export function makeGeneticTrackDatum_Data_Type2(
export function makeGeneticTrackDatum_Data(
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

was there a specific reason to have mixed case function names?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I guess its just supposed to make these huge type names a little easier to comprehend

@adamabeshouse adamabeshouse force-pushed the download-oncoprinter-data branch 3 times, most recently from 35a62dd to 44107b3 Compare June 2, 2020 21:26
@adamabeshouse adamabeshouse force-pushed the download-oncoprinter-data branch 2 times, most recently from a3e5e3c to bd092d0 Compare June 5, 2020 18:09
Abeshouse, Adam A./Sloan Kettering Institute added 7 commits June 5, 2020 16:39
…um clinical track) in oncoprinter

Signed-off-by: Abeshouse, Adam A./Sloan Kettering Institute <abeshoua@mskcc.org>
Signed-off-by: Abeshouse, Adam A./Sloan Kettering Institute <abeshoua@mskcc.org>
… oncoprinter

Signed-off-by: Abeshouse, Adam A./Sloan Kettering Institute <abeshoua@mskcc.org>
…coprint coloring for it. Also fix typescript errors

Signed-off-by: Abeshouse, Adam A./Sloan Kettering Institute <abeshoua@mskcc.org>
Signed-off-by: Abeshouse, Adam A./Sloan Kettering Institute <abeshoua@mskcc.org>
Signed-off-by: Abeshouse, Adam A./Sloan Kettering Institute <abeshoua@mskcc.org>
Signed-off-by: Abeshouse, Adam A./Sloan Kettering Institute <abeshoua@mskcc.org>
@adamabeshouse adamabeshouse merged commit 0585d3d into cBioPortal:master Jun 6, 2020
@adamabeshouse adamabeshouse deleted the download-oncoprinter-data branch June 6, 2020 02:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
3 participants