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

Make CellsByIdentities into S3 generic #202

Open
wants to merge 2 commits into
base: develop
Choose a base branch
from

Conversation

samuel-marsh
Copy link

Hi Seurat Team,

This PR is definitely something optional and totally fine if rejected. Recently, I have been updating scCustomize to work with new version of liger which has substantial improvements. It's been wonderful that SeuratObject has so many data access functions as S3 generics (Cells, Features, WhichCells, etc) because I can easily reexport them and extend them to function with liger objects in seamless manner so that same function works with both Seurat and liger objects.

One function, that I recently wrote version for liger, which would be great to have as generic is CellsByIdentities. I realize this doesn't fit use cases of most other generics in SeuratObject because it's only designed to work with full objects and Assay or other classes.

For this PR:

  • I updated the generics.R script copying format of other generics.
  • I updated seurat.R CellsByIdentities again following formats used for other generics.
  • I tried to make changes with littlest noise possible so that it doesn't have inadvertent downstream effects for function within Seurat and I don't think these changes should cause issue elsewhere in code.

Again, totally realize this may either not be priority or not something that you want to change and that's all good and I can implement liger version with different function name if so.

Thanks!
Sam

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

Successfully merging this pull request may close these issues.

1 participant