Revision 1844b9138ca898d4c8f0e6d9ac30e90217a6e978 authored by Luke Zappia on 12 August 2022, 09:35:06 UTC, committed by GitHub on 12 August 2022, 09:35:06 UTC
* Add check for number of normalized dispersions

In sc.pp.highly_variable_genes() when flavor='cell_ranger' and
n_top_genes is set check that enough normalized dispersions have been
calculated and if not raise a warning and set n_top_genes to the number
of calculated dispersions.

Fixes #2230

* Use .size instead of len()

* Add test for n_top_genes warning

* Add release note

* Remove blank line

Co-authored-by: Isaac Virshup <ivirshup@gmail.com>
1 parent 4623bb7
Raw File
CONTRIBUTING.md
Contributing
============

Contributions to Scanpy are highly welcome!

Before filing an issue
----------------------
* Search the repository (also google) to see if someone has already reported the same issue.
  This allows contributors to spend less time responding to issues, and more time adding new features!
* Please provide a minimal complete verifiable example for any bug.
  If you're not sure what this means, check out
  [this blog post](http://matthewrocklin.com/blog/work/2018/02/28/minimal-bug-reports)
  by Matthew Rocklin or [this definition](https://stackoverflow.com/help/mcve) from StackOverflow.
* Let us know about your environment. Environment information is available via: `sc.logging.print_versions()`.

Contributing code
-----------------

We love code contributions!
If you're interested in contributing code, please take a look over the [contributing guide](https://scanpy.readthedocs.io/en/latest/dev/index.html) in the main documentation.
back to top