seurat-object icon indicating copy to clipboard operation
seurat-object copied to clipboard

Make CellsByIdentities into S3 generic

Open samuel-marsh opened this issue 2 months ago • 0 comments

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

samuel-marsh avatar Apr 24 '24 10:04 samuel-marsh