Skip to content
This repository has been archived by the owner on Jun 16, 2023. It is now read-only.

Li-Fraumeni syndrome ----> Li-Fraumeni syndrome (TP53) in v12 #516

Closed
jharenza opened this issue Mar 3, 2023 · 2 comments
Closed

Li-Fraumeni syndrome ----> Li-Fraumeni syndrome (TP53) in v12 #516

jharenza opened this issue Mar 3, 2023 · 2 comments
Labels

Comments

@jharenza
Copy link
Member

jharenza commented Mar 3, 2023

What data file(s) does this issue pertain to?

histologies-base.tsv

What release are you using?

v12

Put your question or report your issue here.

The cancer_predisposition of Li-Fraumeni syndrome became Li-Fraumeni syndrome (TP53) in v12, and NF-1 became "Neurofibromatosis, Type 1 (NF-1) in v12. I think this is the only one we are using thus far, and we use it in the TP53 score module, which feeds into HGG molecular subtyping. We should change the code to a partial match (grepl) to avoid another break in the future.

@jharenza
Copy link
Member Author

This has been updated in v12 modules

@jharenza jharenza added the v12 label Apr 26, 2023
@jharenza
Copy link
Member Author

jharenza commented May 1, 2023

Closed with d3b-center/OpenPedCan-analysis#321

@jharenza jharenza closed this as completed May 1, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

1 participant