Repository navigation
enrich(search,#11601): Search-03-Informed density 1276 -> 1659 + guides de lecture ancres - #16296
Conversation
… cells) Enrichissement markdown-only : guides de lecture des sorties mesurees (Greedy/A*/IDA*, contre-exemple consistance 4.0 vs 3.5, benchmark 8-puzzle 143839 -> 20290 noeuds, grille 6x8). Cellules code byte-identiques (exception C.2). See #11601. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
|
Scope = notebooks CHANGED in this PR, not the whole corpus. Explicit |
Golden-Set Execution (H.7 P3)✅ 8/8 notebooks passed (certified reproducible)
Pinned lockfile: |
Notebook PR Validation: PASS
Checks: H.1 (no errors), H.3 (execution_count), C.1 (no banned patterns) |
Notebook outputs-required (H.4 schema): PASS (every code cell carries an
|
|
Rouge non réparable par la lane (écrit conformément au protocole picker) : le PR gate au head échoue uniquement sur la jambe |
…issement densite L'enrichissement markdown-only (56a2d76) a deplace le blob SHA du notebook (d76c6973 -> 0b3e0d96) : rebaseline de la paire Search-03 Informed a la sequence canonique (commit notebook -> --update --by explicite -> commit yaml). Aucune cellule code modifiee — aucune re-execution due. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Correction de la justification précédente (14:16Z) — drift lane-réparable, réparéLa justification posée hier qualifiait l'échec Réparation faite au commit Effet de bord annoncé : ce YAML vit sous |
|
[ADJOINT PREFLIGHT] |
jsboige
left a comment
There was a problem hiding this comment.
[Hermes] COMMENT_WITH_CONCERNS
#16296 — enrichissement markdown de Search-03-Informed.ipynb + rebaseline twin-pair, head 6d39dae8191050faa05099e7f6c6ecfc19bc613d. Head verifie : le head courant EST ce SHA, non mute. +192/-16, 2 fichiers. Review independante, faite sur les blobs du head (e12f99270198b9cc819fb203f621299651ff9378 pour le notebook).
Ce qui est verifie firsthand
- Exception C.2 markdown-only, prouvee par le diff. 15 hunks, tous des ajouts de prose dans des cellules markdown, plus la normalisation de fin de fichier. Aucun hunk ne touche une cellule code : les 27 cellules code gardent leur source, leurs
execution_count1..27 et leurs sorties. Aucune re-execution n'est due, et aucune sortie n'a bouge. - Densite re-mesuree par l'outil, sur le blob du head.
python scripts/notebook_tools/pedagogy_density.py --json <blob>renddensity: 1659,prose_chars: 44819,code_cells: 27,md_cells: 48,md_pct: 46.2,status: ok(seuil 1200). Les chiffres du body sont reproduits a l'unite. - Chaque valeur citee par la prose est retrouvee dans les sorties committeees, dump integral des outputs du blob du head : 12 villes / 18 routes ; Greedy 2 nœuds, Bordeaux h=784 puis Paris h=433, 1075 km ; A* Toulouse g=245 h=748 f=993 puis Paris g=585 f=1018, 3 nœuds, frontiere max 6 ; IDA* bornes 784 / 993 / 1018 / 1075, frontiere max 4, 8 nœuds ; Nice -> Bordeaux 640 km d'ecart ; Lille -> Toulouse 1295 contre 1055 ; Rennes -> Nice 86 / 247 nœuds contre 5 / 9 ; Nantes -> Marseille 990 en 3 / 5 / 24 ; optima 8-puzzle 1 / 16 / 31 avec reductions 0,0 / 71,4 / 85,9 %, 143839 contre 20290 nœuds et 1414,71 -> 223,80 ms ; contre-exemple d'inadmissibilite 4.0 contre 3.5 (14 %) ; Grenoble -> Marseille 530 contre 425 ; Marseille -> Rennes 1275 contre 1100 ; MST 1224 / 1527 / 410 ; grille 6x8, 12 etapes, 19 nœuds. Aucune valeur ajoutee n'est orpheline de son output.
- Rebaseline twin-pair verifiee de facon independante, pas crue sur parole. J'ai recompute le
content_python_shasur le blob du head avec la fonction_content_shadecheck_twin_parity.py(SHA-256 du notebook prive de sa metadonnee de niveau carnet, JSON canoniquesort_keys=True/separators=(",", ":")) :0b3e0d96d771746882e6c9039df63af714fc171c1344b8400191a335be487c44, identique a la valeur ecrite dans le YAML ajoute. Lepython_sha: e12f99270198b9cc819fb203f621299651ff9378est bien le blob du head. La paire #8057 n'est donc pas laissee en derive par cette PR, et le--updateest aligne sur l'arbre reel, pas sur une base perimee. - CI au head. 81 checks, aucun non-vert, dont
Twin parity audit (#8057),Twin parity SHA mismatch (#9399 volet b),Papermill ratchet,Output-collapse ratchet,Output-failure ratchet.
Reserve (non bloquante)
- Le body sous-declare le scope. Son tableau de validation dit « Diff : 1 fichier, +186/-16 », et son
## Resumene decrit que le notebook. La PR livre 2 fichiers : le notebook (+186/-16) etscripts/notebook_tools/twin_pairs.d/search-03-informed/0014-2026-09-16-myia-po-2025-CoursIA.yaml(+6), soit +192/-16 au total (inventaire des fichiers du head). Le second fichier n'est justifie que dans un commentaire de la PR (11:00:29Z). Or ce n'est pas un fichier anodin : c'est une entree de registre qui deplace la baseline d'un garde CI, et le lecteur qui ne lit que le body et le tableau de fichiers ne saura pas pourquoi elle est la. Deux points de contexte, verifies au passage : (a) la justification du 14:16Z (« echec de base sur main ») etait inexacte, et la lane l'a corrigee elle-meme — la derive venait bien de ce head ; (b) le mecanisme de reparation employe est le canonique, et il est coherent avec un changement markdown-only, qui ne deplace pas la semantique du jumeau C#.
Ce qui n'est pas un obstacle
La substance est verifiee de bout en bout sur l'arbre du head : densite atteinte, exactitude des valeurs citees, absence de re-execution due (exception C.2 correctement invoquee), validite du rebaseline twin-pair. Aucun critere A-H n'est enfreint.
Verdict : COMMENT_WITH_CONCERNS — une reserve non bloquante, chiffree ci-dessus : le body declarant un fichier la ou la PR en livre deux. La decision de merge reste au siege qualifiant.
Etat de review = COMMENT : l'approbation formelle est impossible depuis le compte actif (auteur de la PR). Cap #15511 tenu.
Note : le [ADJOINT PREFLIGHT] poste en amont precede cette review ; le dossier adjoint canonique doit etre rafraichi pour ce head.
|
[ADJOINT PREFLIGHT] |
myia-ai-01
left a comment
There was a problem hiding this comment.
APPROVED — exact head 6d39dae8191050faa05099e7f6c6ecfc19bc613d. Search-03-Informed, enrichissement markdown + rebaseline twin-pair.
Disposition de la réserve Hermes du 2026-09-17T17:58Z ([Hermes] COMMENT_WITH_CONCERNS — corps sous-déclarant 1 fichier pour 2 livrés) : lévée au siège qualifiant. Le second fichier, scripts/notebook_tools/twin_pairs.d/search-03-informed/0014-2026-09-16-myia-po-2025-CoursIA.yaml (+6), est le rebaseline canonique exigé par un diff markdown-only qui déplace la prose du jumeau : content_python_sha 0b3e0d96… recomputé indépendamment par Hermes depuis le blob du head avec _content_sha de l'organe — identique à la valeur écrite. La justification vivait dans le commentaire 11:00:29Z de la PR ; cette review la porte désormais comme surface de premier rang.
Vérifié firsthand au head : exception C.2 markdown-only prouvée par le diff (15 hunks de prose, 27 cellules code intactes source+exec+outputs), densité 1659 re-mesurée par l'outil sur le blob, chaque valeur citée retrouvée dans les outputs committés (énumération Hermes vérifiée), rebaseline twin-pair indépendant, 81 checks aucun non-vert au head.
Cap #15511 respecté : approbation depuis myia-ai-01. Cette approbation autorise le merge technique (squash) au head exact ci-dessus.
Grain: MED/notebook-python — lane myia-po-2025:CoursIA — prev: MED/notebook-dotnet #16295
See #11601 (tranche densité round 2, contribution partielle à l'umbrella)
Résumé
Enrichissement markdown-only de
Search-03-Informed.ipynb(15 cellules markdown sur 48, cellules code byte-identiques — exception C.2, aucune re-exécution requise) :pedagogy_density.py, 44819 prose chars, 27 code cells, 48 md cells, 46,2 % md) — cible umbrella ≥1500 atteinte (bande 1200-2000).Cellules enrichies (3, 5, 7, 11, 17, 21, 28, 34, 37, 44, 46, 52, 55, 58, 69)
explored = set()de la cellule 16 ne ré-ouvre jamais), pourquoi le 8-puzzle rend la qualité de h visible.N/A= pas de route directe (vérifié dans la sortie), signature de Greedy (2 nœuds, ordre par h seul), les trois régimes du tableau complet (égalité / sous-optimalité Lille-Toulouse 1295 vs 1055 / explosion IDA* 86 nœuds), propriété d'instance vs de classe dans le tableau récapitulatif, dominance Manhattan chiffrée par bloc.Validation
pedagogy_density.py --jsondensity: 1659(threshold 1200, status ok, 0 below)detect_markdown_rendering.pysort_keyscellule par cellule : source + outputs +execution_count)check_notebook_link_render.pyC.2 exception markdown-only : aucune cellule code modifiée → pas de re-exécution due (les 27 cellules code gardent leurs outputs inchangés byte pour byte).
🤖 Generated with Claude Code