feat(ComparisonReport)!: Make reports_ a dict instead of a list

[auguste-probabl]

Remove report_names_.

Closes #1577

Supersedes #1961

Co-authored-by: Himanshu Sharma [email protected]

[auguste-probabl]

Notice that we often have to write weird statements like list(report.reports_.values())[0]. Personally I don't think one phrasing is better than the other, so I would just not touch it.

[github-actions]

Documentation preview @ 68b9348

[github-actions]

Coverage Report for skore/

File	Stmts	Miss	Cover	Missing
skore/src/skore
__init__.py	23	0	100%
_config.py	31	0	100%
exceptions.py	4	4	0%	4, 15, 19, 23
skore/src/skore/_sklearn
__init__.py	6	0	100%
_base.py	198	14	92%	45, 58, 127, 130, 183, 186–187, 189–192, 225, 228–229
find_ml_task.py	61	0	100%
types.py	27	1	96%	28
skore/src/skore/_sklearn/_comparison
__init__.py	7	0	100%
feature_importance_accessor.py	39	2	94%	90, 109
metrics_accessor.py	178	3	98%	173, 253, 1215
report.py	107	0	100%
utils.py	54	0	100%
skore/src/skore/_sklearn/_cross_validation
__init__.py	7	0	100%
feature_importance_accessor.py	24	0	100%
metrics_accessor.py	182	1	99%	244
report.py	135	1	99%	487
skore/src/skore/_sklearn/_estimator
__init__.py	9	0	100%
data_accessor.py	58	0	100%
feature_importance_accessor.py	144	2	98%	223–224
metrics_accessor.py	356	8	97%	200, 202, 209, 300, 369, 373, 388, 423
report.py	167	2	98%	448–449
skore/src/skore/_sklearn/_plot
__init__.py	3	0	100%
base.py	7	0	100%
style.py	29	0	100%
utils.py	141	7	95%	59, 83–85, 89, 344–345
skore/src/skore/_sklearn/_plot/data
__init__.py	2	0	100%
table_report.py	183	1	99%	682
skore/src/skore/_sklearn/_plot/metrics
__init__.py	6	0	100%
confusion_matrix.py	70	4	94%	91, 99, 121, 229
feature_importance_display.py	67	21	68%	92, 115–116, 118, 136–140, 142–149, 152–154, 156
metrics_summary_display.py	9	0	100%
precision_recall_curve.py	278	5	98%	459, 559, 563, 623, 743
prediction_error.py	225	5	97%	181, 188, 424, 507, 687
roc_curve.py	290	8	97%	389, 512, 517, 618, 623, 627, 696, 818
skore/src/skore/_sklearn/train_test_split
__init__.py	0	0	100%
train_test_split.py	58	0	100%
skore/src/skore/_sklearn/train_test_split/warning
__init__.py	8	0	100%
high_class_imbalance_too_few_examples_warning.py	19	1	94%	83
high_class_imbalance_warning.py	20	0	100%
random_state_unset_warning.py	10	0	100%
shuffle_true_warning.py	9	0	100%
stratify_is_set_warning.py	10	0	100%
time_based_column_warning.py	21	0	100%
train_test_split_warning.py	3	0	100%
skore/src/skore/_utils
__init__.py	6	2	66%	8, 13
_accessor.py	90	3	96%	34, 146, 190
_environment.py	27	0	100%
_fixes.py	8	0	100%
_index.py	5	0	100%
_logger.py	22	4	81%	15–17, 19
_measure_time.py	10	0	100%
_parallel.py	38	3	92%	23, 33, 124
_patch.py	13	5	61%	21, 23–24, 35, 37
_progress_bar.py	46	0	100%
_repr_html.py	8	0	100%
_show_versions.py	38	0	100%
_testing.py	55	0	100%
skore/src/skore/project
__init__.py	2	0	100%
project.py	48	0	100%
summary.py	74	0	100%
widget.py	165	6	96%	436, 439–441, 525–526
TOTAL	3940	113	97%

Tests	Skipped	Failures	Errors	Time
1039	5 💤	0 ❌	0 🔥	4m 11s ⏱️