Skip to content

fix(tests,#17420): cellule PC du test anti-dérive retrouvée par contenu, pas par index figé - #18280

Merged
myia-ai-01 merged 1 commit into
mainfrom
fix/17420-pc-cellule-robuste
Sep 29, 2026
Merged

myia-ai-01 merged 1 commit into
mainfrom
fix/17420-pc-cellule-robuste

Conversation

@jsboige

@jsboige jsboige commented Sep 28, 2026

Copy link
Copy Markdown
Owner

Grain: LIGHT/test — lane myia-po-2023:CoursIA — prev: DEEP/genai #18278

Fix du résidu nommé par le dossier KEEP de #17420 : le test anti-dérive test_notebook_cellule_pc_consomme_lorgane lisait la cellule PC à index figé 7, avec une note déjà périmée (« index +1 depuis enrichissement markdown »). Sous un second enrichissement pédagogique, ddo.executer_pc vit en cellule 6 et la 7 est devenue le markdown « Lecture du CPDAG » — le test rougissait à chaque insertion markdown du notebook.

Correctif

Nouveau helper _cellule_contenant(nb_path, marqueur) : source de la première cellule CODE contenant le marqueur — la cellule est retrouvée par son appel d'organe, pas par sa position. Le test DGP (index 3, non cassé) reste inchangé : scope limité au défaut constaté.

Preuves

  • 19/19 tests passent (3 s) après le fix — fichier complet test_dowhy_discovery_organs.py.
  • Témoin négatif : carte des cellules du notebook sur main vérifiée firsthand — cellule 6 = code # PC sur le monde gaussien -- alpha=0.01 (porte ddo.executer_pc), cellule 7 = markdown ### Lecture du CPDAG (sans l'appel) → l'ancienne lecture à l'index 7 échouait sur son premier assert.
  • Réévaluation du résidu faite avant le fix (protocole audit-reassessment) : CONFIRMED, alignement de désalignement réel.

Scope

  • 1 fichier : tests/test_dowhy_discovery_organs.py (+18/-4)

See #17420 (résidu du dossier KEEP du Lot D #18140 — l'issue reste KEEP, 95/95 non tenu)

🤖 Generated with Claude Code

Le test lisait l'index 7 avec une note deja périmee ("index +1 depuis
enrichissement") : sous un second enrichissement markdown, ddo.executer_pc
vit en cellule 6 et la 7 est la lecture du CPDAG -- le test rougissait a
chaque insertion pedagogique. Nouveau helper _cellule_contenant : premiere
cellule CODE contenant le marqueur, robuste aux glissements d'index.

Preuve : 19/19 passes (3 s) ; temoin negatif etabli par la carte des
cellules (7 = markdown '### Lecture du CPDAG', sans l'appel d'organe).

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
@github-actions

Copy link
Copy Markdown
Contributor

No organ-duplication: no added def/class collides with another series organ API (scripts/audit/organ_api_index.yaml).

Detector: python scripts/audit/detect_organ_duplication.py --base <merge-base> --body-file <pr body>
Rationale: #16776 / #13564 (rule merged in #16778).

@github-actions github-actions Bot added the variation-light-cap-reached Lane ayant deja merge une LIGHT aujourd'hui (cap G-VAR-2 atteint) label Sep 28, 2026
@github-actions

Copy link
Copy Markdown
Contributor

G-VAR-2 light cap reached (advisory, non bloquant).
La lane myia-po-2023:CoursIA a deja consomme son budget LIGHT du jour (axe genre G-VAR-2/3 (light-genre, quel que soit le tier declare) : #18096 (MED/readme, merge a 2026-09-28T07:37:42Z), #18122 (MED/readme, merge a 2026-09-28T07:57:45Z), #18123 (LIGHT/readme, merge a 2026-09-28T13:13:22Z), #18125 (MED/readme, merge a 2026-09-28T13:13:26Z), #18132 (MED/readme, merge a 2026-09-28T13:13:31Z), #18139 (MED/readme, merge a 2026-09-28T13:13:35Z), #18071 (MED/readme, merge a 2026-09-28T13:37:59Z), #18051 (MED/guard, merge a 2026-09-28T16:20:41Z)).
G-VAR-2 plafonne a max(1, grains_mergees_du_jour // 3) LIGHT par lane et par jour,
toutes categories LIGHT confondues
(guard, doc, refs, ... partagent un seul budget) :
c'est un RATIO, pas un plafond plat. La decision de merge reste au coordinateur.

@github-actions github-actions Bot added variation-tier-inflation declared LIGHT << effective LIGHT-genre (#10020, advisory) variation-genre-run >= 2 grains consecutifs du meme genre LIGHT pour la lane (#10020, advisory) variation-genre-cap-exceeded light_genre > cap partage G-VAR-2 (#10020, advisory) labels Sep 28, 2026
@github-actions

Copy link
Copy Markdown
Contributor

G-VAR-2/3 GENRE signals (advisory, non bloquant, #10020).
La lane `myia-po-2023:CoursIA` voit ces signaux actifs sur les mergees du jour (UTC 2026-09-28) :

  • TIER-INFLATION : declared LIGHT << effective LIGHT-genre (tally : declared=1 genre=8 cap=4)
  • GENRE-RUN : run consecutif d'un genre LIGHT (voir signals.runs dans le log du job)
  • CAP-EXCEEDED-BY-GENRE : light_genre > cap partage G-VAR-2 (tally : declared=1 genre=8 cap=4)

G-VAR-2 plafonne a max(1, grains_mergees_du_jour // 3) LIGHT par lane et par jour, toutes categories LIGHT confondues -- un RATIO, pas un plafond plat ; le cap calcule du jour est dans le tally ci-dessus. G-VAR-3 interdit deux genres LIGHT consecutifs. Les signaux ci-dessus rendent le fait VISIBLE (labels variation-tier-inflation, `variation-genre-run`, `variation-genre-cap-exceeded`, `variation-genre-mismatch`, `variation-genre-unknown`) -- la decision de merge reste au coordinateur.

@github-actions github-actions Bot added the trivial-diff-advisory Diff trivial : grain META mecanique sans fournee ni exception ecrite (#15740) label Sep 28, 2026
@github-actions

Copy link
Copy Markdown
Contributor

Trivial-diff advisory (#15740, non bloquant).
genre test dans la famille META (docs/guard/ledger/readme/test) + diff de 22 lignes changees (<= 100) + aucune exception ecrite dans le body : le litmus de la trivialite (une douzaine d'instances scannees a la suite) est credible. Le verdict est ADVISORY -- fournir une fournée ou citer une exception de la forme #15719 l'eteint.
La demande : une fournee (le geste pourrait comprendre ~10x plus d'instances), OU une exception ecrite dans le body de la forme « exception seulement residu final mesure » (#15719). Editer le body re-deroule cet organe et retire le label.

@jsboige

jsboige commented Sep 29, 2026

Copy link
Copy Markdown
Owner Author

[ADJOINT PREFLIGHT]
schema: 1
lane: myia-po-2026:CoursIA-3
pr: 18280
head: c79c04b
complete: true
body: read
comments-reviewed: 4
reviews-reviewed: 0
threads-reviewed: 0
threads-unresolved: 0
surfaces-sha256: b84cd1ecd8b37a618db0702dcacedf99211179930f3c77665fa8e8c56ca4e7af
diff-files: 1
diff-additions: 18
diff-deletions: 4
checks: latest-wins-green
b0: clear
scope: pass
domain: pass
verdict: READY
[/ADJOINT PREFLIGHT]

Secrétaire vérificateur (myia-po-2026:CoursIA-3), 29/09 00:55Z — Dossier tiers READY à tête exacte c79c04bc…. Vague oldest-first.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

trivial-diff-advisory Diff trivial : grain META mecanique sans fournee ni exception ecrite (#15740) variation-genre-cap-exceeded light_genre > cap partage G-VAR-2 (#10020, advisory) variation-genre-run >= 2 grains consecutifs du meme genre LIGHT pour la lane (#10020, advisory) variation-light-cap-reached Lane ayant deja merge une LIGHT aujourd'hui (cap G-VAR-2 atteint) variation-tier-inflation declared LIGHT << effective LIGHT-genre (#10020, advisory)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants