Closed jharenza closed 1 year ago
Although a private repo, I am tracking here that the upstream data wrangling for the *aliquots.tsv
files has been submitted in PR here and will be reviewed by @ewafula. @ewafula please comment here after you have a chance to review! Thanks.
@jharenza, I'll work to review. Hopefully will donentomorrow AM.
@jaclyn-taroni I also found an instance in which the aliquot id was broken up for 7316-135 into 3 different IDs when doing the search for parental aliquot upstream, so I updated that sample in the CBTN file as well. Ready for re-review.
Purpose/implementation Section
What scientific question is your analysis addressing?
What was your approach?
00-prepare-extraction-table.Rmd
01-integrate-subtyping.Rmd
to add theextraction_type
tobase_histology
and then topbta-histologies.tsv
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?
Double check the extraction status looks OK and if this is sufficient for downstream work/ any modifications are suggested.
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)?
pbta_extraction_status
pbta-histologies.tsv
also has the newly subtyped YAP1 EPN samples nowWhat is your summary of the results?
Reproducibility Checklist
Documentation Checklist
README
and it is up to date.analyses/README.md
and the entry is up to date.