improve default fmt

Author: GaetandeCast

examples/model_evaluation/plot_estimator_report.py

@@ -405,7 +405,7 @@ def operational_decision_cost(y_true, y_pred, amount):
 # More plotting options are available via ``heatmap_kwargs``, which are passed to
 # seaborn's heatmap. For example, we can customize the colormap and number format:
 cm_display = report.metrics.confusion_matrix()
-cm_display.plot(heatmap_kwargs={"cmap": "Greens", "fmt": "d"})
+cm_display.plot(heatmap_kwargs={"cmap": "Greens", "fmt": ".2e"})
 plt.show()
 
 # %%

skore/src/skore/_sklearn/_plot/metrics/confusion_matrix.py

@@ -51,7 +51,6 @@ def __init__(
 
     _default_heatmap_kwargs: dict = {
         "cmap": "Blues",
-        "fmt": ".2f",
         "annot": True,
         "cbar": True,
     }
@@ -126,7 +125,7 @@ def _plot_single_estimator(
         self.figure_, self.ax_ = plt.subplots()
 
         heatmap_kwargs_validated = _validate_style_kwargs(
-            self._default_heatmap_kwargs,
+            {"fmt": ".2f" if normalize else "d", **self._default_heatmap_kwargs},
             heatmap_kwargs or {},
         )
 

skore/tests/unit/displays/confusion_matrix/test_estimator.py

@@ -189,7 +189,7 @@ def test_default_heatmap_kwargs(pyplot, forest_binary_classification_with_train_
     assert len(display.ax_.texts) > 0
     assert all(isinstance(text, mpl.text.Text) for text in display.ax_.texts)
     for text in display.ax_.texts:
-        assert "." in text.get_text()
+        assert "." not in text.get_text()
     assert len(display.figure_.axes) == 2