BRCAChallenge / brca-exchange

Overall management and deployment of the BRCA Exchange web portal and pipeline scripts
http://brcaexchange.org
28 stars 32 forks source link

Add two more fields to the exLOVD / Multifactorial Analysis tile #1206

Closed melissacline closed 4 years ago

melissacline commented 4 years ago

This is a request from Michael Parsons. The exLOVD data (a.k.a. the BRCA ex-UV data) has two additional fields that would be valuable for us to pull in: the Case-Control LR and the Pathology LR. See this entry, which corresponds to this variant on BRCA Exchange.

The Pathology LR and Case-Control LR have probably been part of the exLOVD data forever, but we didn't realize that there was good information that we weren't pulling in yet. Hopefully, this is "just" a minor change to that web-scraping script (yeah, I know...).

zfisch commented 4 years ago

@melissacline could you provide a wireframe or if it's as simple as adding a couple rows, a description of where you would like the data displayed in the UI?

melissacline commented 4 years ago

You bet. One wireframe coming up!

melissacline commented 4 years ago

Here is a quick wireframe. @zfisch Let me know if you need anything else.

zfisch commented 4 years ago

@melissacline need a couple things to wrap this up:

  1. Help text, e.g. https://brcaexchange.org/help#co_occurrence_lr_exlovd
  2. Data dictionary text, e.g. Segregation_LR_exLOVD Log ratio describing the likelihood of variant pathogenicity, given co-segregation data 0.00001 (I could probably guess what would go here but better to confirm)
melissacline commented 4 years ago

@zfisch, here are some wrap-up things for you.

  1. Help text:
  1. Data dictionary text: if you give me the field names, I'll take care of that.

Thanks!

zfisch commented 4 years ago

@melissacline I actually need the text for Pathology LR and Case-Control LR -- we already have co-occurrence, segregation, and summary family history text (unless you want those updated!)

The new fields in the output are called:

Segregation_LR_exLOVD Pathology_LR_exLOVD

Reminder that we translate exLOVD to ExUV in the ui.

Changes can be made on the addExUVLRFields branch or can be done in their own PR afterwards if you'd like.

melissacline commented 4 years ago

Okay, I see. The existing help text is fine. Here's help text for the new fields:: Pathology_LR: This value is the likelihood ratio estimated from tumor pathology data from cancer patients who carry the variant. From ExUV Case-Control LR: This value is the likelihood ratio estimated from analysis of matched case (diagnosed with cancer) and control (unaffected by cancer) patients. From ExUV.

To doublecheck, the new fields are called Pathology_LR_exLOVD and what?

zfisch commented 4 years ago

eep! good catch, thanks!

Case_control_LR_exLOVD Pathology_LR_exLOVD

melissacline commented 4 years ago

Great - I've got a pull request up for you!