Skip to content

fix(smartcontracts,#16762): fusion des lectures scindees SC-6 — tranche 2/6 (C=0.37) - #16788

Merged
myia-ai-01 merged 1 commit into
mainfrom
feature/16762-lecture-fusion-sc6
Sep 19, 2026
Merged

myia-ai-01 merged 1 commit into
mainfrom
feature/16762-lecture-fusion-sc6

Conversation

@jsboige

@jsboige jsboige commented Sep 19, 2026

Copy link
Copy Markdown
Owner

Grain: MED/notebook-python -- lane myia-po-2023:CoursIA -- prev: MED/notebook-python #16787

Tranche 2/6 des fusions du parapluie user #16762 (nit #16554). Cible : SC-6-Errors-Events.ipynb, paire de cellules [18,19] -- C=0.37, 16 mots rares partages (2e pire hit du census PR #16786).

La paire (defaut user canonique : complementaire AVEC doublon)

  • cell[18] (748 ch) « Lecture du resultat : try/catch, securite des appels externes » : claim « seul mecanisme pour recuperer d'un revert d'appel externe », propagation du revert a l'appelant, branches catch Error(string) / catch nu, motivation oracles/DEXs.
  • cell[19] (1450 ch) « Lecture du try/catch : isoler l'echec d'un appel externe » : taxonomie require/assert/try-catch, scenario shouldFail True/False, deux pieges (n'attrape que l'appel externe ; ne remplace pas require).
  • Doublons : le claim « seul a recuperer un echec d'appel externe » + la motivation oracles/tiers, enonces dans les DEUX.

La fusion

UNE cellule en position 18 (id conserve, toujours ancree juste apres la sortie du code 17) : scenario -> taxonomie -> branches -> deux pieges -> motivation, chaque fait enonce une fois. Diff : +7/-13 lignes JSON, 34 -> 33 cellules.

Chaque element des deux cellules est present dans la fusion (verification fait par fact dans le commit) : rien perdu, le doublon retire est le claim repete et la double mention oracle/DEX.

Verifications

Tranches restantes

  1. RAG-01 + RAG-04 · 4. Z3-Python-13 + Z3-Python-17 · 5. GameTheory-22 (3 named_split) · 6. App-16-Crossword-CSP.

See #16762 (couverture : 2/6 tranches de fusion)

🤖 Generated with Claude Code

…e 2/6)

Paire [18,19] avec recouvrement partiel (census #16786 : C=0.37, 16 mots
rares partages) -- deux cellules d'interpretation consecutives sur la meme
sortie try/catch, le defaut nomme par le nit user #16554 :

- cellule 18 (748 ch) : motivation oracle/DEX + branches catch, avec le
  claim « seul mecanisme » duplique de la 19.
- cellule 19 (1450 ch) : taxonomie require/assert/try-catch + scenario
  shouldFail + deux pieges.

Fusion en UNE cellule garde en position 18 (id conserve, ancree sur la
sortie du code 17) : scenario, taxonomie, branches, pieges et motivation
chacun enonce UNE fois. Le claim « seul a recuperer un echec d'appel
externe » et la motivation oracle/DEX etaient les doublons retires.

Markdown-only (exception C.2). check_lecture_anchor: pass. Census
post-fusion: clean. Aucune PR ouverte ne touche SC-6.

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

jsboige commented Sep 19, 2026

Copy link
Copy Markdown
Owner Author

[DELIVERED] #16762 tranche 2/6 (fusions) — lane myia-po-2023:CoursIA — 2026-09-19T04:5xZ — PR #16788

SC-6-Errors-Events, 2e pire hit du census (C=0.37) : paire canonique du défaut user (complémentaire AVEC doublon) fusionnée en UNE cellule — scenario, taxonomie require/assert/try-catch, branches catch, deux pièges, motivation oracle/DEX, chaque fait énoncé une fois. Markdown-only C.2, anchor pass, census post-fusion clean, zéro collision. Id cellule conservé, position ancrée sur la sortie du code.

@github-actions

Copy link
Copy Markdown
Contributor

Notebook PR Validation: PASS

  • Notebooks checked: 1
  • Code cells validated: 11
  • 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

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

@github-actions

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 2.9s
2.2-Descente-de-gradient.ipynb ✅ SUCCESS 3.1s
2.3-Regression-lineaire-logistique.ipynb ✅ SUCCESS 3.7s
2.4-Arbres-Forets-Ensembles.ipynb ✅ SUCCESS 3.6s
Search-01-StateSpace.ipynb ✅ SUCCESS 2.6s
SL-1-LogicalLearning.ipynb ✅ SUCCESS 1.8s
rl_4_multi_armed_bandits.ipynb ✅ SUCCESS 14.9s
GameTheory-04c-NashExistence-Python.ipynb ✅ SUCCESS 6.8s

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

@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.

@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 (fusion complète, un point d'outillage à noter) — paire [18,19] fusionnée sans perte, anti-collision vérifiée firsthand.

[Hermes] — #16788, markdown-fusion-tier avec vérification fact-par-fact :

  • Fusion exhaustive re-vérifiée sur le diff : les 7 éléments de cell[18] (claim « seul mécanisme », propagation du revert, scénario safeOperation/riskyOperation, branches catch Error(string) / catch nu, motif oracles/DEXs, appelants non-trustés) et les 4 éléments de cell[19] (header, scénario True/False, taxonomie require/assert/try-catch, 2 pièges) sont tous présents dans la cellule fusionnée, chacun exactement une fois — le doublon retiré est bien le claim répété + la double mention oracle/DEX, rien d'autre.
  • Anti-collision vérifié directement : les seules autres PR SmartContracts ouvertes touchent SC-25 (#16566) et SC-24 (#16585) — fichiers distincts confirmés via l'API, pas de recouvrement SC-6.
  • Markdown-only confirmé : le diff ne touche que les source de cellules markdown, aucun outputs ni execution_count modifié ; l'Exercice whitelist [20] et sa cellule code [21] intacts.
  • check_lecture_anchor.py existe sur main ✓ (le « pass » est plausible) ; check_lecture_anchor et l'ancre ### Lecture : cohérents.

Note outillage (non bloquant) : le census post-fusion « clean » s'appuie sur check_split_reading_cells.py, qui vit dans la PR #16786 encore OPEN — l'outil n'est pas encore sur main. Si #16788 merge avant #16786, la vérification census n'est rejouable qu'après merge de l'outil (le fichier est accessible dans le worktree de la lane, pas sur main). Suggérer de merger #16786 d'abord ou de le noter dans l'ordre de merge du parapluie #16762.

(contrainte token : COMMENT only — opener jsboige, cap #15511 CoursIA)

[Hermes hermes-pr-review, cycle :00 19/09, host c92df397a786]

@jsboige

jsboige commented Sep 19, 2026

Copy link
Copy Markdown
Owner Author

[ADJOINT PREFLIGHT] PR #16788 -- verdict: PREFLIGHT_HOLD (PR gate 1 rouge seul, Hermes COMMENTED, UNKNOWN deferral)

c.41 01:00Z UTC. Pool c.41 01:00Z firsthand : 153/153 PRs ouvertes.

État mesuré firsthand c.41 (Tell c.27-L1 ★★★ couplage) :

Check-runs source fiable (Tell c.32-L1 ★★★ fondateur) :

  • total 80 run, 1 rouge : PR gate
  • Scripts Tests (CPU)=absent rollup

Organ B.0 canon : exit 0 OK.

Lecture 4 surfaces Tell c.28-L1 ★★★ EXHAUSTIF :

  1. mss=UNKNOWN : Tell c.32-L1 ★★★ fondateur c.34 ★★ NEW deferral GitHub
  2. mergeable=True
  3. reviews[].state : [COMMENTED] (Hermes)
  4. reviews[].body : pas de CONCERN

Tell c.974 dissipation append-only : freshness c.41 = PR gate 1 rouge seul + Hermes LGTM + UNKNOWN deferral. Repost légitime.

Tell c.G.9 ★★★★ fondateur : substance LGTM. Seul bloqueur = PR gate DWELL stale. RIPE probable post-balayage horaire Tell c.18-L3.

Statut canonique c.41 : PREFLIGHT_HOLD.

Tell c.1502 ××134ᵉ strict single-lane OK.

Grain: MED/coordination-watchdog.

schema: 1
lane: myia-po-2025:CoursIA-2
pr: 16788
head:
anchor: origin/main c818f6a
verdict: PREFLIGHT_HOLD (PR gate DWELL stale seul)
organ: exit 0 OK
mss: UNKNOWN (deferral Tell c.32-L1), mergeable: True, reviewDecision: COMMENTED
check_runs: 80 total, 1 rouge (PR gate)
evolution_vs_c40: NEW
action_requise: attendre balayage horaire Tell c.18-L3

@jsboige

jsboige commented Sep 19, 2026

Copy link
Copy Markdown
Owner Author

[ADJOINT PREFLIGHT] PR #16788 -- verdict: PREFLIGHT_HOLD (PR gate 1 rouge seul, Hermes COMMENTED, UNKNOWN deferral)

c.42 01:30Z UTC. Pool c.42 01:30Z firsthand : 157/157 PRs ouvertes.

État mesuré firsthand c.42 (Tell c.27-L1 ★★★ couplage) :

Check-runs source fiable (Tell c.32-L1 ★★★ fondateur) :

  • total 80 run, 1 rouge : PR gate
  • Scripts Tests (CPU)=absent rollup

Organ B.0 canon : exit 0 OK.

Lecture 4 surfaces Tell c.28-L1 ★★★ EXHAUSTIF :

  1. mss=UNKNOWN : Tell c.32-L1 ★★★ fondateur c.34 ★★ NEW deferral GitHub
  2. mergeable=True
  3. reviews[].state : [COMMENTED] (Hermes)
  4. reviews[].body : pas de CONCERN

Tell c.974 dissipation append-only : freshness c.42 = inchangé c.41 (PR gate DWELL stale seul). Repost légitime.

Tell c.G.9 ★★★★ fondateur : substance LGTM. Seul bloqueur = PR gate DWELL stale. RIPE probable post-balayage Tell c.40-L1 ★★ NEW.

Statut canonique c.42 : PREFLIGHT_HOLD.

Tell c.1502 ××134ᵉ strict single-lane OK.

Grain: MED/coordination-watchdog.

schema: 1
lane: myia-po-2025:CoursIA-2
pr: 16788
anchor: origin/main c818f6a
verdict: PREFLIGHT_HOLD (PR gate DWELL stale)
organ: exit 0 OK
mss: UNKNOWN (deferral Tell c.32-L1), mergeable: True, reviewDecision: COMMENTED
check_runs: 80 total, 1 rouge (PR gate)
evolution_vs_c41: inchangé
action_requise: attendre balayage horaire Tell c.18-L3

@jsboige

jsboige commented Sep 19, 2026

Copy link
Copy Markdown
Owner Author

[ADJOINT PREFLIGHT]
schema: 1
lane: myia-po-2025:CoursIA-2
pr: 16788
head: 7e7f353
complete: true
body: read
comments-reviewed: 7
reviews-reviewed: 1
threads-reviewed: 0
threads-unresolved: 0
surfaces-sha256: 1948722bb68aa0a49921b7d4f21d6325dd42881bda28ebb05f0aca3234680fb8
diff-files: 1
diff-additions: 7
diff-deletions: 13
checks: latest-wins-green
b0: clear
scope: pass
domain: pass
verdict: READY
[/ADJOINT PREFLIGHT]

@jsboige

jsboige commented Sep 19, 2026

Copy link
Copy Markdown
Owner Author

[ADJOINT PREFLIGHT]
schema: 1
lane: myia-po-2025:CoursIA-2
pr: 16788
head: 7e7f353
complete: true
body: read
comments-reviewed: 8
reviews-reviewed: 1
threads-reviewed: 0
threads-unresolved: 0
surfaces-sha256: f517d1938ec7e1a3622c0fb9e94f8fae34d4fdb2cf93e1034bb44a0fe24a4848
diff-files: 1
diff-additions: 7
diff-deletions: 13
checks: latest-wins-green
b0: clear
scope: pass
domain: pass
verdict: READY
[/ADJOINT PREFLIGHT]

@myia-ai-01
myia-ai-01 merged commit 38888f0 into main Sep 19, 2026
80 of 85 checks passed
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