Skip to content

Expose batch results as a row-by-row iterator#1128

Open
mdekstrand wants to merge 3 commits into
lenskit:mainfrom
mdekstrand:feature/batch-iter
Open

Expose batch results as a row-by-row iterator#1128
mdekstrand wants to merge 3 commits into
lenskit:mainfrom
mdekstrand:feature/batch-iter

Conversation

@mdekstrand

Copy link
Copy Markdown
Member

This updates the batch inference code to expose a generator of individual result rows.

@mdekstrand mdekstrand added this to the 2026.3 milestone Jun 30, 2026
@mdekstrand mdekstrand self-assigned this Jun 30, 2026
@mdekstrand mdekstrand added the batch Batch recommendation facilities label Jun 30, 2026
@mdekstrand mdekstrand moved this from Backlog to In Progress in LensKit Development Jun 30, 2026
@codecov

codecov Bot commented Jun 30, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 91.17647% with 3 lines in your changes missing coverage. Please review.
✅ Project coverage is 89.52%. Comparing base (a3282e8) to head (48b8bde).

Files with missing lines Patch % Lines
src/lenskit/batch/_runner.py 88.88% 2 Missing ⚠️
src/lenskit/batch/_results.py 88.88% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1128      +/-   ##
==========================================
- Coverage   89.53%   89.52%   -0.01%     
==========================================
  Files         256      256              
  Lines       17167    17184      +17     
==========================================
+ Hits        15370    15384      +14     
- Misses       1797     1800       +3     

☔ 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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

batch Batch recommendation facilities

Projects

Status: In Progress

Development

Successfully merging this pull request may close these issues.

1 participant