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 outer concatenation along axis=1 for missing layers #1291

Merged
merged 3 commits into from Jan 10, 2024

Conversation

ivirshup
Copy link
Member

@ivirshup ivirshup commented Jan 10, 2024

Copy link

codecov bot commented Jan 10, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (522d7ea) 85.14% compared to head (bad632d) 85.15%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1291      +/-   ##
==========================================
+ Coverage   85.14%   85.15%   +0.01%     
==========================================
  Files          34       34              
  Lines        5432     5436       +4     
==========================================
+ Hits         4625     4629       +4     
  Misses        807      807              
Flag Coverage Δ
gpu-tests 52.09% <100.00%> (+0.05%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files Coverage Δ
anndata/_core/merge.py 94.33% <100.00%> (+0.04%) ⬆️

@ivirshup ivirshup merged commit c5531b7 into scverse:main Jan 10, 2024
15 checks passed
meeseeksmachine pushed a commit to meeseeksmachine/anndata that referenced this pull request Jan 10, 2024
ivirshup added a commit that referenced this pull request Jan 11, 2024
…ayers (#1292)

Co-authored-by: Isaac Virshup <ivirshup@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Warn when anndata.concat join=outer fails because of a mismatch of layers
2 participants