Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Sample number and text extension update #125

Merged
merged 3 commits into from
Dec 17, 2024

Conversation

allyhawkins
Copy link
Member

Closes #123 and closes #122

Here I just updated the statements that summarized the total samples and diseases found on the Portal to be 700 samples from 55 types of cancer. We probably could leave it to be over 50 types of cancer if we prefer that?

While I was here I updated the extensions for the .h5ad files in all the places I could find.

The only other number updates should be the diagnosis and modality breakdown in the first section of the results. I plan to do that after I update the figure as a separate PR to address #124.

Copy link

Click the link below to download the manuscript build as a ZIP file.
This build is associated with commit fa72f38.

Manuscript build

There were 1 spelling errors.
Check the build link above for details.

Copy link
Member

@jaclyn-taroni jaclyn-taroni left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You missed the discussion section – line 5 in 05.discussion.md uses the old numbers.

@@ -94,7 +94,7 @@ The ADT by cell counts matrix is normalized by first determining the ambient pro
We skip normalization for cells with low-quality ADT expression, as indicated by `DropletUtils::cleanTagCounts()`.
Although `scpca-nf` normalizes ADT counts, the workflow does not perform any dimensionality reduction of ADT data; only the RNA counts data are used as input for dimensionality reduction.
The normalized ADT data are saved as an `altExp` within the processed `SingleCellExperiment` containing the normalized RNA data and is output to a file with the suffix `_processed.rds`.
All files containing `SingleCellExperiment` objects and associated `altExp` objects are converted to `AnnData` objects and exported as separate RNA (`_rna.hdf5`) and ADT (`_adt.hdf5`) `AnnData` objects.
All files containing `SingleCellExperiment` objects and associated `altExp` objects are converted to `AnnData` objects and exported as separate RNA (`_rna.h5ad`) and ADT (`_adt.h5ad`) `AnnData` objects.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you file issues tracking the change to the figures in https://github.com/AlexsLemonade/scpca-paper-figures/ as well please? The little icons say HDF5.

Copy link
Member

@jaclyn-taroni jaclyn-taroni left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm going to approve – I trust the discussion section issue will get fixed and don't need to look at this again!

Copy link

Click the link below to download the manuscript build as a ZIP file.
This build is associated with commit 8807d36.

Manuscript build

There were 1 spelling errors.
Check the build link above for details.

@allyhawkins allyhawkins merged commit b74a970 into main Dec 17, 2024
1 check passed
@allyhawkins allyhawkins deleted the allyhawkins/sample-number-update branch December 17, 2024 16:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants