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

A little suggestion of update to make ROGUE suitable for large dgCMatrix data #8

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

Conversation

mimi3421
Copy link

@mimi3421 mimi3421 commented Jun 3, 2022

Dear author,

I have used ROGUE in place of the default HVG algorithm for feature sellection in the fastMNN pipeline and found that it returned a more concise UMAP graph and a more reasonable result in functional analysis after clustering.

I make a little change to reduced the RAM footprint and summit here if someone need it.

As log(0+1)=0, I make a little change in the Entropy function so that only the non-zero log result will be updated when dgCMatrix is supplied as the input data matrix, which would greatly reduce the RAM consumption.

mimi3421 added 2 commits June 4, 2022 04:08
Make ROGUE suitable for large dgCMatrix data
Update the old ROGUE to new
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