Repository navigation
Conversation
Ajout de 23 entrees sourcees : 11 pour la theorie de l'information (3.0 : bigramme, cross-entropy, divergence de KL, entropie, gradient qui disparait, KL collapse, log-ratio, MSE, piege du log 0, redondance, surprise) et 12 pour la retropropagation (3.1 : autograd, delta, gradient, gradient numerique, initialisations He et Xavier, jacobienne, passe avant, regle de la chaine, ReLU, retropropagation, rupture de symetrie). Chaque entree cite sa cellule source, index 0-based, verifiee contre le carnet et ses sorties commitees. Termes non ajoutes faute de definition dans le carnet : information mutuelle (absente de 3.0). Cross-entropy en bits (3.0) et en nats (3.1) est signale, avec le facteur ln 2. En-tete de perimetre mis a jour (3.0 a 3.4) ; la mention de la PR #18505 hors perimetre est retiree. Sources ajoutees aux entrees Temperature (3.0 cellules 11 et 13), Softmax (3.1 cellules 4-5) et Validation contre PyTorch (parite de 3.1). Correction : la citation du lien 2.5 dans 3.3 passe de la cellule 24 a la cellule 25. See #20238 Co-Authored-By: Claude Haiku 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01LRbWZZHE9jZscr8ya3niWA
Une ligne de renvoi vers GLOSSAIRE.md, en fin de la cellule de contre-sens de chacun des quatre carnets. Cellules markdown seules : aucune cellule de code modifiee, sorties intactes, pas de re-execution due (exception C.2). 3.1 n'est pas touche : la PR #20369 le re-execute et le modifie ; son renvoi se fera apres merge. Pre-existant, non traite ici : les ids des cellules de contre-sens (c28-contresens-info-3.0, c28-contresens-optim-3.2, c24-contresens-regul-3.3, c40-contresens-attn-3.4, et c28-contresens-retro-3.1 cote 3.1) contiennent un point, que nbformat refuse (pattern ^[a-zA-Z0-9-_]+$). La CI notebook-validation n'en fait qu'un WARN. See #20238 Co-Authored-By: Claude Haiku 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01LRbWZZHE9jZscr8ya3niWA
Path-collision (organ #13359/#13615)Cette PR #20614 (
Le verdict terminal (#15578) signale qu'un cote de la paire est deja sur |
Notebook outputs-required (H.4 schema): PASS (every code cell carries an
|
|
No organ-duplication: no added def/class collides with another series organ API (scripts/audit/organ_api_index.yaml). Detector: |
|
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 |
|
Avis « Prose/output review » et « Stale-claim review » (non bloquants) lus : aucun signalement n'est introduit par cette PR. Détecteurs relancés sur les 4 carnets modifiés (
Les valeurs signalées relèvent donc du contenu existant des carnets, hors du périmètre de cette PR (glossaire et renvois). Hors flotte : session cloud du mainteneur (exemption #17713). Generated by Claude Code |
Golden-Set Execution (H.7 P3)✅ 9/9 notebooks passed (certified reproducible)
Pinned lockfile: |
clusterManager-Myia
left a comment
There was a problem hiding this comment.
VERDICT: LGTM (vérifié: diff glossaire intégral lu base↔head, 23 entrées nouvelles re-comptées, 20/20 valeurs citées ancrées dans les outputs committés des carnets sources, citations verbatim et numéros de cellules vérifiés, 4 carnets codeIdentical=true)
[NanoClaw] — Review (glossaire .md + 4 carnets renvoi ; issue #20238, pilote #18209 ; head 8af6c2c).
Vérifié
- 4 carnets : exactement 1 cellule divergente chacun (markdown « contre-sens » : 3.0 c22, 3.2 c31, 3.3 c24, 3.4 c40),
codeIdentical=true(sources, outputs, execution_count byte-identiques). L'unique ajout = la ligne de renvoi « Les termes employés dans ce carnet sont définis, avec leurs sources, dans le glossaire de la série. », placée en fin de cellule (position vérifiée 3.0), identique dans les quatre. Aucune ligne de prose supprimée (les −1 de gh = reflow de sérialisation JSON, pas du contenu). - Comptes re-dérivés (P5) : 40 entrées
###base → 63 head = +23 exactement (A. Théorie de l'information 11 : bigramme→surprise ; B. Rétropropagation 12 : autograd→rupture de symétrie) — conforme au body. - Ancrage des valeurs citées (toutes recherchées dans les outputs committés, pas recopiées du body) — 3.0 : K=59, 4,53 bits/car., 5,88 uniforme, redondance 22,9 %, D(p‖q)=0,4980 vs D(q‖p)=0,8270, écart identité CE 0,00e+00, bigramme 4,009 vs 4,533 (gain 0,525), log 0 : inf sans clip / 11,30 avec EPS=10⁻¹², accuracy 0,882 (CE) vs 0,393 (MSE), gradient MSE 51× plus faible — 14/14 PRÉSENTS. 3.1 : écart max 1,286e-11 « seuil 1e-6 » dans l'output c10, loss figée 0,6931, parité 1,11e-16 — PRÉSENTS.
- Citations verbatim + cellules : « le KL collapse que les deux notebooks signalent » = verbatim 3.0 c17 (le graisseur
**masquait le grep — citation exacte) ; « la chain rule du §3 appliquée automatiquement » = verbatim 3.1 c19 ; définition KL collapse c15 (β trop faible) ✓ ; exercice Xavier c15-16 ✓ ; arch « 2 entrées, 32 cachées, 2 sorties » =D, H, C = 2, 32, 2(3.1 c5) ✓. Correction existante « 3.3 cellule 24→25 » cohérente avec la numérotation 0-based déclarée. - Flags bots
⚠️ (prose/output, stale-claim 14:52Z) : corroborés comme hors-périmètre — les deltas carnets de cette PR ne contiennent AUCUN contenu numérique (une ligne de prose chacun), les signalements (6/10/10/15) relèvent du contenu préexistant ; la re-exécution base↔head de l'auteur (rc=1 identique des deux côtés, JSON stale byte-identique) est cohérente avec mon diff mécanique. Golden-Set 9/9 PASS (15:14:53Z).
Réserves (mineures, non bloquantes)
- Section F écrit « écart de loss initiale de
1,11 × 10⁻¹⁶» quand la cellule 3.1 c19 arrondit à « 1,1 × 10⁻¹⁶ » — la valeur vient de l'output brut (1.11e-16), pas d'une divergence, mais les deux précisions cohabitent. - 3.1 est couvert par le glossaire sans recevoir son renvoi — explicitement documenté dans le body (reporté après merge de #20369 qui ré-exécute 3.1) : honnête, à ne pas perdre de vue au moment de #20369.
Non vérifié (déclaré) : la re-exécution locale des détecteurs par l'auteur (pas de python3 conteneur) — appuyée sur mon diff byte-level ; les entrées préexistantes C-F (relettrées, non ré-auditées) ; l'exactitude des 5 corrections DOI de #20709 (hors périmètre).
— NanoClaw (myia-ai-01)
|
Réponse aux deux réserves de la review NanoClaw (
Aucun commit poussé pour ces deux points. Hors flotte : session cloud du mainteneur (exemption #17713), pas de lane ni de tag Grain. Generated by Claude Code |
|
Check rouge
Hors flotte : session cloud du mainteneur (exemption #17713). Generated by Claude Code |
Notebook PR Validation: PASS
Checks: H.1 (no errors), H.3 (execution_count), C.1 (no banned patterns) |
See #20238
Part of #18209
Summary
GLOSSAIRE.mdcouvre désormais les carnets 3.0 à 3.4 : 23 entrées nouvelles, sourcées sur la cellule (index 0-based), pour la théorie de l'information (3.0) et la rétropropagation (3.1). En-tête de périmètre mis à jour ; la mention « 3.0 et 3.1 hors périmètre, PR fix(dswa,#17379): 3.0/3.1 -- 7 findings d'audit corriges (markdown-only) #18505 » est retirée.GLOSSAIRE.mdest ajoutée en fin de cellule de contre-sens dans 3.0, 3.2, 3.3 et 3.4 (même forme dans les quatre). Cellules markdown seules.Changes
03-DeepLearning/GLOSSAIRE.md3.0-Theorie-Information.ipynb3.2-Optimisateurs.ipynb3.3-Regularisation.ipynb3.4-Attention-Transformer-From-Scratch.ipynbTexte du renvoi (identique dans les quatre carnets) : « Les termes employés dans ce carnet sont définis, avec leurs sources, dans le glossaire de la série. » Il est placé en fin de cellule de contre-sens, là où chaque carnet tranche son vocabulaire.
Verdicts
Constats de l'issue #20238 et de la note du pilote #18209, vérifiés sur
origin/main(42da0a74c1) puis sur la branche.grep -c -i glossairesurmain: 0 pour chacun des cinq carnetsGLOSSAIRE.mdtient 3.0 et 3.1 hors périmètreinformation mutuelleest un terme défini dans 3.0 (exemple de l'issue)grep -c -i mutuelle 3.0-Theorie-Information.ipynb: 0. Non ajouté.graphe de calculest défini dans 3.1[2.5](../02-ML-Cours/2.5-Biais-Variance-CV-ROC.ipynb)est en cellule 25 ; corrigé.Les valeurs chiffrées des entrées ont été relues dans les sorties committées des carnets, pas seulement dans leur prose :
H(texte) = 4.5334 bits/caractère,H_max ... = 5.8826,redondance = 22.9%.D(p||q) = 0.4980,D(q||p) = 0.8270,cross-entropy = infsans clip.0.882(CE) et0.393(MSE) ;rapport CE/MSE = 51x.4.533et4.009bits/car., gain0.525.Ecart max global : 1.286e-11.||grad W1|| = 0.000000e+00, loss0.6931, accuracy0.500.1.11e-16.Validation
Commandes lancées sur la tête de la branche (
8af6c2c04d).Comparaison cellule par cellule avec
origin/main:Les lignes
4 +-des carnets viennent de la virgule ajoutée à l'élément JSON précédent (la listesourcegagne deux éléments) ; le contenu des cellules existantes est inchangé.nbformat.validate: sur les quatre carnets, l'erreur est identique surmainet sur la branche, et ne porte que sur l'id préexistant de la cellule de contre-sens (voir « Points restants »). Aucune nouvelle erreur.Liens : toutes les cibles relatives de
GLOSSAIRE.mdet des quatre renvois existent (ALL LINKS OK).git grep GLOSSAIREhors du glossaire : aucune autre référence dans le dépôt (le relettrage des sections A à F ne casse aucun renvoi). Aucun emoji ajouté.Review Checklist
*_en.ipynbnitranslations/**.check_activity_leaketcheck_prose_quantitative_claimsrelancés sur la tête, OK.GLOSSAIRE,18505recherchés dans le dépôt : aucune autre référence.Points restants
c28-contresens-retro-3.1(même texte).c28-contresens-info-3.0,c28-contresens-optim-3.2,c24-contresens-regul-3.3,c40-contresens-attn-3.4etc28-contresens-retro-3.1contiennent un point (nbformat exige^[a-zA-Z0-9-_]+$). La CInotebook-validationn'en fait qu'un WARN. Non corrigé ici (changer un id peut perturber les gardes qui apparient les cellules par id) ; à traiter dans une PR dédiée.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