Revision: Update Table 1 to include Patient Ns - Githubissues

AlexsLemonade / OpenPBTA-analysis

The analysis repository for the Open Pediatric Brain Tumor Atlas Project

Other

99 stars 66 forks source link

Revision: Update Table 1 to include Patient Ns #1643

Closed jharenza closed 1 year ago

jharenza commented 1 year ago

Purpose/implementation Section

What scientific question is your analysis addressing?

What was your approach?

Response to Reviewer 1, point 6 - add Patient Ns to Table 1

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?

logic ok?

Is there anything that you want to discuss further?

I also updated the table headings so not to have to manually update in the paper later

Note: this may need a rerun after release due to added EPN subtypes.

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)?

Table 1 changes

What is your summary of the results?

Added Patient N column

Reproducibility Checklist

[x] The dependencies required to run the code in this pull request have been added to the project Dockerfile.
[x] This analysis has been added to continuous integration.

Documentation Checklist

[x] This analysis module has a README and it is up to date.
[x] This analysis is recorded in the table in analyses/README.md and the entry is up to date.
[x] The analytical code is documented and contains comments.