Skip to content

Add better compression support to CLI - #712

Merged
mdekstrand merged 4 commits into
lenskit:mainfrom
mdekstrand:feature/cli-update
Apr 12, 2025
Merged

Add better compression support to CLI#712
mdekstrand merged 4 commits into
lenskit:mainfrom
mdekstrand:feature/cli-update

Conversation

@mdekstrand

Copy link
Copy Markdown
Member

This improves the compression support for the CLI, preferring the zstandard module to the zstd command.

@mdekstrand mdekstrand added the cli Command-line interface label Apr 12, 2025
@mdekstrand mdekstrand added this to the 2025.3 milestone Apr 12, 2025
@mdekstrand
mdekstrand marked this pull request as ready for review April 12, 2025 20:57
@codecov

codecov Bot commented Apr 12, 2025

Copy link
Copy Markdown

Codecov Report

Attention: Patch coverage is 16.66667% with 5 lines in your changes missing coverage. Please review.

Project coverage is 90.39%. Comparing base (52e9054) to head (41865bf).
Report is 4 commits behind head on main.

Files with missing lines Patch % Lines
src/lenskit/cli/__init__.py 0.00% 3 Missing ⚠️
src/lenskit/cli/recommend.py 0.00% 1 Missing ⚠️
src/lenskit/cli/train.py 50.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #712      +/-   ##
==========================================
- Coverage   90.42%   90.39%   -0.03%     
==========================================
  Files         137      137              
  Lines        9556     9560       +4     
==========================================
+ Hits         8641     8642       +1     
- Misses        915      918       +3     

☔ View full report in Codecov by Sentry.
📢 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.

@mdekstrand
mdekstrand merged commit 41865bf into lenskit:main Apr 12, 2025
@mdekstrand
mdekstrand deleted the feature/cli-update branch April 12, 2025 21:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cli Command-line interface

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant