File size: 16,128 Bytes
0864c88
 
 
 
 
e407ec0
0864c88
e407ec0
0864c88
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
200
201
202
203
204
205
206
207
208
209
210
211
212
213
214
215
216
217
218
219
220
221
222
223
224
225
226
227
228
229
230
231
232
233
234
235
236
237
238
239
240
241
242
243
244
245
246
247
248
249
250
251
252
253
254
255
256
257
258
259
260
261
262
263
264
265
266
267
268
269
270
271
272
273
274
275
276
277
278
279
280
281
282
283
284
285
286
287
288
289
290
291
292
293
294
295
296
297
298
299
300
301
302
303
304
305
306
307
308
309
310
311
312
313
314
315
316
317
318
319
320
321
322
323
324
325
326
327
328
329
330
331
332
333
334
335
336
337
338
339
340
341
342
343
344
345
346
347
348
349
350
351
352
353
354
355
356
357
358
359
360
361
362
363
364
365
366
367
368
369
370
371
372
373
374
375
376
377
378
379
380
381
382
383
384
385
386
387
388
389
390
391
392
393
394
395
396
397
398
399
400
401
402
403
404
405
406
407
408
409
410
411
412
413
414
415
416
417
418
419
420
421
422
423
"""Registre typé des hooks de métriques document-level et corpus-level.

Pourquoi ce module
------------------
Avant le « chantier 2 » du plan d'évolution post-Sprint 97,
``picarones.app.services.benchmark_runner._compute_document_result``
contenait **11 imports tardifs codés en dur** vers
``picarones.evaluation.metrics.confusion``, ``char_scores``, ``taxonomy``, ``structure``,
``image_quality``, ``line_metrics``, ``hallucination``,
``philological_hooks``, ``searchability_hooks``,
``numerical_sequences_hooks``, ``readability_hooks`` — chacun enrobé
dans un ``try/except Exception`` qui logue un warning. Symétriquement,
la phase d'agrégation contenait 11 fonctions ``_aggregate_*`` ou
``aggregate_*``. Ajouter une nouvelle métrique exigeait de patcher
``runner.py`` à deux endroits, ce qui rendait le fichier monolithique
(1322 lignes) et fragile.

Ce module centralise le mécanisme :

- **Profils** (``minimal`` / ``standard`` / ``philological`` /
  ``diagnostics`` / ``pipeline`` / ``full``) — l'utilisateur choisit
  quel sous-ensemble de métriques calculer selon son use case.
- **Hooks document-level** (:class:`DocumentMetricHook`) enregistrés via
  :func:`register_document_metric` — fonctions appelées pour chaque
  document, leur retour remplit un attribut nommé du ``DocumentResult``.
- **Agrégateurs corpus-level** (:class:`CorpusMetricAggregator`)
  enregistrés via :func:`register_corpus_aggregator` — fonctions
  appelées une fois par moteur pour synthétiser les
  ``DocumentResult`` en attributs ``aggregated_*`` du ``EngineReport``.

Rétrocompat stricte
-------------------
Le profil ``standard`` (défaut) active exactement les 11 hooks et 11
agrégateurs historiques. Comportement, ordre d'exécution, gestion
d'erreurs et octets de sortie : strictement identiques à avant le
chantier 2. La preuve est dans ``tests/test_metric_hooks.py``
(cas-tests qui comparent profil ``standard`` vs comportement legacy
sur fixtures).

Comment ajouter un hook
-----------------------
Pour ajouter une métrique document-level :

>>> from picarones.evaluation.metric_hooks import (
...     register_document_metric, PROFILE_STANDARD, PROFILE_FULL,
... )
>>>
>>> @register_document_metric(
...     name="my_metric",
...     attribute="my_metric",  # nom du champ dans DocumentResult
...     profiles=(PROFILE_STANDARD, PROFILE_FULL),
...     requires_success=True,
... )
... def my_hook(*, ground_truth, hypothesis, image_path, corpus_lang,
...             ocr_result):
...     # Imports tardifs OK ici — le coût n'est payé que si le hook
...     # est dans le profil actif.
...     from my_pkg import compute_my_metric
...     return compute_my_metric(ground_truth, hypothesis)

Pour un nouveau profil, l'ajouter à :data:`KNOWN_PROFILES` (et
référencer dans la doc utilisateur ``docs/profiles/``).
"""

