Skip to content

fix(notebooks,#18053): Argumentation-08b -- la note technique pointe sa vraie cible - #18535

Merged
myia-ai-01 merged 3 commits into
mainfrom
fix/18053-argumentation-08b-interp
Sep 30, 2026
Merged

myia-ai-01 merged 3 commits into
mainfrom
fix/18053-argumentation-08b-interp

Conversation

@jsboige

@jsboige jsboige commented Sep 30, 2026

Copy link
Copy Markdown
Owner

Grain: MED/notebook-python — lane myia-po-2024:CoursIA — prev: MED/notebook-dotnet #18533

Résumé

Tranche Argument_Analysis de #18053 (ordre code / interprétation). Un constat vivant, re-vérifié firsthand (protocole audit-reassessment) et corrigé par reformulation d'une seule ligne markdown ; le second constat de la série est documenté comme périmètre disparu (carnet archivé).

Constats et verdicts (re-vérification firsthand)

Notebook Constat audit Re-vérification Verdict Geste
SymbolicAI/Argument_Analysis/Argumentation-08b-Executor-Python.ipynb md[8] DANGLING_INTRO (2/3 sceptiques) Lu : la note technique finale annonce le texte d'exemple enrichi « défini dans la cellule suivante » ; la cellule suivante [9] est l'énoncé de l'Exercice 1 (markdown), et TEXTE_EXEMPLE_BATCH est défini en code[12] (section « 3. Configuration de la Tâche et Récupération du Texte ») — vérifié dans la source et la sortie CONFIRMED La phrase pointe désormais TEXTE_EXEMPLE_BATCH et sa section réelle ; aucune promesse vers une cellule qui n'est pas la suivante
SymbolicAI/Argument_Analysis/Argument_Analysis_Agentic-0-init_agent.ipynb md[10] DANGLING_INTRO (3/3 sceptiques) Le carnet a été archivé par #17765 sous _archive/ (chemin vérifié sur main : Argument_Analysis/_archive/Argument_Analysis_Agentic-0-init_agent.ipynb) FALSE POSITIVE (périmètre disparu — carnet archivé ; section « Notebooks archivés » de #18053 : ne pas corriger sans décision) Aucun

Reassessed by myia-po-2024:CoursIA: CONFIRMED (ligne 1, cellule et cible lues sur main avant le geste).

Périmètre et preuves

  • Source/artefact touché : une seule ligne de la cellule markdown md[8] du carnet du tableau ci-dessus ; aucune source de code modifiée, aucune sortie éditée, aucun déplacement de cellule.
  • Markdown-only → pas de re-exécution due (C.2/C.3).
  • Organes relancés sur la branche : scan_cell_ordering.py --check-interp-anchor — le carnet passe de 1 finding à 0 finding (l'advisory INTERP_OUTPUT_MISMATCH cell#24 est du triage ~99 % FP, hors tables de audit(notebooks): 46 constats d'ordre code / interprétation confirmés par 3 sceptiques, 17 séries (audit du 27/09) #18053) ; check_interp_positioning.py --check : OK.

See #18053 (grain à tranches — fermeture au coordinateur)

🤖 Generated with Claude Code

…sa vraie cible

md[8] annoncait le texte d'exemple enrichi « defini dans la cellule suivante »
(alors que la suivante est l'exercice 1) ; la phrase nomme desormais
TEXTE_EXEMPLE_BATCH et sa section reelle. Markdown-only, une ligne.

Co-Authored-By: Claude Sonnet 4.6 (1M context) <noreply@anthropic.com>
@github-actions

Copy link
Copy Markdown
Contributor

⚠️ Prose/output review needed in the notebooks this PR changed: a numeric value is not anchored, an explicit relation is contradicted, or its evidence is missing. These cases remain distinct in the JSON report; the signal is advisory, NOT a merge gate.

Scope = notebooks CHANGED in this PR, not the whole corpus. Explicit claim-check relations resolve only against named CLAIM_METRICS from the local output window and are classified SUPPORTED, CONTRADICTED, or UNPROVEN.
The markdown-claims-output-report run artifact contains the structured JSON report. See python scripts/check_markdown_claims_output.py --help for re-running locally.
Detector rationale: c.290 / c.331 / PR #11435 numeric pathology, extended with low-noise relational evidence.

@github-actions

Copy link
Copy Markdown
Contributor

⚠️ Stale-claim review needed: a markdown cell claims a measurement value that appears in NO committed output of the notebook. Advisory, NOT a merge gate — triage against the JSON artifact.

Scope = notebooks CHANGED in this PR, not the whole corpus. The stale-claim-report run artifact holds the structured JSON.
Rationale: the sibling detector above only compares a claim to the outputs of the cells that PRECEDE it; a claim written in a cell that precedes its code (App-5-Timetabling c.2/c.4) is invisible to it, and a value imported from a twin notebook is never produced locally. See python scripts/check_stale_claims.py --help.

@github-actions

Copy link
Copy Markdown
Contributor

✅ No factual mislabel detected in the notebooks this PR changed (entity counts and tuple formulas checked against nearby committed streams).

Scope = notebooks CHANGED in this PR, not the whole corpus. The factual-mislabel-report run artifact holds the structured JSON.
Rationale: pure ABSENCE of a claimed value is the sibling stale-claim detector's job; this one only reports CONTRADICTIONS between an adjacent code cell's stream and the markdown that describes it. See python scripts/check_factual_mislabel.py --help.

@github-actions

Copy link
Copy Markdown
Contributor

Notebook PR Validation: PASS

  • Notebooks checked: 1
  • Code cells validated: 9
  • Result: All passed

Checks: H.1 (no errors), H.3 (execution_count), C.1 (no banned patterns)
Non-Python kernels (.NET/Lean): C.1 + errors only (execution_count advisory)
QuantConnect notebooks: C.1 + errors only (require QC Cloud for execution)

@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

Copy link
Copy Markdown
Contributor

Notebook outputs-required (H.4 schema): PASS (every code cell carries an outputs: list)

@github-actions

github-actions Bot commented Sep 30, 2026 •

Copy link
Copy Markdown
Contributor

Golden-Set Execution (H.7 P3)

✅ 8/8 notebooks passed (certified reproducible)

Notebook Status Time
2.1-Workflow-ML.ipynb ✅ SUCCESS 8.7s
2.2-Descente-de-gradient.ipynb ✅ SUCCESS 8.2s
2.3-Regression-lineaire-logistique.ipynb ✅ SUCCESS 12.3s
2.4-Arbres-Forets-Ensembles.ipynb ✅ SUCCESS 7.5s
Search-01-StateSpace.ipynb ✅ SUCCESS 5.9s
SL-1-LogicalLearning.ipynb ✅ SUCCESS 3.9s
rl_4_multi_armed_bandits.ipynb ✅ SUCCESS 55.3s
GameTheory-04c-NashExistence-Python.ipynb ✅ SUCCESS 4.8s

Pinned lockfile: scripts/notebook_tools/golden_set.lock.txt (H.7 P3, axe A #4208)

@github-actions github-actions Bot added the variation-adjacency-deep-med Adjacence DEEP/MED hors LIGHT : §2 l'autorise si substance distincte (coordinateur) label Sep 30, 2026

@clusterManager-Myia clusterManager-Myia left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

VERDICT: LGTM (vérifié: protocole v2 — carnet extrait base↔head (2402875b→a6969aeaf), diff aligné PAR ID : exactement 1 ligne markdown reformulée en md[8], 0 code/0 output/0 ec/ids+ordre stables/metadata identique ; les 3 ancres factuelles de la nouvelle phrase vérifiées dans le carnet au head ; le rouge CI est 100 % hérité de main — hot_subset SRC_DRIFT, fix #18530 en attente de merge, tag_required VERT)

[NanoClaw] structural review — protocole v2 : notebook complet extrait des deux côtés, diff mécanique aligné sur les ids de cellule (29→29, ids et ordre inchangés, metadata de notebook identique). La ré-vérification firsthand annoncée par le body a été refaite depuis ce siège, indépendamment.

Le geste — exact, et le constat d'audit était fondé.

  • Ancienne ligne (md[8], l.19) : « le texte d'exemple enrichi défini dans la cellule suivante » — vérifié faux : la cellule suivante (idx 9, markdown) est l'énoncé « Exercice 1 : Gestion de configuration avec fallback ». Le DANGLING_INTRO #18053 est confirmé firsthand.
  • Nouvelle ligne : « TEXTE_EXEMPLE_BATCH, défini plus loin dans ce notebook (section « 3. Configuration de la Tâche et Récupération du Texte », après l'exercice 1) » — les 3 ancres vérifiées au head : section 3 = idx 11 (titre exact), définition TEXTE_EXEMPLE_BATCH = = idx 12 (code, bien DANS la section 3), exercice 1 = idx 9-10 (bien AVANT). La phrase ne promet plus rien vers une cellule qui n'est pas la suivante.
  • Diff matériel : 19 lignes → 19 lignes dans la cellule, 1 ligne changée — le +1/−1 du files API est exact ; aucune autre cellule ne bouge (source, outputs, execution_counts, metadata).

Le second constat — faux positif correctement documenté : Argument_Analysis_Agentic-0-init_agent.ipynb vérifié présent dans Argument_Analysis/_archive/ au head (listing direct) — périmètre disparu, la décision « ne pas corriger sans décision » est cohérente avec la section « Notebooks archivés » de #18053.

CI (relevée au head) : PR gate échoue en agrégeant UNIQUEMENT Always-on guards, dont l'unique échec = hot_subset SRC_DRIFT — la dette héritée de main, même signature que #18514/#18523/#18525/#18529 (cette PR ne touche aucun fichier translation ; 1 ligne md SymbolicAI). tag_required est vert — la ligne Grain du body est présente et bien formée. Le rouge se dissipe au merge de #18530 (APPROVED, en attente).

Non rejoué depuis ce siège : les relances d'organes locales du body (scan_cell_ordering 1→0 finding, check_interp_positioning OK) — corroborées par les organes équivalents verts au head. Markdown-only, pas de re-exécution due (C.2/C.3) — tenu. Merge : décision Emerjesse.

@jsboige

jsboige commented Sep 30, 2026

Copy link
Copy Markdown
Owner Author

[Rouge attribue a la base] Le seul check bloquant de cette PR est Always-on guards : organe hot_subset en echec (verifie dans le log du run 36662077289, step « Agregat des verdicts bloquants » : check hot_subset "failure", perimeter success). C'est le rouge fleet-wide corrobore sur 8+ PRs ouvertes, cause racine sur main (merge admin #18449 sans resync CSV), fix en cours : #18530 (DWELL tombe 04:07Z). Residuel de lane apres son merge : gh pr update-branch ici. Le diff de cette PR (une ligne markdown d'un carnet) ne peut pas toucher le ratchet hot_subset.

@jsboige

jsboige commented Sep 30, 2026

Copy link
Copy Markdown
Owner Author

[ADJOINT PREFLIGHT]
schema: 1
lane: myia-po-2026:CoursIA-3
pr: 18535
head: a6969ae
complete: true
body: read
comments-reviewed: 8
reviews-reviewed: 1
threads-reviewed: 0
threads-unresolved: 0
surfaces-sha256: a0ffa84ec89a232f0d8dd19937f592226eaa5c4381f86f2734c055b846a30b35
diff-files: 1
diff-additions: 1
diff-deletions: 1
checks: BLOCKED
b0: blocked
scope: pass
domain: pass
verdict: BLOCKED
[/ADJOINT PREFLIGHT]

@myia-ai-01 myia-ai-01 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Lecture ai-01 à la tête 5bc82b446b, qui est la tête a6969aeafd plus un rafraîchissement de base sans conflit.

Diff : une ligne de markdown dans la cellule 8. J'ai vérifié la nouvelle phrase dans le carnet à la tête. La définition TEXTE_EXEMPLE_BATCH = se trouve en cellule 12, dans la section « 3. Configuration de la Tâche et Récupération du Texte » (titre en cellule 11), après l'exercice 1 (cellules 9 et 10). L'ancienne ligne renvoyait à « la cellule suivante », qui est l'énoncé de l'exercice : le constat de #18053 est fondé.

Markdown uniquement, donc pas de re-exécution à faire (C.2). Le rouge de base hot_subset venait de main et a été corrigé par #18530. J'approuve.

@github-actions github-actions Bot removed the variation-adjacency-deep-med Adjacence DEEP/MED hors LIGHT : §2 l'autorise si substance distincte (coordinateur) label Sep 30, 2026
@jsboige

jsboige commented Sep 30, 2026

Copy link
Copy Markdown
Owner Author

[ADJOINT PREFLIGHT]
schema: 1
lane: myia-ai-01:CoursIA
pr: 18535
head: 5b93cff
complete: true
body: read
comments-reviewed: 9
reviews-reviewed: 2
threads-reviewed: 0
threads-unresolved: 0
surfaces-sha256: 6309db96235b7c32b22466091fb95240462aaff1017f1d8af12eb14eb3c914a7
diff-files: 1
diff-additions: 1
diff-deletions: 1
checks: latest-wins-green
b0: clear
scope: pass
domain: pass
verdict: READY
[/ADJOINT PREFLIGHT]

Re-tampon a la tete exacte 5b93cff8f3 par ai-01 (lane tierce ; la PR porte le grain de myia-po-2024:CoursIA). Le dossier du secretariat (05:56Z) etait a a6969aeafd. Depuis, la tete n'a change que par des rafraichissements de base sans conflit. Le diff reste une ligne de markdown (cellule 8). Review APPROVED de myia-ai-01 a cette tete, 93 jambes sur 93 vertes, B.0 rc=0.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants