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

Zenodo CSVs: Update to include sample information #1718

Closed
7 tasks done
sjspielman opened this issue Mar 27, 2023 · 2 comments
Closed
7 tasks done

Zenodo CSVs: Update to include sample information #1718

sjspielman opened this issue Mar 27, 2023 · 2 comments

Comments

@sjspielman
Copy link
Member

sjspielman commented Mar 27, 2023

Part of #1692

Some of the exported files towards #1692 probably need to be revisited since they do not have sample ids, but could. This issue has some inventory for CSVs without this information and for which ones it can reasonably be added in: first, for CSVs that cannot reasonably accommodate this data without fundamental changes, and then for CSVs that can accommodate this data. Feedback much appreciated here!

Cannot accommodate

  • ROC curves in 4A, S5A, S7D. Just not applicable
  • Co-occurrence plot in 3A. The data seems way too derived to nail back down to sample level, but tagging @jashapiro for thoughts.

Could accommodate with some changes

  • Forest plot results from survival models in 4F, 4G, 5D. The models themselves as CSVs cannot take sample ids, but we could change these files to contain samples that went into the model along with their value for variables in the model. Either way some information gets sacrificed, so we should pick which.

Needs updating

  • S7A barplot: This counts the stranded/polyA across diagnoses. Since the purpose of this plot is to show counts, I felt like counts CSV made more sense, but "uncounting" is matter of removing the dplyr::count() that I added! Highly achievable.
  • 3B: mutant samples. Rather than counting the number of samples, make this longer form with sample ids.
  • 3D: this plot summarizes fractions of cancer groups with chromothripsis events. The data can be changed to have TRUE/FALSE for whether each sample has an event
  • 4H: this is a KM curve. Data can be joined in here I'm sure.
@sjspielman
Copy link
Member Author

Noting that these are all filed, and I'll check the final boxes once all PRs are merged to close this out!

@sjspielman
Copy link
Member Author

🚀

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant