Skip to content

Conversation

@auguste-probabl
Copy link
Contributor

@auguste-probabl auguste-probabl commented Aug 27, 2025

Closes #1862

Supersedes #1934

Co-authored-by: ncouturier [email protected]

Co-authored-by: N Couturier

Supersedes probabl-ai#1934
@github-actions
Copy link
Contributor

github-actions bot commented Aug 27, 2025

Coverage

Coverage Report for skore/
FileStmtsMissCoverMissing
skore/src/skore
   __init__.py230100% 
   _config.py310100% 
   exceptions.py440%4, 15, 19, 23
skore/src/skore/_sklearn
   __init__.py70100% 
   _base.py1981492%45, 58, 127, 130, 183, 186–187, 189–192, 225, 228–229
   find_estimators.py270100% 
   find_ml_task.py610100% 
   types.py27196%28
skore/src/skore/_sklearn/_comparison
   __init__.py70100% 
   feature_importance_accessor.py39294%92, 111
   metrics_accessor.py178398%170, 250, 1242
   report.py1060100% 
   utils.py540100% 
skore/src/skore/_sklearn/_cross_validation
   __init__.py70100% 
   feature_importance_accessor.py240100% 
   metrics_accessor.py182199%244
   report.py135199%487
skore/src/skore/_sklearn/_estimator
   __init__.py90100% 
   data_accessor.py580100% 
   feature_importance_accessor.py144298%223–224
   metrics_accessor.py356897%200, 202, 209, 300, 369, 373, 388, 423
   report.py167298%448–449
skore/src/skore/_sklearn/_plot
   __init__.py30100% 
   base.py70100% 
   style.py290100% 
   utils.py141795%59, 83–85, 89, 344–345
skore/src/skore/_sklearn/_plot/data
   __init__.py20100% 
   table_report.py188199%684
skore/src/skore/_sklearn/_plot/metrics
   __init__.py60100% 
   confusion_matrix.py70494%91, 99, 121, 229
   feature_importance_display.py672168%92, 115–116, 118, 136–140, 142–149, 152–154, 156
   metrics_summary_display.py90100% 
   precision_recall_curve.py278598%459, 559, 563, 623, 743
   prediction_error.py225597%181, 188, 424, 507, 687
   roc_curve.py290897%389, 512, 517, 618, 623, 627, 696, 818
skore/src/skore/_sklearn/train_test_split
   __init__.py00100% 
   train_test_split.py580100% 
skore/src/skore/_sklearn/train_test_split/warning
   __init__.py80100% 
   high_class_imbalance_too_few_examples_warning.py16193%79
   high_class_imbalance_warning.py170100% 
   random_state_unset_warning.py100100% 
   shuffle_true_warning.py90100% 
   stratify_is_set_warning.py100100% 
   time_based_column_warning.py21195%73
   train_test_split_warning.py30100% 
skore/src/skore/_utils
   __init__.py6266%8, 13
   _accessor.py86396%34, 146, 190
   _environment.py270100% 
   _fixes.py80100% 
   _index.py50100% 
   _logger.py22481%15–17, 19
   _measure_time.py100100% 
   _parallel.py38392%23, 33, 124
   _patch.py13561%21, 23–24, 35, 37
   _progress_bar.py460100% 
   _repr_html.py80100% 
   _show_versions.py33293%65–66
   _testing.py550100% 
skore/src/skore/project
   __init__.py20100% 
   project.py480100% 
   summary.py740100% 
   widget.py165696%436, 439–441, 525–526
TOTAL395711697% 

Tests Skipped Failures Errors Time
1034 5 💤 0 ❌ 0 🔥 8m 59s ⏱️

@github-actions
Copy link
Contributor

github-actions bot commented Aug 27, 2025

Documentation preview @ 40586f9

@glemaitre glemaitre self-requested a review August 30, 2025 11:40
Copy link
Member

@glemaitre glemaitre left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. I just changed a couple of "fold" to "split" in the documentation. I kept "fold" where we could not use "split".

@auguste-probabl auguste-probabl added this pull request to the merge queue Sep 1, 2025
Merged via the queue into probabl-ai:main with commit 693999d Sep 1, 2025
32 checks passed
@auguste-probabl auguste-probabl deleted the push-pltqqpnqtqqo branch September 1, 2025 08:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

feat: Rename fold into split and split_index to split

2 participants