AlexsLemonade / OpenPBTA-analysis

The analysis repository for the Open Pediatric Brain Tumor Atlas Project
Other
100 stars 67 forks source link

v23 epn add YAP1--MAML2 to fusions list and YAP1 calling, rerun (4/n) #1638

Closed jharenza closed 1 year ago

jharenza commented 1 year ago

Purpose/implementation Section

What scientific question is your analysis addressing?

While the fusion summary file was updated to add YAP1--MAML2 (instead of YAP1--MAMLD2 in this previous PR, the module was not updated to handle this extra fusion. Here, I am adding that fusion to the list (curiously, YAP1--MAMLD2 was not previously in the list), and then adding it to the requirements list for YAP1 subtyping. However, when I did this, the output file now has shifted columns and I am not well-versed in python to update this. just kidding, I think my rstudio rendering was smushed, this seems fine.

Note: this will not pass CI until we add the new files, since this fusion is missing from the summary file.

What was your approach?

What GitHub issue does your pull request address?

Directions for reviewers. Tell potential reviewers what kind of feedback you are soliciting.

Which areas should receive a particularly close look?

Is there anything that you want to discuss further?

Is the analysis in a mature enough form that the resulting figure(s) and/or table(s) are ready for review?

Results

What types of results are included (e.g., table, figure)?

What is your summary of the results?

Reproducibility Checklist

Documentation Checklist