Skip to content
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

fix(API): Be resilient to item serialization failures #909

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

thomass-dev
Copy link
Collaborator

Part of #628.

@thomass-dev thomass-dev marked this pull request as draft December 10, 2024 16:27
Copy link
Contributor

Coverage

pytest coverage report
FileStmtsMissCoverMissing
src/skore
   __init__.py180100% 
   __main__.py811 80%
   exceptions.py40100% 
src/skore/cli
   __init__.py80100% 
   cli.py320100% 
   launch_dashboard.py22120 42%
   quickstart_command.py1220 83%
src/skore/item
   __init__.py220100% 
   cross_validation_aggregation_item.py4613 93%
   cross_validation_item.py4810 98%
   item.py2210 95%
   item_repository.py4321 94%
   media_item.py6041 93%
   numpy_array_item.py2111 92%
   pandas_dataframe_item.py3011 94%
   pandas_series_item.py3011 94%
   polars_dataframe_item.py2811 94%
   polars_series_item.py2311 93%
   primitive_item.py2321 91%
   sklearn_base_estimator_item.py2911 94%
   skrub_table_report_item.py1011 86%
src/skore/persistence
   __init__.py00100% 
   abstract_storage.py2210 95%
   disk_cache_storage.py3311 95%
   in_memory_storage.py200100% 
src/skore/project
   __init__.py40100% 
   create.py5080 88%
   load.py2330 89%
   project.py6322 95%
src/skore/sklearn
   __init__.py30100% 
   find_ml_task.py1923 85%
   types.py20100% 
src/skore/sklearn/cross_validation
   __init__.py20100% 
   cross_validation_helpers.py4041 89%
   cross_validation_reporter.py2721 90%
   plot.py3814 90%
src/skore/sklearn/train_test_split
   __init__.py00100% 
   train_test_split.py3421 94%
src/skore/sklearn/train_test_split/warning
   __init__.py80100% 
   high_class_imbalance_too_few_examples_warning.py1732 78%
   high_class_imbalance_warning.py1821 88%
   random_state_unset_warning.py1111 87%
   shuffle_true_warning.py901 91%
   stratify_is_set_warning.py1111 87%
   time_based_column_warning.py2212 89%
   train_test_split_warning.py510 80%
src/skore/ui
   __init__.py00100% 
   app.py2552 71%
   dependencies.py710 86%
   project_routes.py9172 91%
src/skore/utils
   __init__.py00100% 
   _show_versions.py290100% 
src/skore/view
   __init__.py00100% 
   view.py50100% 
   view_repository.py1621 83%
TOTAL11938391% 

Tests Skipped Failures Errors Time
172 0 💤 0 ❌ 0 🔥 34.694s ⏱️

@thomass-dev thomass-dev force-pushed the main branch 7 times, most recently from ad7922d to 469d1e5 Compare December 31, 2024 14:42
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.

1 participant