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

RFC68: Impl. structural variants table in Study View #4577

Merged
merged 3 commits into from
Aug 2, 2023

Conversation

pvannierop
Copy link
Contributor

@pvannierop pvannierop commented Apr 7, 2023

Depends on: cBioPortal/cbioportal#10119
RFC document: https://docs.google.com/document/d/1_nhO5dpjIGVbnBzch34keHgQUrxYEFudvFDsybDkgrA/edit
Can be activated using ?featureFlags=STUDY_VIEW_STRUCT_VAR_TABLE

Description

This PR will implement the Structural Variants table in Study View:
image

The Structural Variants table allows to define cohorts based on genomic structural variant events where a gene1 and/or gene2 position is defined.

Using the 'hiden checkbox' system gene1::gene2, gene1:: or ::gene2 structural variant queries can be staged for additional inspection in Results View:
image

@pvannierop pvannierop requested a review from alisman April 7, 2023 10:34
@pvannierop pvannierop self-assigned this Apr 7, 2023
@pvannierop pvannierop changed the title RFC68: Impl. structural variants table in study view RFC68: Impl. structural variants table in Study View Apr 7, 2023
@@ -375,6 +388,17 @@ export type OncokbCancerGene = {
isCancerGene: boolean;
};

export type OncokbCancerStructVar = {
Copy link
Collaborator

Choose a reason for hiding this comment

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

Not available from OncokbAPI? looping in @zhx828

Copy link
Collaborator

Choose a reason for hiding this comment

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

Also, if we must keep, lets put this in models folder

Copy link
Member

Choose a reason for hiding this comment

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

I don't think we have Structural Variant class but AnnotateStructuralVariantQuery

Copy link

@BasLee BasLee Jun 13, 2023

Choose a reason for hiding this comment

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

Also, if we must keep, lets put this in models folder

@alisman
Many types are defined in this Store (and in other stores): maybe better to create an issue for this and clean up the stores all at once, extracting relevant types?

@BasLee BasLee force-pushed the sv_table_in_study_view branch 6 times, most recently from cc3cea3 to f6d32f9 Compare July 19, 2023 12:33
@BasLee BasLee force-pushed the sv_table_in_study_view branch 13 times, most recently from 47af713 to 2444b1b Compare July 25, 2023 10:14
@inodb inodb added the cl-prototype Prototype section in changelog. New features not ready for production use label Jul 27, 2023
@alisman alisman merged commit 2683037 into cBioPortal:master Aug 2, 2023
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cl-prototype Prototype section in changelog. New features not ready for production use
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants