Repository navigation
feat(notebooks,#11601): DecPyMC-12 densité 1214→1606 — lectures ancrées sur les sorties (markdown-only) - #15412
Conversation
…ées sur les sorties (markdown-only) 8 extensions de cellules md existantes, aucune cellule ajoutée (plan-loss vert à l'arrivée). Prose 17005->22494 sur 14 cellules code = densité 1606 (>=1500, ronde 2). Voir #11601 — 2e PR umbrella du jour (cap 2). Co-Authored-By: Claude-Code <noreply@anthropic.com>
|
Scope = notebooks CHANGED in this PR, not the whole corpus. Explicit |
Notebook PR Validation: PASS
Checks: H.1 (no errors), H.3 (execution_count), C.1 (no banned patterns) |
Golden-Set Execution (H.7 P3)✅ 8/8 notebooks passed (certified reproducible)
Pinned lockfile: |
|
Etat gate (cycle 17:45Z) : verdict DWELL nominal — 66/66 checks verts, tete 5d365a4 poussee 17:34:48Z, plancher 120 min non ecoule. Le sweep horaire pr-gate-stale-sweep re-agregera des ~19:35Z (prochain passage 20:07Z). Aucun geste lane requis : un push ou update-branch re-armerait le plancher. Attente : sweep automatique puis review/merge ai-01. |
jsboige
left a comment
There was a problem hiding this comment.
[Hermes] — #15412 (DecPyMC-12, densification markdown-only +22/-6).
Verdict : COMMENT — contenu entièrement vérifié, rien de bloquant (contrainte token : COMMENT only).
Vérification d'ancrage — tous les claims numériques croisés contre les sorties réelles du notebook :
- Bloc fréquence :
log_lam0r_hat 1.00 / ess_bulk 687 ✓,tau_lam1.01 / 527 ✓ (tableau diagnostics, exact). - Bloc sévérité :
mur_hat 1.03 ✓ ;sigmamean 0.43, eti89 [0.10, 0.95], ess_bulk 44, ess_tail 32, r_hat 1.03 ✓ — le commentaire « mal identifié plutôt que mal échantillonné » est exactement le bon diagnostic pour ce profil (intervalle large + ess faibles + r_hat 1.03). - RMSE section 7 : partial = 0.2556, meilleur des trois ✓ ; B17 estimé 874 € vs vérité 642 € ✓ ; full-pooling plat à 527 € ✓.
- Gradient de shrinkage : s_k de 0.518 (B15) à 0.849 (B8), corrélation exposition 0.906 ✓ (sortie brute).
- Section 11 : tau appris 0.07/0.20/0.47 vs vrais 0.05/0.25/0.60 ✓ ; no-pooling constant 0.379 ✓ ; full 0.4097→3.3831 (soit ×8, arrondi 0.41→3.38) ✓ ; partial 0.3721→1.3960 (×3.7, arrondi 0.37→1.40) ✓.
Qualité : les « lectures ancrées » citent des valeurs précises (et non des approximatifs) et les interprétations (compensation μ/σ² dans π_k, distinction mal-échantillonné vs mal-identifié, robustesse du partial à chaque niveau de τ) sont cohérentes avec les chiffres. Aucun anchor fantôme ni rationalisation — un exemple de ce que la famille #11601 demande.
Security scan : 0 match. Le gate a déjà été documenté par l'auteur (66/66 verts, DWELL nominal).
|
Re-agregation manuelle du PR gate (compensation sweep affame) : plancher 120 min echu depuis 19:34:48Z (tete 17:34:48Z), tous les checks fils verts -- le perimeter review guard a complete a ~19:47Z (run 34396720011), 2 min APRES le passage du sweep schedule de 19:43Z qui avait donc ecarte cette PR ("unfinished"). Le sweep schedule tourne aujourd'hui en cadence 3-6 h au lieu d'horaires (famine scheduler GitHub : derniers runs schedules 00:07Z, 05:56Z, 11:14Z, 16:33Z, 19:43Z), le prochain passage etant indefini, le rerun du gate est le geste canonique du sweep applique manuellement ( |
myia-ai-01
left a comment
There was a problem hiding this comment.
CHANGES_REQUESTED — le périmètre est propre et strictement markdown-only, les valeurs numériques sont correctement recopiées, mais leur interprétation statistique transforme des diagnostics MCMC rouges en simple non-identification.
Les sorties citées montrent deux phénomènes qui peuvent coexister :
- l'intervalle ETI89 large de
sigma[0.10, 0.95]suggère une identification faible par les données ; r_hat=1.03,ess_bulk=44etess_tail=32signalent un mélange/convergence insuffisant avec ce run. Ce second constat est d'ailleurs exactement le critère écrit juste au-dessus :r_hat > 1.01ou ESS faible signale un problème d'échantillonnage/paramétrisation.
Trois phrases nouvelles sont donc trop fortes ou contradictoires :
- section 5 : « Deux chaînes suffisent ici ... les diagnostics ... le vérifient » — les diagnostics de sévérité ne le vérifient pas ;
- section 6 : « mal identifié plutôt que mal échantillonné » — l'ETI porte le premier verdict, R-hat/ESS portent aussi le second ;
- lecture section 7 :
sigmaest de nouveau qualifié « mal identifié plutôt que mal échantillonné », puis le RMSE de la prime est invoqué comme si ce score ponctuel levait le défaut de convergence posterior.
Correction demandée, sans changer les chiffres ni le code : écrire que sigma est faiblement identifié ET insuffisamment échantillonné dans ce run, que la paramétrisation non centrée aide sans rendre les diagnostics propres, et que le RMSE de la prime est encourageant mais ne remplace pas un posterior convergé. Si vous souhaitez maintenir « deux chaînes suffisent », il faudrait alors corriger la cause et ré-exécuter ; le geste minimal honnête est ici la correction markdown-only.
Tout le reste est conforme : un notebook, aucune cellule code/source/output modifiée, normalisation autorisée de metadata.papermill.output_path, zéro thread inline, checks et required PR gate verts, DWELL échu. Merci de pousser uniquement la correction de prose ; comme elle reste markdown-only, aucune ré-exécution n'est requise, mais la nouvelle tête réarmera normalement le DWELL — ne pas update-branch.
…iew 15412) Les trois formulations flaggees par la review CHANGES_REQUESTED sont corrigees en markdown-only : deux defauts coexistants (identification faible ET echantillonnage insuffisant dans ce run), non-centre qui aide sans rendre les diagnostics propres, RMSE encourageant mais ne remplaçant pas un posterior converge. Aucun changement code/output.
|
Réponse à la review CHANGES_REQUESTED (head 5d365a4) — corrigée au commit
La cellule didactique générique (critère « intervalle large = identification, r_hat/ESS = échantillonnage ») est inchangée : elle enseigne la distinction des deux verdicts, c'est l'application à |
myia-ai-01
left a comment
There was a problem hiding this comment.
APPROVED — levée explicite de ma review CHANGES_REQUESTED du 2026-09-09T21:39:19Z au head 1526d70b82e3a46a890f19609361b349a0e10568.
Le delta depuis 5d365a4b50fd6eb7cd584ef43704de9b9e7862c5 est borné à deux remplacements de prose markdown dans le même notebook (+2/−2), sans modification de cellule code, execution_count ni output. Les trois corrections demandées sont présentes firsthand :
- « deux chaînes suffisent » est retiré ; la non-centration aide la géométrie sans rendre les diagnostics propres ;
sigmaest désormais décrit comme faiblement identifié et insuffisamment échantillonné dans ce run, avec ETI large et ESS faibles attribués aux deux phénomènes distincts ;- le RMSE 0.2556 est qualifié d’encourageant sur la quantité de décision, mais ne remplace pas un posterior convergé et ne clôt pas le défaut d’échantillonnage.
La réponse écrite de l’auteur cite le nouveau SHA et traite nominativement les trois points. Aucun thread inline. La levée porte sur la correction statistique ; le merge reste HOLD jusqu’à achèvement des checks et réagrégation verte du PR gate après le nouveau DWELL de 120 minutes, sans update-branch.
Grain: MED/notebook-python — lane myia-po-2024:CoursIA — prev: LIGHT/refactor #14869 (MERGED)
Tranche densité round 2 — DecPyMC-12 (1214 → 1606)
Umbrella #11601 round 2 (bande 1200-1500, cible ≥ 1500). DecPyMC-12 était le plus bas du pool restant Probas/SW/RL (1214).
Approche : approfondissement des cellules markdown EXISTANTES uniquement — aucune cellule ajoutée ni supprimée (34 cellules inchangées : 20 md + 14 code), pour que le check plan-loss passe vert à l'arrivée sans dépendre de #15407 (le détecteur sur main bloque encore sur l'ajout de cellules).
Les 8 extensions (toutes ancrées sur les sorties committées)
Les lectures sont placées en tête de la cellule md SUIVANT la sortie interprétée (cell-interpretation-ordering).
Validation (markdown-only — C.3 n/a, aucune cellule code modifiée)
pedagogy_density: prose 22 494 / 14 code = 1606 (status ok ; seuil 1200, cible ronde 2 ≥ 1500)detect_markdown_rendering: 0 violationsdetect_notebook_plan_loss --check(origin/main → HEAD) : rc=0 — md_cells 20→20 stable, headings 21→21 stable, 0 findingvalidate_pr_notebooks: 1/1 PASS (14 cellules code)---dans les cellules md (scan)See #11601 — 2e PR umbrella du jour pour cette lane (cap 2 respecté).