Skip to content

Replace xopen with lenskit.data.xopen - #1239

Merged
mdekstrand merged 6 commits into
lenskit:mainfrom
mdekstrand:feature/1238-xopen
Oct 12, 2026
Merged

mdekstrand merged 6 commits into
lenskit:mainfrom
mdekstrand:feature/1238-xopen

Conversation

@mdekstrand

Copy link
Copy Markdown
Member

This adds a new xopen function to lenskit.data, replacing the xopen package and reducing our dependency load.

Closes #1238.

@mdekstrand mdekstrand added this to the 2026.5 milestone Oct 12, 2026
@mdekstrand mdekstrand self-assigned this Oct 12, 2026
@mdekstrand mdekstrand added dependencies Pull requests that update a dependency file data Data management support. labels Oct 12, 2026
@codecov

codecov Bot commented Oct 12, 2026 •

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 93.93939% with 2 lines in your changes missing coverage. Please review.
✅ Project coverage is 90.44%. Comparing base (245875b) to head (f640005).

Files with missing lines Patch % Lines
src/lenskit/data/_xopen.py 90.90% 2 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1239      +/-   ##
==========================================
- Coverage   90.51%   90.44%   -0.07%     
==========================================
  Files         263      264       +1     
  Lines       17614    17634      +20     
==========================================
+ Hits        15943    15949       +6     
- Misses       1671     1685      +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 merged commit 5c3d71c into lenskit:main Oct 12, 2026
43 of 44 checks passed
@mdekstrand
mdekstrand deleted the feature/1238-xopen branch October 12, 2026 22:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

data Data management support. dependencies Pull requests that update a dependency file

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Remove use of xopen

1 participant