from __future__ import annotations

import logging
from dataclasses import dataclass
from typing import Any, Callable, Iterable

logger = logging.getLogger(__name__)


# ──────────────────────────────────────────────────────────────────────────
# Profils
# ──────────────────────────────────────────────────────────────────────────

PROFILE_MINIMAL = "minimal"
"""Profil le plus léger : juste CER/WER de ``compute_metrics``. Aucun
hook document-level ni agrégateur ne s'exécute. Cible : tests rapides
ou bench massif où seul le CER global compte."""

PROFILE_STANDARD = "standard"
"""Profil par défaut. Active tous les hooks et agrégateurs historiques
de Picarones (Sprints 5 + 10 + 39+42 + 55-60 + 84-87). Comportement
strictement identique au runner pré-chantier-2."""

PROFILE_PHILOLOGICAL = "philological"
"""Profil orienté édition critique : standard + emphase philologique.
Aujourd'hui équivalent à standard ; réservé pour des hooks futurs
spécifiques aux corpus médiévaux et imprimés anciens."""

PROFILE_DIAGNOSTICS = "diagnostics"
"""Profil orienté diagnostic : standard + leviers d'amélioration,
prédiction de complexité, baseline historique. Réservé pour des
hooks futurs (chantiers 3-4)."""

PROFILE_ECONOMICS = "economics"
"""Profil orienté décision budget : minimal + métriques économiques
(throughput effectif, coût marginal). Réservé pour des hooks futurs."""

PROFILE_PIPELINE = "pipeline"
"""Profil pour les benchmarks de pipelines composées (axe B). Active
les hooks pertinents aux jonctions du DAG. Réservé pour des hooks
futurs spécifiques aux pipelines."""

PROFILE_FULL = "full"
"""Profil exhaustif : tous les hooks de tous les profils. Coût
maximal mais reproductibilité scientifique maximale."""

KNOWN_PROFILES: frozenset[str] = frozenset({
    PROFILE_MINIMAL,
    PROFILE_STANDARD,
    PROFILE_PHILOLOGICAL,
    PROFILE_DIAGNOSTICS,
    PROFILE_ECONOMICS,
    PROFILE_PIPELINE,
    PROFILE_FULL,
})


def validate_profile(profile: str) -> None:
    """Lève ``ValueError`` si ``profile`` n'est pas connu.

    Le runner appelle cette fonction au démarrage pour rejeter
    rapidement une faute de frappe utilisateur (``--profile philolagic``).
    """
    if profile not in KNOWN_PROFILES:
        raise ValueError(
            f"profil inconnu : {profile!r}. "
            f"Profils valides : {sorted(KNOWN_PROFILES)}"
        )


# ──────────────────────────────────────────────────────────────────────────
# Modèles de hook
# ──────────────────────────────────────────────────────────────────────────


@dataclass(frozen=True)
class DocumentMetricHook:
    """Hook calculé pour chaque document.

    Attributs
    ---------
    name:
        Identifiant lisible utilisé dans les logs et les warnings.
    attribute:
        Nom du champ du :class:`DocumentResult` à remplir (par
        exemple ``"confusion_matrix"`` ou ``"taxonomy"``). Doit
        correspondre exactement à un attribut existant — le runner
        passe le résultat via ``setattr``.
    profiles:
        Ensemble des profils dans lesquels ce hook s'active.
    func:
        Fonction calculant la métrique. Signature attendue :
        ``func(*, ground_truth, hypothesis, image_path, corpus_lang,
        ocr_result) -> Any``. Tous les arguments sont passés en
        keyword pour que les hooks puissent ignorer ceux qu'ils
        n'utilisent pas avec ``**_``.
    requires_success:
        Si ``True``, le hook n'est appelé que quand
        ``ocr_result.success`` (texte hyp non-vide). Évite de gaspiller
        du temps sur des documents en erreur OCR.
    requires_token_confidences:
        Si ``True``, le hook n'est appelé que quand
        ``ocr_result.token_confidences`` est non-vide. Réservé à la
        calibration (Sprint 42).
    """

    name: str
    attribute: str
    profiles: frozenset[str]
    func: Callable[..., Any]
    requires_success: bool = False
    requires_token_confidences: bool = False


@dataclass(frozen=True)
class CorpusMetricAggregator:
    """Agrégateur calculé une fois par moteur sur tous les documents.

    Attributs
    ---------
    name:
        Identifiant lisible.
    attribute:
        Nom du champ du :class:`EngineReport` à remplir (par
        exemple ``"aggregated_confusion"``).
    profiles:
        Profils dans lesquels l'agrégateur s'active.
    func:
        ``func(document_results: list[DocumentResult]) -> Any``.
    """

    name: str
    attribute: str
    profiles: frozenset[str]
    func: Callable[..., Any]


# ──────────────────────────────────────────────────────────────────────────
# Registres globaux
# ──────────────────────────────────────────────────────────────────────────


_DOCUMENT_HOOKS: list[DocumentMetricHook] = []
_CORPUS_AGGREGATORS: list[CorpusMetricAggregator] = []


def _check_profiles(profiles: Iterable[str]) -> frozenset[str]:
    frozen = frozenset(profiles)
    unknown = frozen - KNOWN_PROFILES
    if unknown:
        raise ValueError(
            f"profils inconnus : {sorted(unknown)}. "
            f"Profils valides : {sorted(KNOWN_PROFILES)}"
        )
    return frozen


def register_document_metric(
    *,
    name: str,
    attribute: str,
    profiles: Iterable[str],
    requires_success: bool = False,
    requires_token_confidences: bool = False,
) -> Callable[[Callable[..., Any]], Callable[..., Any]]:
    """Décorateur d'enregistrement d'un hook document-level."""
    profiles_set = _check_profiles(profiles)

    def decorator(func: Callable[..., Any]) -> Callable[..., Any]:
        # Idempotence : si le même func est ré-enregistré (ré-import
        # du module en test), on ignore silencieusement.  Si un autre
        # func tente le même ``name``, on lève.
        for existing in _DOCUMENT_HOOKS:
            if existing.name == name:
                if existing.func is func:
                    return func
                raise ValueError(
                    f"hook document '{name}' déjà enregistré par "
                    f"{existing.func.__module__}.{existing.func.__qualname__}"
                )
        _DOCUMENT_HOOKS.append(DocumentMetricHook(
            name=name,
            attribute=attribute,
            profiles=profiles_set,
            func=func,
            requires_success=requires_success,
            requires_token_confidences=requires_token_confidences,
        ))
        return func

    return decorator


def register_corpus_aggregator(
    *,
    name: str,
    attribute: str,
    profiles: Iterable[str],
) -> Callable[[Callable[..., Any]], Callable[..., Any]]:
    """Décorateur d'enregistrement d'un agrégateur corpus-level."""
    profiles_set = _check_profiles(profiles)

    def decorator(func: Callable[..., Any]) -> Callable[..., Any]:
        for existing in _CORPUS_AGGREGATORS:
            if existing.name == name:
                if existing.func is func:
                    return func
                raise ValueError(
                    f"agrégateur corpus '{name}' déjà enregistré par "
                    f"{existing.func.__module__}.{existing.func.__qualname__}"
                )
        _CORPUS_AGGREGATORS.append(CorpusMetricAggregator(
            name=name,
            attribute=attribute,
            profiles=profiles_set,
            func=func,
        ))
        return func

    return decorator


# ──────────────────────────────────────────────────────────────────────────
# Sélection + exécution selon profil
# ──────────────────────────────────────────────────────────────────────────


def select_document_hooks(profile: str) -> list[DocumentMetricHook]:
    """Retourne les hooks document-level actifs pour ``profile``.

    L'ordre d'enregistrement est préservé pour garantir que les
    warnings et logs apparaissent dans le même ordre qu'avant le
    chantier 2 (cf. test de rétrocompat).
    """
    validate_profile(profile)
    return [h for h in _DOCUMENT_HOOKS if profile in h.profiles]


