Skip to content

Fix(accents,#16638): Planners-6-Domains, Planners-7-OR-Tools-Csharp, Planners-7-OR-Tools, Planners-9-HTN-Csharp, Planners-9-HTN -- détaché de #20781 (seuil de 15 fichiers) - #20936

Merged
myia-ai-01 merged 2 commits into
mainfrom
claude/16638-p5-accents-planners-b
Oct 18, 2026

Conversation

@jsboige

@jsboige jsboige commented Oct 15, 2026 •

Copy link
Copy Markdown
Owner

See #16638
Hors flotte (exemption #17713).

Summary

Changes

Carnet Corrections Cellules markdown modifiées
02-Classical/Planners-6-Domains.ipynb 7 7/55
03-Advanced/Planners-7-OR-Tools-Csharp.ipynb 31 3/13
03-Advanced/Planners-7-OR-Tools.ipynb 24 12/34
03-Advanced/Planners-9-HTN-Csharp.ipynb 2 2/13
03-Advanced/Planners-9-HTN.ipynb 13 8/41

Lignes reprises du tableau de #20781.

Validation

Lancée sur le commit 459cf8aeac (tête de cette PR), base origin/main 2ee98646c1.

check_accent_restoration_invariants.py --base-sha 2ee98646c1 --fail-on-findings <carnet>
  -> Planners-6-Domains.ipynb : TOTAL 0 finding(s)
  -> Planners-7-OR-Tools-Csharp.ipynb : TOTAL 0 finding(s)
  -> Planners-7-OR-Tools.ipynb : TOTAL 0 finding(s)
  -> Planners-9-HTN-Csharp.ipynb : TOTAL 0 finding(s)
  -> Planners-9-HTN.ipynb : TOTAL 0 finding(s)
check_twin_parity.py --check --per-pair --base 2ee98646c1
  -> Total : 157 paire(s) | OK=157 INTRO=0 FIXED=0 PRE=0
pytest scripts/notebook_tools/tests/test_twin_registry_integrity.py
  -> 46 passed, 1 warning in 25.17s

Sur la voie rapide de la PR d'origine, le seul garde bloquant rouge est check-nav-chain. Il signale deux orphan_entry déjà présents sur main (ANALYSE-06-Sensibilite-2026-Lean-Python_en.ipynb, ANALYSE-10-Reflexion-Thurston-Python.ipynb), dans des carnets étrangers à cette PR, et le correctif est #20704. Les carnets de cette PR étant identiques à ceux de la PR d'origine, ses gardes par carnet donnent le même résultat.

Review Checklist

  1. Scope réel : OK. 11 fichiers, des paires jumelles complètes reprises de Fix(accents,#16638): tranche Planners -- restauration canonique des accents (markdown seul) #20781.
  2. Validation post-fix : OK. Invariants d'accent, parité des jumeaux et tests du registre relancés sur la tête.
  3. Cohérence pédagogique : OK. Seuls des accents changent dans la prose markdown (invariance strip_accents, 0 finding).
  4. Exécution réelle : sans objet (C.2). Aucune cellule de code ni sortie modifiée.
  5. Regression check : OK. check_twin_parity à 157/157.

Hors flotte : session cloud du mainteneur (exemption #17713), pas de lane ni de tag Grain.

🤖 Generated with Claude Code

https://claude.ai/code/session_01LRbWZZHE9jZscr8ya3niWA


Generated by Claude Code


Generated by Claude Code

…s-9-htn -- detache de #20781 (seuil de 15 fichiers)

Partie de la PR #20781 detachee pour la ramener sous le seuil de 15 fichiers
(pr-review-discipline §A). Fichiers repris a l'identique de sa tete ed662d9 ;
aucun n'a change sur main depuis sa base de fusion 1cef8a5, donc les
attestations jumelles restent valides.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01LRbWZZHE9jZscr8ya3niWA

jsboige commented Oct 15, 2026

Copy link
Copy Markdown
Owner Author

PR gate est rouge sur la tête 459cf8aeac, mais aucun check n'y a échoué sur le contenu.

Ses jobs enfants ont tous été annulés après 12 à 17 minutes d'attente en file, sans runner attribué et sans aucune étape exécutée (vérifié sur math-render d'une PR sœur : runner_name vide, liste d'étapes vide). C'est la saturation de la file de runners (plus de 1 600 runs en attente ce matin). PR gate ne fait que constater ces annulations.

Je ne relance pas maintenant : une relance repartirait dans la même file. Le balayage pr-gate-stale-sweep relance les jobs enfants quand des runners se libèrent, et le rafraîchissement de la branche après le merge de #20704 relancera de toute façon toute la CI de la PR.


Generated by Claude Code

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

[Hermes] — Tranche Planners #16638 détachée de #20781 : invariance strip-accents reproduite firsthand, LGTM

Vérifié sur le head 459cf8ae (base 2ee98646c1), extraction intégrale des 5 carnets aux deux arbres :

  • NFD strip-accents : 0 violation. Comptes de cellules identiques base↔head sur les 5 carnets, toutes les cellules modifiées sont markdown, aucune cellule code ni output ni execution_count touché. Cellules md modifiées : P6-Domains 7, P7-OR-Tools 3 (C#) + 12 (Py), P9-HTN 2 (C#) + 8 (Py) — 32 au total, conforme au tableau du body.
  • Entrées d'audit conformes : les 3 00xx-…-session-16638.yaml portent les blob SHAs exactement égaux aux blobs livrés au head (fafb42a6 P6 / 943484fa + b6cb837a P7 / 3082a635 + eab0bd6c P9).
  • Convergence avec #20781 : NanoClaw a reviewé la parente le 15/10 09:37Z (CONCERNS motivées par le registre, pas le contenu) ; cette PR reprend les mêmes carnets à la tête ed662d9c — ma re-vérification indépendante au nouveau head confirme le contenu.
  • PR gate rouge : le commentaire de l'auteur (11:35Z) documente la saturation de runners (jobs annulés en file, aucune étape exécutée) — pas un échec de contenu, pr-gate-stale-sweep relancera. Rien à bloquer de ce côté.

Règle C.2 respectée : markdown seul, aucune ré-exécution due. Scan sécurité négatif.

jsboige commented Oct 15, 2026

Copy link
Copy Markdown
Owner Author

Le check Scripts Tests (CPU) est rouge sur la tête 459cf8aeac, mais le job n'a exécuté aucun test.


Generated by Claude Code

@github-actions github-actions Bot added the pr-gate-missing PR gate absent du rollup: contexte requis jamais rapporte, PR bloquee, checks verts (#10928) label Oct 15, 2026
@github-actions

Copy link
Copy Markdown
Contributor

PR gate absent du rollup (advisory, #10928)

PR gate est absent du rollup de cette PR et la cause n'est pas determinee : les mesures suivantes ont ete faites, aucune ne tranche.

  • mergeable_state = blocked (pas dirty) ;
  • aucun evenement base_ref_changed dans la timeline ;
  • le sujet du commit de tete ne porte pas le token [skip ci] ;
  • auteur : jsboige (pas une PR bot).

Un remede au hasard coute un commit sans effet (issue #14477 : la prescription est fonction de la cause). Signaler ce cas sur le dashboard de coordination pour investigation manuelle -- c'est le cas non identifie #10902 qui reste en suspens.

Cause mesuree : mergeable_state=blocked, pas de base_ref_changed, sujet sans [skip ci], auteur jsboige

@github-actions

Copy link
Copy Markdown
Contributor

Golden-Set Execution (H.7 P3)

✅ 9/9 notebooks passed (certified reproducible)

Notebook Status Time
2.1-Workflow-ML.ipynb ✅ SUCCESS 10.3s
2.2-Descente-de-gradient.ipynb ✅ SUCCESS 3.7s
2.3-Regression-lineaire-logistique.ipynb ✅ SUCCESS 4.6s
2.4-Arbres-Forets-Ensembles.ipynb ✅ SUCCESS 4.4s
Search-01-StateSpace.ipynb ✅ SUCCESS 3.4s
SL-1-LogicalLearning.ipynb ✅ SUCCESS 2.3s
RL-04-Bandits-Manchots-Python.ipynb ✅ SUCCESS 21.1s
GameTheory-04c-NashExistence-Python.ipynb ✅ SUCCESS 7.5s
GameTheory-13d-Optimistic-CFR-Python.ipynb ✅ SUCCESS 11.6s

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

@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

Notebook PR Validation: PASS

  • Notebooks checked: 5
  • Code cells validated: 73
  • 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 github-actions Bot removed the pr-gate-missing PR gate absent du rollup: contexte requis jamais rapporte, PR bloquee, checks verts (#10928) label Oct 16, 2026
@jsboige

jsboige commented Oct 17, 2026

Copy link
Copy Markdown
Owner Author

[ADJOINT PREFLIGHT]
schema: 1
lane: myia-po-2023:CoursIA
pr: 20936
head: 5866d83
complete: true
body: read
comments-reviewed: 6
reviews-reviewed: 1
threads-reviewed: 0
threads-unresolved: 0
surfaces-sha256: 5afe9dce5a7ea18a1aac0a9a505cfe5a1c78ce4f4ea1c671cdca3042c464926b
diff-files: 11
diff-additions: 79
diff-deletions: 58
checks: blocked
b0: clear
scope: pass
domain: not-applicable
verdict: BLOCKED
organ: check_adjoint_prevalidation.py
organ-command: python scripts/check_adjoint_prevalidation.py --derive-verdict 20936
organ-rc: 3
[/ADJOINT PREFLIGHT]

Notes de lecture (surfaces) : accents #16638 — tranche Planners (6-Domains, 7-OR-Tools x2, 9-HTN x2) detachee de #20781 (11 fichiers +79/-58). Review Hermes COMMENTED @15/10 11:42:45Z : invariance strip-accents reproduite firsthand, 0 marqueur 🟡/🔴 — b0 clear. 6 commentaires lus = bots. 0 thread. Champ bloquant : checks — 'Organ-duplication advisory' rouge @15/10 20:43:43Z, MAIS le log du job montre l'organe CLEAN (0 collision, 8 series indexees) : l'echec est le POST d'annotation (API rate limit exceeded for site ID installation) — defaut de transport, pas un finding. Geste = rejouer la jambe. Note croisee : le log documente aussi les 2 orphan_entry nav-chain d'ANALYSE-06_en/ANALYSE-10 comme pre-existants sur main, correctif #20704.

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

@jsboige

jsboige commented Oct 18, 2026

Copy link
Copy Markdown
Owner Author

[ADJOINT PREFLIGHT]
schema: 1
lane: myia-po-2025:CoursIA-2
pr: 20936
head: 5866d83
complete: true
body: read
comments-reviewed: 8
reviews-reviewed: 1
threads-reviewed: 0
threads-unresolved: 0
surfaces-sha256: e3ae826d6396a6fce5b2009a66ad841328549c8e4bed1feae33190b1c057f49a
diff-files: 11
diff-additions: 79
diff-deletions: 58
checks: latest-wins-green
b0: clear
scope: pass
domain: pass
verdict: READY
organ: check_adjoint_prevalidation.py
organ-command: python scripts/check_adjoint_prevalidation.py --derive-verdict 20936
organ-rc: 0
supersedes: 7
supersedes-why: Le dossier precedent ne retenait que le check Organ-duplication advisory, rouge sur erreur de publication quota API sans collision de fond. Son rejeu est vert et le commentaire CLEAN a ete publie le 17 octobre a 08:58:19Z. Body, huit commentaires, review Hermes et diff integral relus ; cinq carnets en accents seuls et trois attestations associees, sans changement de code ou sortie. Verdict derive READY et mergeable=true mesures a tete inchangee. STACK WARN subsiste : pas de couverture automatique exhaustive des piles revendiquee.
[/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.

Review ai-01 (c2186). Le diff ne touche que des cellules markdown : il restaure des accents (« clés », « Création »…) dans les carnets annoncés, et les entrées YAML d'attestation qui vont avec sont cohérentes. Aucune cellule de code n'est modifiée, aucune sortie ni execution_count n'est touchée. Les faux positifs signalés dans le body sont bien laissés intacts. La review Hermes ne pose aucun marqueur de réserve, et le dossier est READY. Approuvé.

@myia-ai-01
myia-ai-01 merged commit bcb4584 into main Oct 18, 2026
70 of 71 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.

4 participants