Return ID columns in empty feature frames

[d33bs]

What changed

Each feature function returned a bare pandas.DataFrame() with no columns when its loader had no image or no label image. The functions now return an empty frame that has the ID columns. The functions are compute_colocalization, compute_granularity, compute_intensity, compute_neighbors, and compute_texture.

Why

ObjectLoader sets image or label_image to None when its channel or compartment is missing for an image set. The old empty frame had no columns. A downstream merge keyed on an ID column crashed on this frame. The fix gives the merge a key to join on.

Testing

Each function gets a new test. The test passes a loader with no image and no objects, then checks that the frame has the Metadata_Object_ObjectID column and zero rows.

[coderabbitai]

Warning

Review limit reached

Next included review available in 30 minutes.

View limit details

Limit details: You’ve used the included review currently available.

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Learn how review limits work.

Review configuration:

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: 93d46fc9-4c8e-420a-b089-5724e842eef4

📥 Commits

Reviewing files that changed from the base of the PR and between 62af460 and 00d7d28.

📒 Files selected for processing (10)

• src/zedprofiler/featurization/colocalization.py
• src/zedprofiler/featurization/granularity.py
• src/zedprofiler/featurization/intensity.py
• src/zedprofiler/featurization/neighbors.py
• src/zedprofiler/featurization/texture.py
• tests/featurization/test_colocalization.py
• tests/featurization/test_granularity.py
• tests/featurization/test_intensity.py
• tests/featurization/test_neighbors.py
• tests/featurization/test_texture.py

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
⚠️ Please upload report for BASE (main@62af460). Learn more about missing BASE report.

Additional details and impacted files

@@           Coverage Diff           @@
##             main      #53   +/-   ##
=======================================
  Coverage        ?   95.05%           
=======================================
  Files           ?       17           
  Lines           ?     1658           
  Branches        ?        0           
=======================================
  Hits            ?     1576           
  Misses          ?       82           
  Partials        ?        0           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[d33bs]

@coderabbitai review

[coderabbitai]

⚠️ Action not completed

Review rate limited.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

[MikeLippincott]

LGTM, nice fix to this issue!