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

Jackstraw method computed on genes rather than cells #19

Closed
nilseling opened this issue Aug 29, 2019 · 2 comments
Closed

Jackstraw method computed on genes rather than cells #19

nilseling opened this issue Aug 29, 2019 · 2 comments

Comments

@nilseling
Copy link

Hi,

could it be that the 'expr_values' matrix needs to be transposed here:
https://github.com/RubD/Giotto/blob/fefab049ce9747acd8a15d75ed67daa1ab16ebd3/R/dimension_reduction.R#L187

I guess the jackstraw permutations are performed on the genes rather than the cells at this point.
Best,

Nils

@RubD
Copy link
Collaborator

RubD commented Aug 29, 2019

Hi Nils,

Thanks for your comment.

Here we wanted to identify which PCs (based on genes) should be used to cluster cells downstream. In order to identify stable (or significant) PCs the jackstraw method randomly and iteratively permutes a set of genes and then computes how this affects each PC. So following this logic it seems that permutations on genes is correct, right?

Best,
Ruben

@nilseling
Copy link
Author

Hi Ruben,
hmm, maybe I'm confused. For me the jackstraw method only returns a comparable result to signPCA(gobject, method = "screeplot") when transposing the matrix. But I only had a quick look and might have to think about this further.
Best,
Nils

RubD pushed a commit that referenced this issue Nov 24, 2019
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

No branches or pull requests

2 participants