Skip to content

[packaging] Make polars installation optional#3684

Open
vkarak wants to merge 3 commits into
reframe-hpc:masterfrom
vkarak:packaging/make-polars-optional
Open

[packaging] Make polars installation optional#3684
vkarak wants to merge 3 commits into
reframe-hpc:masterfrom
vkarak:packaging/make-polars-optional

Conversation

@vkarak

@vkarak vkarak commented Jun 16, 2026

Copy link
Copy Markdown
Contributor

This PR introduces an extra called no-analytics that if passed, it will skip the polars dependency. However, this dependency is needed if RFM_ENABLE_RESULTS_STORAGE is set. We expand the validation check in the CLI for this. Also to avoid failures in lightweight installations, polars is only imported in the functions that is needed.

Finally, this PR introduces also documentation improvements.

Future note: polars will likely be completely removed from dependencies.

Closes #3676.

Signed-off-by: Vasileios Karakasis <vkarak@gmail.com>
@vkarak vkarak force-pushed the packaging/make-polars-optional branch from 996f3f9 to 1a77edd Compare June 16, 2026 21:18
@vkarak

vkarak commented Jun 16, 2026

Copy link
Copy Markdown
Contributor Author

@dmargala With this you will be able to pip install reframe-hpc[no-analytics] and it will not install polars.

@vkarak vkarak marked this pull request as draft June 16, 2026 21:24
@codecov

codecov Bot commented Jun 16, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 76.92308% with 3 lines in your changes missing coverage. Please review.
✅ Project coverage is 91.90%. Comparing base (58b83f2) to head (6aad161).

Files with missing lines Patch % Lines
reframe/frontend/cli.py 66.66% 3 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #3684      +/-   ##
==========================================
- Coverage   91.91%   91.90%   -0.02%     
==========================================
  Files          62       62              
  Lines       13772    13779       +7     
==========================================
+ Hits        12659    12663       +4     
- Misses       1113     1116       +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.

@vkarak

vkarak commented Jun 16, 2026

Copy link
Copy Markdown
Contributor Author

Converting it back to draft; trying to understand why the CI doesn't pick polars as a dependency although it should do it by default.

@vkarak

vkarak commented Jun 17, 2026

Copy link
Copy Markdown
Contributor Author

It seems the uv sync --group dev done to run unit tests removes polars, thus the errors.

@vkarak vkarak marked this pull request as ready for review June 17, 2026 21:39
vkarak added 2 commits June 18, 2026 00:00
Signed-off-by: Vasileios Karakasis <vkarak@gmail.com>
Signed-off-by: Vasileios Karakasis <vkarak@gmail.com>
@vkarak vkarak force-pushed the packaging/make-polars-optional branch from cccfc9e to 6aad161 Compare June 17, 2026 22:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: Todo

Development

Successfully merging this pull request may close these issues.

1 participant