def select_corpus_aggregators(profile: str) -> list[CorpusMetricAggregator]:
    """Retourne les agrégateurs corpus-level actifs pour ``profile``."""
    validate_profile(profile)
    return [a for a in _CORPUS_AGGREGATORS if profile in a.profiles]


def run_document_hooks(
    profile: str,
    *,
    ground_truth: str,
    hypothesis: str,
    image_path: str,
    corpus_lang: str,
    ocr_result: Any,
) -> dict[str, Any]:
    """Exécute tous les hooks document-level actifs pour ``profile``.

    Retourne un dict ``{attribute_name: value}`` que le runner peut
    appliquer au ``DocumentResult`` via ``setattr`` ou ``**kwargs``.

    Pré-conditions :
    - les hooks à ``requires_success=True`` ne tournent que si
      ``ocr_result.success`` ;
    - les hooks à ``requires_token_confidences=True`` ne tournent
      que si ``ocr_result.token_confidences`` est non vide.

    Toute exception levée par un hook est loggée en warning et
    le hook est sauté (``attribute`` absent du dict retourné). Aucun
    hook ne fait jamais échouer le calcul des autres — discipline
    historique préservée.
    """
    out: dict[str, Any] = {}
    for hook in select_document_hooks(profile):
        if hook.requires_success and not getattr(ocr_result, "success", False):
            continue
        if hook.requires_token_confidences and not getattr(
            ocr_result, "token_confidences", None,
        ):
            continue
        try:
            value = hook.func(
                ground_truth=ground_truth,
                hypothesis=hypothesis,
                image_path=image_path,
                corpus_lang=corpus_lang,
                ocr_result=ocr_result,
            )
        except Exception as exc:  # noqa: BLE001
            logger.warning(
                "[%s] fonctionnalité dégradée : %s", hook.name, exc,
            )
            continue
        if value is not None:
            out[hook.attribute] = value
    return out


def run_corpus_aggregators(
    profile: str,
    document_results: list,
) -> dict[str, Any]:
    """Exécute tous les agrégateurs corpus-level pour ``profile``.

    Retourne un dict ``{attribute_name: value}`` à appliquer au
    ``EngineReport``. Comme pour les hooks doc-level, une exception
    dans un agrégateur est loggée et l'agrégateur sauté.
    """
    out: dict[str, Any] = {}
    for agg in select_corpus_aggregators(profile):
        try:
            value = agg.func(document_results)
        except Exception as exc:  # noqa: BLE001
            logger.warning(
                "[aggregate_%s] fonctionnalité dégradée : %s", agg.name, exc,
            )
            continue
        if value is not None:
            out[agg.attribute] = value
    return out


# ──────────────────────────────────────────────────────────────────────────
# Helpers test-only
# ──────────────────────────────────────────────────────────────────────────


def _reset_for_tests() -> None:
    """Vide les registres globaux. **Réservé aux tests** — désactive
    toutes les métriques en production."""
    _DOCUMENT_HOOKS.clear()
    _CORPUS_AGGREGATORS.clear()


def _all_document_hook_names() -> list[str]:
    return [h.name for h in _DOCUMENT_HOOKS]


def _all_corpus_aggregator_names() -> list[str]:
    return [a.name for a in _CORPUS_AGGREGATORS]


__all__ = [
    "PROFILE_MINIMAL",
    "PROFILE_STANDARD",
    "PROFILE_PHILOLOGICAL",
    "PROFILE_DIAGNOSTICS",
    "PROFILE_ECONOMICS",
    "PROFILE_PIPELINE",
    "PROFILE_FULL",
    "KNOWN_PROFILES",
    "validate_profile",
    "DocumentMetricHook",
    "CorpusMetricAggregator",
    "register_document_metric",
    "register_corpus_aggregator",
    "select_document_hooks",
    "select_corpus_aggregators",
    "run_document_hooks",
    "run_corpus_aggregators",
]