Skip to content

Implement collectible warnings and warning collection for batch inference - #1168

Merged
mdekstrand merged 6 commits into
lenskit:mainfrom
mdekstrand:feature/1113-warning-bundles
Jul 30, 2026
Merged

Implement collectible warnings and warning collection for batch inference#1168
mdekstrand merged 6 commits into
lenskit:mainfrom
mdekstrand:feature/1113-warning-bundles

Conversation

@mdekstrand

Copy link
Copy Markdown
Member

This adds a warning collection / counting capability, and enables it for batch inference.

Closes #1113.

@mdekstrand mdekstrand added batch Batch recommendation facilities internals Internal infrastructure (parallelism, math, etc.) labels Jul 30, 2026
@mdekstrand mdekstrand self-assigned this Jul 30, 2026
@codecov

codecov Bot commented Jul 30, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 76.19048% with 15 lines in your changes missing coverage. Please review.
✅ Project coverage is 89.57%. Comparing base (5815b92) to head (c364df6).
⚠️ Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
src/lenskit/logging/_warnings.py 72.54% 14 Missing ⚠️
src/lenskit/knn/user.py 75.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1168      +/-   ##
==========================================
- Coverage   89.61%   89.57%   -0.05%     
==========================================
  Files         261      262       +1     
  Lines       17329    17385      +56     
==========================================
+ Hits        15530    15572      +42     
- Misses       1799     1813      +14     

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

@mdekstrand mdekstrand moved this from Backlog to In Progress in LensKit Development Jul 30, 2026
@mdekstrand
mdekstrand merged commit c364df6 into lenskit:main Jul 30, 2026
40 of 42 checks passed
@github-project-automation github-project-automation Bot moved this from In Progress to Done in LensKit Development Jul 30, 2026
@mdekstrand
mdekstrand deleted the feature/1113-warning-bundles branch July 30, 2026 20:31
mdekstrand added a commit that referenced this pull request Jul 30, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

batch Batch recommendation facilities internals Internal infrastructure (parallelism, math, etc.)

Projects

Development

Successfully merging this pull request may close these issues.

Add general warning-bundling facility

1 participant