8000 chore(skore)!: Rename metrics.report_metrics to metrics.summarize by glemaitre · Pull Request #1813 · probabl-ai/skore · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

chore(skore)!: Rename metrics.report_metrics to metrics.summarize #1813

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 2 commits into from
Jun 9, 2025

Conversation

glemaitre
Copy link
Member

closes #1794

Rename the .metrics.report_metrics() method to .metrics.summarize().

@glemaitre glemaitre changed the title api(skore): Rename metrics.report_metrics to metrics.summarize chore(skore)!: Rename metrics.report_metrics to metrics.summarize Jun 7, 2025
Copy link
Contributor
github-actions bot commented Jun 7, 2025

Coverage

Coverage Report for skore/
FileStmtsMissCoverMissing
venv/lib/python3.13/site-packages/skore
   __init__.py230100% 
   _config.py280100% 
   exceptions.py440%4, 15, 19, 23
venv/lib/python3.13/site-packages/skore/project
   __init__.py20100% 
   project.py490100% 
   summary.py740100% 
   widget.py138596%375–377, 447–448
venv/lib/python3.13/site-packages/skore/sklearn
   __init__.py70100% 
   _base.py1691491%45, 58, 126, 129, 182, 185–186, 188–191, 224, 227–228
   find_estimators.py270100% 
   find_ml_task.py610100% 
   types.py26196%26
venv/lib/python3.13/site-packages/skore/sklearn/_comparison
   __init__.py50100% 
   metrics_accessor.py198298%170, 1283
   report.py950100% 
   utils.py550100% 
venv/lib/python3.13/site-packages/skore/sklearn/_cross_validation
   __init__.py50100% 
   metrics_accessor.py2070100% 
   report.py122199%467
venv/lib/python3.13/site-packages/skore/sklearn/_estimator
   __init__.py70100% 
   feature_importance_accessor.py143298%216–217
   metrics_accessor.py382897%191, 193, 200, 291, 360, 364, 379, 414
   report.py164298%438–439
venv/lib/python3.13/site-packages/skore/sklearn/_plot
   __init__.py20100% 
   base.py50100% 
   style.py280100% 
   utils.py118595%50, 74–76, 80
venv/lib/python3.13/site-packages/skore/sklearn/_plot/metrics
   __init__.py50100% 
   confusion_matrix.py69494%90, 98, 120, 228
   precision_recall_curve.py260598%459, 559, 563, 623, 743
   prediction_error.py215597%179, 186, 422, 505, 687
   roc_curve.py272897%386, 509, 514, 617, 622, 626, 697, 819
venv/lib/python3.13/site-packages/skore/sklearn/train_test_split
   __init__.py00100% 
   train_test_split.py490100% 
venv/lib/python3.13/site-packages/skore/sklearn/train_test_split/warning
   __init__.py80100% 
   high_class_imbalance_too_few_examples_warning.py17194%80
   high_class_imbalance_warning.py180100% 
   random_state_unset_warning.py100100% 
   shuffle_true_warning.py10190%46
   stratify_is_set_warning.py100100% 
   time_based_column_warning.py21195%73
   train_test_split_warning.py40100% 
venv/lib/python3.13/site-packages/skore/utils
   __init__.py6266%8, 13
   _accessor.py52296%67, 108
   _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.py450100% 
   _show_versions.py33293%65–66
   _testing.py460100% 
TOTAL34178797% 

Tests Skipped Failures Errors Time
847 5 💤 0 ❌ 0 🔥 6m 43s ⏱️

Copy link
Contributor
github-actions bot commented Jun 7, 2025

Documentation preview @ 15d4ed0

Copy link
Contributor
@MarieSacksick MarieSacksick left a comment

Choose a reason for hiding this comment

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

LGTM!

@MarieSacksick MarieSacksick added this pull request to the merge queue Jun 9, 2025
Merged via the queue into probabl-ai:main with commit 10c5a5a Jun 9, 2025
27 checks passed
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.

Rename report_metrics to summarize
2 participants
0