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) generic assay color range from light to dark (except Treatment Response) #3465

Merged
merged 2 commits into from Oct 27, 2020

Conversation

dippindots
Copy link
Member

@dippindots dippindots commented Oct 19, 2020

Fix cBioPortal/cbioportal#7959

Describe changes proposed in this pull request:

  • a generic assay track data color range from light to dark (except Treatment Response)

before:
image

after:
image

Copy link
Contributor

@pvannierop pvannierop left a comment

Choose a reason for hiding this comment

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

First, can you provide some rationale on why this is needed? Second, I remember long discussions on the colors used for treatment data (predecessor of generic assay data). If I recall well this related to another feature of generic assay data, the 'pivot threshold'. When provided, the pivot threshold represents an externally defined value that sets a color boundary. For treatment data the color would be dark red-light red-white-light blue-dark blue. So white color would represent the pivot threshold. In light of this I would recommend to expand your PR to handle this situation because there is more to it then simply changing the blue color gradient. This behavior can be activated by defining a pivot_threshold value in the meta file (see docs).

@dippindots
Copy link
Member Author

Hi @pvannierop , thanks for the comment. The reason of this is because we noticed that when the pivot threshold is not specified or is not applicable, the color range is from dark to light which seems not a normal expression for low to high. And this is only affect the profile which don't specify pivot threshold. But I do noticed that for Treatment IC50 has been affected by this change, just curious that for this case (in the following screenshot), the range from 0 - 100 should use color from light to dark?
image

@alisman alisman merged commit c1684e8 into cBioPortal:master Oct 27, 2020
@dippindots dippindots changed the title (Oncoprint) generic assay color range from light to dark (Oncoprint) generic assay color range from light to dark (except Treatment Response) Oct 27, 2020
@pvannierop
Copy link
Contributor

@alisman I see this has been merged. This issue is still under discussion here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
4 participants