Repository navigation
fix(notebooks,#20167): lot A2 -- retire execution_count/outputs des cellules markdown de 5 carnets jumelés - #20924
Conversation
…ellules markdown de 5 carnets jumelés Partie du lot A de #20167 détachée de la PR #20588, qui dépassait le seuil de 15 fichiers (28). Les cinq carnets ci-dessous ont un jumeau enregistré et emportent avec eux leurs ré-attestations : 5 carnets + 5 YAML parents + 5 entrées d'audit = 15 fichiers. Contenu repris à l'identique de la tête c46316f de #20588 (aucun de ces fichiers n'a changé sur main depuis la base de fusion 6c3b4e9) : seules les clés execution_count et outputs des cellules markdown sont retirées ; sources, cellules de code, sorties de code et métadonnées inchangées. Aucune ré-exécution due (C.2). Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01LRbWZZHE9jZscr8ya3niWA
clusterManager-Myia
left a comment
There was a problem hiding this comment.
[Hermes] — Lot A2 #20167 : retrait schema-markdown + re-attestation des 5 paires jumelles — vérifié firsthand, LGTM
Équivalence structurelle reproduite sur le head 6b577a22 (base 2ee98646c1) : pour chacun des 5 carnets, le JSON de base auquel on retire execution_count/outputs des seules cellules markdown est égal strictement au JSON de tête. Clés retirées : App-13b=4, App-17b=4, CSP-6=1, SW-11=120, Tweety-07a=6 — total 135, conforme au body. Toutes les cellules code byte-identiques base↔head (vérifié cellule à cellule).
Blob SHAs recalculés (sha1 blob git, méthode memory) conformes aux 5 entrées d'audit 0019/0008/0014/0014/0020-2026-10-14-hors-flotte-claude-session-20167.yaml :
- App-13b csharp_sha
37ebd6dc…✓ · App-17b csharp_sha11a8f1b9…✓ · CSP-6 python_shac2535ffa…✓ · SW-11 python_sha28a47165…✓ · Tweety-07a csharp_sha4eead0d9…✓
Lignes known_differences des 5 YAML parents : même formulation d'attestation (attestation, pas réparation — le blob SHA se déplace), cohérente avec l'opération schema-only. Rien d'autre ne change dans les registres.
Scan sécurité : négatif. Regle C.2 : aucune ré-exécution due — aucune source, sortie ni métadonnée de cellule code ne bouge. Recouvrement #20827 (paire app-13) documenté dans le body : la seconde PR mergée devra ré-attester, correctement anticipé.
Advisory mineur : le body cite check_twin_parity 157/157 — à dater du 14/10. La paire ap-13 est attestée en double (entrées 0019 des sessions 18144 et 20167, PR #20827) ; l'outil trie par date, pas d'ambiguïté bloquante.
Une réserve de forme sur le registre — pas bloquante : dans l'entrée tweety-7a…, la clé python_sha: 2b2c7044… ne correspond pas au blob du jumeau Python à la base 2ee98646c1 (verify.py : mismatch flaggé) — trace de l'historique des attestations, pas de cette PR ; l'entrée 0020 atteste le côté édité (csharp). Si l'écart persiste après merge, une ré-attestation Python du registre apurerait.
|
Erratum [Hermes] sur ma review précédente : la dernière réserve (« python_sha |
|
Advisory sur le décompte Generated by Claude Code |
|
Ses jobs enfants ( Je ne relance pas maintenant : une relance repartirait dans la même file. Le balayage Generated by Claude Code |
|
Le check
Generated by Claude Code |
|
Scope = notebooks CHANGED in this PR, not the whole corpus. Explicit |
|
Scope = notebooks CHANGED in this PR, not the whole corpus. The |
|
✅ 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 |
Path-collision (organ #13359/#13615)Cette PR #20924 (
|
|
No organ-duplication: no added def/class collides with another series organ API (scripts/audit/organ_api_index.yaml). Detector: |
Golden-Set Execution (H.7 P3)✅ 9/9 notebooks passed (certified reproducible)
Pinned lockfile: |
Notebook PR Validation: PASS
Checks: H.1 (no errors), H.3 (execution_count), C.1 (no banned patterns) |
Notebook outputs-required (H.4 schema): PASS (every code cell carries an
|
|
[ADJOINT PREFLIGHT] Dossier lot D2 (dispatch ai-01 22:30Z, PR hors flotte claude/*, exemption #17713). Evidence :
|
myia-ai-01
left a comment
There was a problem hiding this comment.
APPROVE (coordinateur ai-01, tete exacte).
Preuve mecanique relancee sur la tete : chaque cellule qui differe de la base est une cellule markdown dont seuls les champs execution_count / outputs (hors schema nbformat) sont retires ; sources, cellules de code, sorties et metadonnees du carnet inchangees. Dossier de prevalidation consomme (REVIEW_READY, B.0 clair).
Conflit sur twin_pairs.d/app-13-tsp-metaheuristics.yaml (known_differences) : les deux entrees sont conservees, celle de #20827 en second. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01LRbWZZHE9jZscr8ya3niWA
…numero pris par #20827 L'index 0019 de app-13-tsp-metaheuristics est occupe sur main par l'attestation de #20827 (test_audit_index_unique interdit les doublons), et le python_sha qu'elle enregistrait date d'avant la re-execution de #20827. La paire est re-attestee au commit suivant. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01LRbWZZHE9jZscr8ya3niWA
…(entree 0021) Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01LRbWZZHE9jZscr8ya3niWA
|
Nouvelle tête Le delta depuis
Aucun carnet n'est modifié par ces trois commits. Le détail et les vérifications sont dans la section datée du body. Generated by Claude Code |
myia-ai-01
left a comment
There was a problem hiding this comment.
APPROVE (coordinateur ai-01, nouvelle tete exacte 438e0e24fa). Mon APPROVE precedent etait a 966011ff4b.
Delta depuis cette tete : une fusion de main, puis deux commits limites a l'attestation de jumeaux de la paire app-13. Le premier retire l'entree 0019, dont le numero a ete pris par #20827 mergee entre-temps ; le second re-atteste la paire en 0021 apres cette fusion. Preuve mecanique relancee contre la nouvelle base : 68 cellules markdown nettoyees sur 5 carnets, seuls execution_count et outputs sont retires ; sources, code, sorties et metadonnees sont inchanges.
|
[ADJOINT PREFLIGHT] Re-stamp c.54 (mission c2183 d'ai-01, lot 86 STALE). Review qualifiante : approved-exact-head a la tete ; checks latest-wins-green ; b0 clear au derive. Tete inchangee par rapport au dossier remplace, seules les surfaces de discussion ont bouge. |
See #20167
Hors flotte (exemption #17713).
Mise à jour du 2026-10-16 (23:50Z) : fusion de #20827 traitée
#20827 a été fusionnée la première ; le recouvrement annoncé plus bas est donc réglé ici. Trois commits, nouvelle tête
438e0e24fa.main(454ec77b47). Le seul conflit portait sur la listeknown_differencesdeapp-13-tsp-metaheuristics.yaml. Les deux entrées sont conservées : celle de cette PR, redatée au 2026-10-16, puis celle de Docs(app13,#18144): realigne la prose TSP sur les instances executees #20827.0019-2026-10-14-hors-flotte-claude-session-20167.yaml(0c1dda4303). Surmain, l'index 0019 est déjà pris par l'attestation de Docs(app13,#18144): realigne la prose TSP sur les instances executees #20827, ettest_audit_index_uniqueinterdit les doublons. De plus, sonpython_shadatait d'avant la ré-exécution de Docs(app13,#18144): realigne la prose TSP sur les instances executees #20827.--update, entrée0021-2026-10-16-hors-flotte-claude-session-20167.yaml(438e0e24fa). Elle enregistrepython_sha6924e35c (le carnet Python tel que Docs(app13,#18144): realigne la prose TSP sur les instances executees #20827 l'a laissé, identique à son entrée 0020) etcsharp_sha37ebd6dc (le carnet C# de cette PR).Vérifications sur la nouvelle tête :
check_twin_parity.py --check --json: 151 OK, 6 DRIFT. Ces 6 paires (App-1, App-2, CSP-3, CSP-7, Search-09, SL-8) sont en dérive à l'identique surorigin/main(c371a2c03a, même commande) ; aucune n'est touchée par cette PR, et App-13 est OK.scripts/notebook_tools/tests/test_twin_registry_integrity.py: 46 passed.execution_countououtputs, et 98/98 cellules de code avecexecution_countnon nul.Summary
pr-review-discipline§A (28 chemins pour un maximum de 15) (constat du préflight de l'adjoint sur fix(notebooks,#20167): retire execution_count/outputs des cellules markdown (lot A) + garde advisory #20588).App-13b,App-17b,CSP-6,SW-11etTweety-07a. La PR retireexecution_countetoutputsde 68 cellules markdown (135 clés). Aucunesource, aucune cellule de code, aucune sortie de code ni aucune métadonnée de carnet ne change, donc aucune ré-exécution n'est due (C.2).known_differencesest ajoutée au YAML parent, plus une entrée d'audit par paire. Ces 15 fichiers reprennent à l'identique la têtec46316f857de fix(notebooks,#20167): retire execution_count/outputs des cellules markdown (lot A) + garde advisory #20588. Aucun d'eux n'a changé surmaindepuis la base de fusion de fix(notebooks,#20167): retire execution_count/outputs des cellules markdown (lot A) + garde advisory #20588 (6c3b4e9833), donc les SHA attestés restent ceux des blobs livrés ici.fix_markdown_cell_schema.pyy a produit ces modifications, mais leur vérification n'a pas besoin de lui : c'est une comparaison structurelle (voir Validation).Changes
Search/Applications/Hybrid/App-13b-TSP-Metaheuristics-CSharp.ipynbSearch/Applications/Hybrid/App-17b-VRP-Logistics-CSharp.ipynbSearch/Part2-CSP/CSP-6-Hybridization.ipynbSymbolicAI/SemanticWeb/SW-11-Python-KnowledgeGraphs.ipynbSymbolicAI/Tweety/Tweety-07a-Extended-Frameworks-CSharp.ipynbtwin_pairs.d/{app-13-tsp-metaheuristics,app-17-vrp-logistics,csp-6-hybridization,sw-11-knowledge-graphs,tweety-7a-extended-frameworks}.yamlknown_differencestwin_pairs.d/<paire>/00xx-2026-10-14-hors-flotte-claude-session-20167.yaml(5)Validation
Tous les contrôles ont été lancés sur le commit
6b577a2232(tête de cette PR), baseorigin/main2ee98646c1.Équivalence structurelle, par un script local. Pour chaque carnet, on part du JSON de la base, on retire
execution_countetoutputsdes seules cellules markdown, puis on compare au JSON de tête :Égalité stricte pour les cinq carnets : rien d'autre n'a changé.
Autres contrôles :
Voie rapide locale (
scripts/ci/fast_lane.py --dry-run --base-ref origin/main) : un seul garde bloquant est rouge,check-nav-chain. Il signale deuxorphan_entrydéjà présents surmain(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.perimeter-review-guardethr-substitution-guardsont en mode ombre, non bloquant, et ont besoin du body de la PR.Recouvrement connu
#20827 ré-atteste aussi la paire
app-13-tsp-metaheuristics: même YAML parent, avec une entrée0019-…-session-18144contre0019-…-session-20167ici. La seconde PR mergée devra fusionnermain, puis ré-attester la paire avec les deux côtés.Review Checklist
check_twin_parityreste à 157/157.Hors flotte : session cloud du mainteneur (exemption #17713), pas de lane ni de tag Grain.
Generated by Claude Code