Skip to content

fix: add element paths to concat errors - #2568

Open
stanbot8 wants to merge 1 commit into
scverse:mainfrom
stanbot8:agent/issue-552-element-paths
Open

fix: add element paths to concat errors#2568
stanbot8 wants to merge 1 commit into
scverse:mainfrom
stanbot8:agent/issue-552-element-paths

Conversation

@stanbot8

@stanbot8 stanbot8 commented Jul 22, 2026

Copy link
Copy Markdown

Concat errors can omit the AnnData element path that caused the error. Reuse the concat element error context manager at each annotation boundary. Closes #552. All tests pass.

@codecov

codecov Bot commented Jul 22, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 97.87234% with 3 lines in your changes missing coverage. Please review.
✅ Project coverage is 86.01%. Comparing base (8660a9d) to head (9553eed).
✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
src/anndata/_core/merge.py 97.87% 3 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2568      +/-   ##
==========================================
- Coverage   87.83%   86.01%   -1.82%     
==========================================
  Files          49       49              
  Lines        7799     7872      +73     
==========================================
- Hits         6850     6771      -79     
- Misses        949     1101     +152     
Files with missing lines Coverage Δ
src/anndata/_core/merge.py 86.48% <97.87%> (-7.32%) ⬇️

... and 7 files with indirect coverage changes

@stanbot8
stanbot8 force-pushed the agent/issue-552-element-paths branch 3 times, most recently from d6e2259 to 57c0257 Compare July 24, 2026 08:03
@stanbot8
stanbot8 force-pushed the agent/issue-552-element-paths branch from 57c0257 to d2190c6 Compare July 24, 2026 14:46
@stanbot8
stanbot8 force-pushed the agent/issue-552-element-paths branch 2 times, most recently from 9553eed to aa4c0ee Compare July 31, 2026 16:02
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.

Element specific errors on concatenation

1 participant