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

Add json prop for custom patient view sample type colors #10336

Merged
merged 3 commits into from
Aug 31, 2023

Conversation

BasLee
Copy link

@BasLee BasLee commented Aug 22, 2023

Extend the legend colors of sample types in the patient view by configuring custom sample types from a json file on the classpath.
Applies the same mechanism that is used to configure default clinical tracks in the oncoprint.
See also the frontend PR: cBioPortal/cbioportal-frontend#4704

Copy link
Member

@inodb inodb left a comment

Choose a reason for hiding this comment

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

LGTM! Thanks!

Let's revisit when refactoring how this property passing should work

@inodb inodb added the devops label Aug 30, 2023
@BasLee BasLee force-pushed the patient-view-sample-colors-scale branch from ff70205 to 957ccf7 Compare August 30, 2023 15:13
@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 1 Code Smell

0.0% 0.0% Coverage
0.0% 0.0% Duplication

warning The version of Java (11.0.20) you have used to run this analysis is deprecated and we will stop accepting it soon. Please update to at least Java 17.
Read more here

@inodb inodb merged commit 3f07318 into cBioPortal:master Aug 31, 2023
### Define custom sample type colors
Define the colors of custom sample types in the patient view using a json object with for each sample type a color:
```
skin.patient_view.custom_sample_type_colors_json=classpath:/skin-patient-view-custom-sample-type-colors.json
Copy link
Contributor

Choose a reason for hiding this comment

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

@inodb this documentation is wrong now, correct? i mean, why is there a classpath there?

jagnathan pushed a commit to pughlab/cbioportal that referenced this pull request Nov 8, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants