Skip to content

[python] do not attempt to resize empty measurements in tiledbsoma.io.prepare_experiment#4103

Merged
bkmartinjr merged 6 commits intomainfrom
bkm/soma-184-empty-measurement
Jun 3, 2025
Merged

[python] do not attempt to resize empty measurements in tiledbsoma.io.prepare_experiment#4103
bkmartinjr merged 6 commits intomainfrom
bkm/soma-184-empty-measurement

Conversation

@bkmartinjr
Copy link
Copy Markdown
Member

Issue and/or context:

If an experiment has empty Measurements (i.e., var axis is empty dataframe), prepare_experiment will fail. It should just ignore these empty dataframes as they do not need resizing.

Fixes SOMA-184

Changes:

Add code to ignore empty var dataframes.

@codecov
Copy link
Copy Markdown

codecov Bot commented Jun 3, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 89.35%. Comparing base (5ea6a3b) to head (bf207a7).
Report is 1 commits behind head on main.

Additional details and impacted files
@@             Coverage Diff             @@
##             main    #4103       +/-   ##
===========================================
+ Coverage   66.11%   89.35%   +23.24%     
===========================================
  Files         158       59       -99     
  Lines       21110     7066    -14044     
  Branches     1238        0     -1238     
===========================================
- Hits        13956     6314     -7642     
+ Misses       6741      752     -5989     
+ Partials      413        0      -413     
Flag Coverage Δ
libtiledbsoma ?
python 89.35% <ø> (-0.03%) ⬇️

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

Components Coverage Δ
python_api 89.35% <ø> (-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 June 3, 2025 20:11
@bkmartinjr bkmartinjr requested review from jp-dark and ktsitsi June 3, 2025 20:12
@bkmartinjr bkmartinjr merged commit 7758642 into main Jun 3, 2025
18 checks passed
@bkmartinjr bkmartinjr deleted the bkm/soma-184-empty-measurement branch June 3, 2025 21:00
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.

2 participants