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

Backport PR #2414 on branch 1.9.x (matplotlib 3.7 compat) #2419

Merged

Conversation

meeseeksmachine
Copy link

Backport PR #2414: matplotlib 3.7 compat

@lumberbot-app lumberbot-app bot added this to the 1.9.2 milestone Feb 16, 2023
@codecov
Copy link

codecov bot commented Feb 16, 2023

Codecov Report

❗ No coverage uploaded for pull request base (1.9.x@97c2617). Click here to learn what that means.
The diff coverage is n/a.

❗ Current head 72ea692 differs from pull request most recent head 8fb038a. Consider uploading reports for the commit 8fb038a to get more accurate results

Additional details and impacted files
@@           Coverage Diff            @@
##             1.9.x    #2419   +/-   ##
========================================
  Coverage         ?   71.83%           
========================================
  Files            ?       98           
  Lines            ?    11543           
  Branches         ?        0           
========================================
  Hits             ?     8292           
  Misses           ?     3251           
  Partials         ?        0           

adamgayoso and others added 11 commits February 16, 2023 14:19
* Fix using custom layer with highly_variable_genes

* Add tests

* Add release note

* Move release note to correct section
* 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 scverse#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>
@ivirshup ivirshup merged commit 1fbbfcd into scverse:1.9.x Feb 16, 2023
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.

None yet

5 participants