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

Adding OncoKB and CancerHotspots annotation into cBioPortal database #5213

Closed
5 tasks
jjgao opened this issue Nov 8, 2018 · 7 comments
Closed
5 tasks

Adding OncoKB and CancerHotspots annotation into cBioPortal database #5213

jjgao opened this issue Nov 8, 2018 · 7 comments

Comments

@jjgao
Copy link
Member

jjgao commented Nov 8, 2018

We would like to do this for two reasons:

  1. OncoKB and CancerHotspots annotation has become a bottleneck when loading oncoprint for big queries oncokb calls are slow for large query #5167
  2. Supporting OncoKB analysis in study view #5212

A high-level to-do list:

  • extend database scheme to add OncoKB and CancerHotspots annotation
  • change import pipeline to add annotation
  • set up a process and scripts to update annotation when there is new OncoKB or CancerHotspots releases
    • the scripts could be used for importing too
  • documentation of the process so that others can use it
  • modify frontend code

Questions:

  • What data to add?
    • To address # 1, we could add information about whether an alteration is Oncogenic or a hotspot into our database.
    • For # 2, we may need additional information such as oncokb levels. (would on-the-fly loading of levels for oncogenic events fast enough? If so, maybe we don't have to save them in cBioPortal?)
  • Where to add annotation?
    • For mutations, we could add it to either mutation_event or mutation table
      • note: level data has to add to mutation table since it's cancer type specific
    • For CNAs, cna_event?
    • How about fusions? Currently we are modeled as mutations but we will separate them into the structural_variant table.
    • Or should we separate them and have a variant_annotation table? Maybe cleaner?
    • Should we consider re-using the DRIVER_FILTER/TIERS columns?

cc'ing @pieterlukasse @schultzn

@stale
Copy link

stale bot commented Mar 11, 2020

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the stale label Mar 11, 2020
@zhx828
Copy link
Member

zhx828 commented Mar 11, 2020

I think we like to keep this open.

@stale stale bot removed the stale label Mar 11, 2020
@stale
Copy link

stale bot commented Jun 9, 2020

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the stale label Jun 9, 2020
@jjgao
Copy link
Member Author

jjgao commented Jun 10, 2020

unstale

@stale stale bot removed the stale label Jun 10, 2020
@stale
Copy link

stale bot commented Sep 9, 2020

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the stale label Sep 9, 2020
@jjgao jjgao removed the stale label Sep 16, 2020
@stale
Copy link

stale bot commented Dec 15, 2020

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the stale label Dec 15, 2020
@jjgao
Copy link
Member Author

jjgao commented Dec 16, 2020

This is being implemented based on RFC56.

@jjgao jjgao closed this as completed Dec 16, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants