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

DietSeurat doesn't drop layers from objects #9464

Closed
rharao opened this issue Nov 8, 2024 · 4 comments
Closed

DietSeurat doesn't drop layers from objects #9464

rharao opened this issue Nov 8, 2024 · 4 comments
Labels
bug Something isn't working duplicate This issue or pull request already exists

Comments

@rharao
Copy link
Contributor

rharao commented Nov 8, 2024

(I have double-checked to confirm that DietSeurat is provided in Seurat and not SeuratObject.)

While working on a reprex to answer a question in another issue on this repo, I found out that each of the following calls:

  • DietSeurat(pbmc_small, layers = "counts") # Only assay is "RNA"
  • DietSeurat(pbmc_small, layers = "counts", assays = "RNA")
  • DietSeurat(pbmc_small, layers = list(RNA = "counts"))

results in an object with all layers retained:

An object of class Seurat 
230 features across 80 samples within 1 assay 
Active assay: RNA (230 features, 20 variable features)
 3 layers present: counts, data, scale.data

Layers not mentioned in the layers argument to DietSeurat should be dropped from the result.

@rharao rharao added the bug Something isn't working label Nov 8, 2024
@samuel-marsh
Copy link
Collaborator

Hi,

Not member of dev team but hopefully can be helpful. This has been reported before (#8054 annd #8197) and hopefully fix can be implemented soon.

@dcollins15 hope you don’t mind me tagging you here. It would be great to have eyes on this from Seurat team as it seems like pretty straightforward fix.

Thanks,
Sam

@samuel-marsh samuel-marsh added the duplicate This issue or pull request already exists label Nov 9, 2024
@rharao
Copy link
Contributor Author

rharao commented Nov 9, 2024 via email

@samuel-marsh
Copy link
Collaborator

No worries!

@rharao
Copy link
Contributor Author

rharao commented Dec 20, 2024

#8197 has been merged into develop.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working duplicate This issue or pull request already exists
Projects
None yet
Development

No branches or pull requests

2 participants