AlexsLemonade / OpenPBTA-analysis

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

add extent tumor resection/lgg to tp53/telomerase survival analysis #1342

Closed jharenza closed 2 years ago

jharenza commented 2 years ago

Purpose/implementation Section

What scientific question is your analysis addressing?

After discussion with @adamcresnick, I am adding to the survival analysis two additional terms: extent of tumor resection and LGG group. This is to make sure our results are not strictly due to tumors having been fully resected or belonging to LGG group.

What was your approach?

I add in extent of tumor resection and LGG group for univariate analysis. I also add these terms to the multivariate analyses.

What GitHub issue does your pull request address?

NA

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

Which areas should receive a particularly close look?

NA

Is there anything that you want to discuss further?

We will want to create a new forest plot for this result. Update: done in #1343

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

yes

Results

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

What is your summary of the results?

tp53_score+telomerase_score+extent_of_tumor_resection+lgg_group

tp53_score+telomerase_score+extent_of_tumor_resection+hgg_group

Additive models: tp53_score+telomerase_score+extent_of_tumor_resection for each cancer group

Reproducibility Checklist

Documentation Checklist

jharenza commented 2 years ago

I've left some comments about some of the models here, since I'm not sure what exactly they are contributing. More comments and explanations would be helpful, or some of these models can be removed.

In addition, I'm not sure why there are separate models, tp53_score+telomerase_score+extent_of_tumor_resection+hgg_group and tp53_score+telomerase_score+extent_of_tumor_resection+lgg_group. Is there a reason not to just do: tp53_score+telomerase_score+extent_of_tumor_resection+hgg_group + lgg_group?

I actually didn't think about combining LGG and HGG groups, but I did just make the change. I was going back and forth about this because I was thinking that they tell a slightly different story re: tp53 (everything else being similar in separate vs combined analyses).

I am ok doing one model, cleaning this up, and updating to use one FP for the main figure, so I did that in the latest commit.

Also not sure why there is a diff for results/TableS3-RNA-results-table.xlsx ?

I kept seeing this, too, so must have accidentally added it somewhere, so removed here.