Skip to content

Conversation

@MarieSacksick
Copy link
Contributor

@MarieSacksick MarieSacksick commented Sep 9, 2025

Fixes:

Warning: You have Plotly version 5.24.1, which is not compatible with this version of Kaleido (1.0.0).
This means that static image generation (e.g. `fig.write_image()`) will not work.
Please upgrade Plotly to version 6.1.1 or greater, or downgrade Kaleido to version 0.2.1.

@github-actions
Copy link
Contributor

github-actions bot commented Sep 9, 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__.py60100% 
   _base.py1981492%45, 58, 127, 130, 183, 186–187, 189–192, 225, 228–229
   find_ml_task.py610100% 
   types.py27196%28
skore/src/skore/_sklearn/_comparison
   __init__.py70100% 
   feature_importance_accessor.py39294%90, 109
   metrics_accessor.py178398%173, 253, 1215
   report.py1070100% 
   utils.py540100% 
skore/src/skore/_sklearn/_cross_validation
   __init__.py90100% 
   data_accessor.py45393%128, 131, 134
   feature_importance_accessor.py240100% 
   metrics_accessor.py182199%244
   report.py135199%487
skore/src/skore/_sklearn/_estimator
   __init__.py90100% 
   data_accessor.py66198%76
   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.py102793%61–62, 200, 224–226, 230
   utils.py770100% 
skore/src/skore/_sklearn/_plot/data
   __init__.py20100% 
   table_report.py185199%706
skore/src/skore/_sklearn/_plot/metrics
   __init__.py60100% 
   confusion_matrix.py70494%92, 100, 122, 230
   feature_importance_display.py672168%88, 121–122, 124, 142–146, 148–155, 158–160, 162
   metrics_summary_display.py80100% 
   precision_recall_curve.py280598%455, 555, 559, 619, 751
   prediction_error.py227597%179, 186, 422, 505, 705
   roc_curve.py292897%385, 508, 513, 614, 619, 623, 692, 832
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.py19194%83
   high_class_imbalance_warning.py200100% 
   random_state_unset_warning.py100100% 
   shuffle_true_warning.py90100% 
   stratify_is_set_warning.py100100% 
   time_based_column_warning.py210100% 
   train_test_split_warning.py30100% 
skore/src/skore/_utils
   __init__.py6266%8, 13
   _accessor.py90396%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.py380100% 
   _testing.py550100% 
skore/src/skore/project
   __init__.py20100% 
   project.py480100% 
   summary.py75198%120
   widget.py165696%436, 439–441, 525–526
TOTAL400511897% 

Tests Skipped Failures Errors Time
1074 5 💤 0 ❌ 0 🔥 4m 25s ⏱️

@github-actions
Copy link
Contributor

github-actions bot commented Sep 9, 2025

Documentation preview @ 82aef57

@MarieSacksick MarieSacksick marked this pull request as ready for review September 9, 2025 13:15
@thomass-dev
Copy link
Collaborator

thomass-dev commented Sep 9, 2025

Could you please investigate why we have these logs in the PR (looks like the output of plotly)?

image

Copy link
Collaborator

@thomass-dev thomass-dev left a comment

Choose a reason for hiding this comment

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

See my previous comment.

@auguste-probabl
Copy link
Contributor

Can you describe the issue with text if an image is not absolutely necessary? It makes it harder to search.

@MarieSacksick MarieSacksick marked this pull request as draft October 7, 2025 13:43
@MarieSacksick
Copy link
Contributor Author

MarieSacksick commented Oct 8, 2025

@thomass-dev do you remember where you saw these logs? I don't have them locally, and can't find the part of the CI that is so verbose.
I guess it should be in sphinx-build, in the section Run ... /sphinx/build, but I find no such text?

@thomass-dev
Copy link
Collaborator

@MarieSacksick yes it was in the sphinx-build, but i don't see anymore such logs https://github.com/probabl-ai/skore/actions/runs/18338574842/job/52228225673.

@thomass-dev thomass-dev marked this pull request as ready for review October 8, 2025 10:07
@thomass-dev thomass-dev merged commit c242b0c into main Oct 8, 2025
38 checks passed
@thomass-dev thomass-dev deleted the update_plotly branch October 8, 2025 10:07
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.

4 participants