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

scv.tl.rank_velocity_genes() Doesn't Rank All Genes #1207

Open
jwalewski opened this issue Feb 23, 2024 · 1 comment
Open

scv.tl.rank_velocity_genes() Doesn't Rank All Genes #1207

jwalewski opened this issue Feb 23, 2024 · 1 comment
Labels
bug Something isn't working

Comments

@jwalewski
Copy link

When I run scv.tl.rank_velocity_genes() on my own data (grouped by "Cell_type", and there are at least hundreds of cells per type), I see only four genes being ranked. Despite this, I can see that many, many genes have expression and contribution to RNA velocity in the phase maps. What's going on here?? The docs say that there's no limit to the number of genes ranked other than a cap at 100: (https://scvelo-dev.readthedocs.io/scvelo.tl.rank_velocity_genes/?highlight=rank_velocity_genes). Otherwise what could be limiting the numbers?

I would post the example here, but the data is yet to be published.

The example code in general would be to run rank_velocity_genes(), see the number of genes listed by cluster, and then create phase maps of more genes than listed.

Not applicable as this is not an error, but again, the output would be more genes with data in phase maps than the ranking of the velocity genes.

Versions
scvelo==0.2.5  scanpy==1.9.4  anndata==0.11.0.dev31+g49ca3bd  loompy==3.0.7  numpy==1.24.4  scipy==1.11.2  matplotlib==3.7.1  sklearn==1.3.0  pandas==2.1.0
@jwalewski jwalewski added the bug Something isn't working label Feb 23, 2024
@jwalewski
Copy link
Author

jwalewski commented Feb 28, 2024

I think I see why this is the case now.

min_likelihood = 0.1 if min_likelihood is None else min_likelihood

Even in cases where the user sets the minimum likelihood to none, this line sets the minimum likelihood to 0.1. In other words, the default value is effectively 0.1.

This would be very helpful to know - perhaps it can be included in the docstring and the official documentation?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant