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

Fix getting log1p base #2546

Merged
merged 2 commits into from
Jul 7, 2023
Merged

Conversation

sshen8
Copy link
Contributor

@sshen8 sshen8 commented Jul 6, 2023

Fixes error if the log1p dict doesn't have a base key

Fixes #2497, fixes scverse/scanpy-tutorials#65, fixes #2181

@codecov
Copy link

codecov bot commented Jul 6, 2023

Codecov Report

Merging #2546 (2337f56) into master (5ccce79) will not change coverage.
The diff coverage is 100.00%.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #2546   +/-   ##
=======================================
  Coverage   72.14%   72.14%           
=======================================
  Files         104      104           
  Lines       11678    11678           
=======================================
  Hits         8425     8425           
  Misses       3253     3253           
Impacted Files Coverage Δ
scanpy/preprocessing/_highly_variable_genes.py 95.36% <100.00%> (ø)
scanpy/tools/_rank_genes_groups.py 94.18% <100.00%> (ø)

@flying-sheep
Copy link
Member

@ivirshup this is probably a good idea if scverse/anndata#999 is slated to go into some future anndata version.

can we merge that other PR anyway? you didn’t respond there

@flying-sheep flying-sheep linked an issue Jul 7, 2023 that may be closed by this pull request
3 tasks
@flying-sheep flying-sheep added this to the 1.9.3 milestone Jul 7, 2023
@flying-sheep
Copy link
Member

I added a test that fails on master

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants