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

Arbitrary 2-Dimensional Charts in Study View #223

Open
1 of 3 tasks
adamabeshouse opened this issue Aug 24, 2021 · 8 comments
Open
1 of 3 tasks

Arbitrary 2-Dimensional Charts in Study View #223

adamabeshouse opened this issue Aug 24, 2021 · 8 comments

Comments

@adamabeshouse
Copy link

adamabeshouse commented Aug 24, 2021

Stemming from discussion - users will be able to select any 2 data sources and plot them against each other. These plots will be interactable in the same way as other study view charts: for filtering the cohort interactively.

Finally, from there we can start to build out into other data types. (TBD)

@adamabeshouse
Copy link
Author

A couple of product considerations @cBioPortal/product:

  1. What is the best way to plot a sample attribute vs a patient attribute? Would it make sense to plot values by sample and "distribute" a patient value to all of its samples? This is how we do it for clinical tracks in the oncoprint

  2. For scatter plots, we have simplified by using density. For box plots, this is a little trickier, because each "box" is really just 1-dimensional, not 2-dimensional - making it 2 dimensional is just using random jitter to make it look nicer. What's the best way to handle this when simplifying into a heatmap?

@Sjoerd-van-Hagen
Copy link

Hi @adamabeshouse
I agree with your solution for 1. It is straightforward and always works.
I am not sure I understand your second question. We are trying to bring the plots tab into the study view, right? So why are we not just displaying a box plot when a user selects a categorical and a numerical variable? What is different here compared to the plots tab?

@adamabeshouse
Copy link
Author

@Sjoerd-van-Hagen we decided with the FGA vs mutation count plot to use a density plot aka 2d histogram because the smallness of the chart means the full size points become impossible to read and interact with for large studies. So that's my question - how do we handle a box plot given the smallness of the chart?

@Sjoerd-van-Hagen
Copy link

@adamabeshouse would a smaller box plot not work? We do not have to draw the individual points if it is too small. When the user scales it up to a certain point we can start showing them. Does that make sense?

@adamabeshouse
Copy link
Author

@Sjoerd-van-Hagen this is exactly my question - how do we not draw the individual points in a box plot? maybe a violin plot or something like that? or a 1-dimensional heatmap that is thickened for easier visibility?

@Sjoerd-van-Hagen
Copy link

I think I would just draw the boxplot without drawing the points in the initial version but perhaps the rest of @cBioPortal/product have a stronger preference for another solution?

@adamabeshouse
Copy link
Author

Thats a good point, I hadnt thought of that

@schultzn
Copy link

schultzn commented Sep 5, 2021 via email

@JREastonMarks JREastonMarks transferred this issue from cBioPortal/cbioportal Mar 1, 2024
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

3 participants