Skip to content

[python] concurrency safe monkey patch for AnnData IO path#4223

Merged
bkmartinjr merged 2 commits intomainfrom
bkm/soma-469
Sep 5, 2025
Merged

[python] concurrency safe monkey patch for AnnData IO path#4223
bkmartinjr merged 2 commits intomainfrom
bkm/soma-469

Conversation

@bkmartinjr
Copy link
Copy Markdown
Member

@bkmartinjr bkmartinjr commented Sep 4, 2025

SOMA-449 / #4219 demonstrate a race in the H5AD IO path for tiledbsoma.io.

This race can occur with any storage system (it was reported against GCS, but testing revealed it against a local filesystem as well).

@codecov
Copy link
Copy Markdown

codecov Bot commented Sep 4, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 89.72%. Comparing base (687c428) to head (7189db4).
⚠️ Report is 69 commits behind head on main.

Additional details and impacted files
@@             Coverage Diff             @@
##             main    #4223       +/-   ##
===========================================
+ Coverage   69.58%   89.72%   +20.13%     
===========================================
  Files         169       61      -108     
  Lines       19070     7286    -11784     
  Branches     1233        0     -1233     
===========================================
- Hits        13270     6537     -6733     
+ Misses       5366      749     -4617     
+ Partials      434        0      -434     
Flag Coverage Δ
libtiledbsoma ?
python 89.72% <100.00%> (-0.03%) ⬇️

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

Components Coverage Δ
python_api 89.72% <100.00%> (-0.03%) ⬇️
libtiledbsoma ∅ <ø> (∅)
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@bkmartinjr bkmartinjr marked this pull request as ready for review September 4, 2025 23:11
@bkmartinjr bkmartinjr merged commit a2f7a7f into main Sep 5, 2025
17 checks passed
@bkmartinjr bkmartinjr deleted the bkm/soma-469 branch September 5, 2025 13:56
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.

3 participants