Skip to content

Original identities not respected in ATAC analysis - #117

Open
f6v wants to merge 1 commit into
liulab-dfci:masterfrom
f6v:patch-1
Open

Original identities not respected in ATAC analysis#117
f6v wants to merge 1 commit into
liulab-dfci:masterfrom
f6v:patch-1

Conversation

@f6v

@f6v f6v commented Feb 17, 2021

Copy link
Copy Markdown

I noticed that unlike RNA analysis, the original identities aren't assigned to the Seurat object when using ATACRunSeurat. I propose to add the same behaviour where:

  • The orig.ident is assigned to to SeuratObject
  • The UMAP plot coloured by the orig.ident is saved
    There was also a minor discrepancy in the param name: orig.ident in RNA and orign.ident in the ATAC function.

I noticed that unlike RNA analysis, the original identities aren't assigned to the Seurat object when using `ATACRunSeurat`. I propose to add the same behaviour where:
- The `orig.ident` is assigned to to `SeuratObject`
- The UMAP plot coloured by the `orig.ident` is saved
There was also a minor discrepancy in the param name: `orig.ident` in RNA and ``orign.ident` in the ATAC function.
@f6v

f6v commented Mar 19, 2021

Copy link
Copy Markdown
Author

@DongqingSun96 @baigal628 does this look fine to you?

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