Skip to content

test: Adapt coefficients() logic for meta-estimators #1616

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

Conversation

Schefflera-Arboricola
Copy link
Contributor

followup for PR #1575

@auguste-probabl auguste-probabl changed the title test: modified coefficients() logic (and test) for meta-estimators test: Adapt coefficients() logic for meta-estimators Apr 30, 2025
Copy link
Contributor

Coverage

Coverage Report for skore/
FileStmtsMissCoverMissing
venv/lib/python3.12/site-packages/skore
   __init__.py220100% 
   _config.py280100% 
   exceptions.py440%4, 15, 19, 23
venv/lib/python3.12/site-packages/skore/persistence
   __init__.py00100% 
venv/lib/python3.12/site-packages/skore/persistence/item
   __init__.py55198%97
   altair_chart_item.py19194%14
   item.py22195%86
   matplotlib_figure_item.py36197%19
   media_item.py220100% 
   numpy_array_item.py27196%16
   pandas_dataframe_item.py29196%14
   pandas_series_item.py29196%14
   pickle_item.py220100% 
   pillow_image_item.py25196%15
   plotly_figure_item.py20195%14
   polars_dataframe_item.py27196%14
   polars_series_item.py22195%14
   primitive_item.py23291%13, 15
   sklearn_base_estimator_item.py29196%15
venv/lib/python3.12/site-packages/skore/persistence/repository
   __init__.py20100% 
   item_repository.py59591%15–16, 202–203, 226
venv/lib/python3.12/site-packages/skore/persistence/storage
   __init__.py40100% 
   abstract_storage.py220100% 
   disk_cache_storage.py33196%44
   in_memory_storage.py200100% 
venv/lib/python3.12/site-packages/skore/project
   __init__.py20100% 
   project.py83297%280, 392
venv/lib/python3.12/site-packages/skore/sklearn
   __init__.py60100% 
   _base.py1711491%45, 58, 126, 129, 182, 185–186, 188–191, 224, 227–228
   find_ml_task.py610100% 
   types.py130100% 
venv/lib/python3.12/site-packages/skore/sklearn/_comparison
   __init__.py50100% 
   metrics_accessor.py173398%163, 316, 1294
   report.py67198%17
venv/lib/python3.12/site-packages/skore/sklearn/_cross_validation
   __init__.py50100% 
   metrics_accessor.py198298%318, 1088
   report.py110298%23, 431
venv/lib/python3.12/site-packages/skore/sklearn/_estimator
   __init__.py70100% 
   feature_importance_accessor.py143298%223–224
   metrics_accessor.py3461097%174, 176, 183, 219, 249, 290, 317, 321, 336, 371
   report.py148199%24
venv/lib/python3.12/site-packages/skore/sklearn/_plot
   __init__.py20100% 
   base.py60100% 
   style.py280100% 
   utils.py122595%51, 75–77, 81
venv/lib/python3.12/site-packages/skore/sklearn/_plot/metrics
   __init__.py40100% 
   precision_recall_curve.py173199%660
   prediction_error.py1640100% 
   roc_curve.py176199%649
venv/lib/python3.12/site-packages/skore/sklearn/train_test_split
   __init__.py00100% 
   train_test_split.py51198%16
venv/lib/python3.12/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.py12191%15
   shuffle_true_warning.py10190%46
   stratify_is_set_warning.py12191%15
   time_based_column_warning.py23291%17, 73
   train_test_split_warning.py40100% 
venv/lib/python3.12/site-packages/skore/utils
   __init__.py60100% 
   _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.py360100% 
   _show_versions.py330100% 
TOTAL32198997% 

Tests Skipped Failures Errors Time
816 8 💤 0 ❌ 0 🔥 54.514s ⏱️

Copy link
Contributor

Documentation preview @ c921237

Copy link
Contributor

@auguste-probabl auguste-probabl left a comment

Choose a reason for hiding this comment

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

Thanks!

@auguste-probabl auguste-probabl added this pull request to the merge queue Apr 30, 2025
Merged via the queue into probabl-ai:main with commit 908020f Apr 30, 2025
28 of 29 checks passed
Muhammad-Rebaal pushed a commit to Muhammad-Rebaal/skore that referenced this pull request May 5, 2025
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.

2 participants