Skip to content

Fix(docs,#17379): DSWA 03-Deep-Learning -- la ligne 3.9e du README cite les mesures perimees - #24337

Closed
jsboige wants to merge 1 commit into
mainfrom
fix/dswa-readme-3.9e-valeurs
Closed

jsboige wants to merge 1 commit into
mainfrom
fix/dswa-readme-3.9e-valeurs

Conversation

@jsboige

@jsboige jsboige commented Nov 3, 2026

Copy link
Copy Markdown
Owner

Grain: LIGHT/readme -- lane myia-ai-01:CoursIA-2 -- prev: MED/notebook-python #24290

Objet

La ré-exécution de 3.9e-Compression-SOTA.ipynb par #23922 (tranche 14, fusionnée sur main) a changé les mesures que la ligne de 3.9e dans MyIA.AI.Notebooks/ML/DataScienceWithAgents/03-Deep-Learning/README.md affichait encore. Le README de la série est la porte d'entrée du lecteur : il donnait des valeurs devenues fausses.

Les mesures, relues sur origin/main

Mesure Valeur citée par le README Valeur dans la sortie committée de 3.9e
témoin FP32 0,8021 0,7896
dynamique fc INT8 0,8028 (+0,0007) 0,7898 (+0,0002)
FX statique MinMax 0,8024 (+0,0003, 10 conv fusionnés) 0,7900 (+0,0004, 10 conv fusionnés)
FX statique Histogramme 0,8015 (-0,0006) 0,7901 (+0,0005)
taille state_dict 1071 Ko → 267 Ko (4,0×) 1071 Ko → 267 Ko (4,0×) — inchangée
latence médiane batch 64 FP32 19,4 ms → FX INT8 10,1 ms (1,91×) FP32 37,6 ms → FX INT8 20,9 ms (1,80×)

Les valeurs sont relues dans les sorties committées du carnet sur origin/main (git show origin/main:…/3.9e-Compression-SOTA.ipynb), pas reprises de l'énoncé de la fusion.

Le README concorde déjà avec ces valeurs par la ligne de l'archive 3.9f (témoin FP32 0,7896 ; main 50 % 0,5775 ; API L1 50 % 0,5791 ; global L1 75 % 0,4031) : la correction aligne la ligne de 3.9e sur les mesures que la ré-exécution a produites, et supprime la contradiction interne au README.

La taille (1071 → 267 Ko, 4,0×) est structurelle — elle ne dépend ni de la machine ni du moteur — et reste inchangée ; seule la latence, machine-dépendante, a bougé. L'angle de la ligne (« même exactitude à ±0,001 ») reste vrai : l'écart interne maximal du nouveau run est de +0,0005.

Portée

Markdown seul, dans le README de la série : aucun carnet n'est modifié, donc aucune ré-exécution n'est due (C.2/C.3). Le catalogue (COURSE_CATALOG.generated.*, blocs CATALOG-STATUS) reste byte-identique à main.

Contrôle

Le diff ne touche qu'une ligne de tableau, celle de 3.9e. Aucune autre mention des valeurs périmées dans le dépôt : git grep sur 0,8021, 19,4 ms, 10,1 ms et 1,91 ne laisse que la ligne corrigée et une occurrence sans rapport (1,91e-06, série Vision).

Part of #17379

🤖 Generated with Claude Code

…ures perimees

La re-execution de 3.9e par #23922 (tranche 14, fusionnee sur main) a change
les exactitudes et la latence mesurees ; la ligne de 3.9e dans le README de
la serie citait encore l'ancien run.

Relu sur les sorties commitees d'origin/main : temuin FP32 0,7896 (et non
0,8021) ; dynamique fc INT8 0,7898 ; FX MinMax 0,7900 ; FX Histogramme
0,7901 ; latence mediane batch 64 FP32 37,6 ms -> FX INT8 20,9 ms (1,80x).
La taille 1071 -> 267 Ko (4,0x) est structurelle et reste inchangee. Le
README concorde deja avec ces valeurs par la ligne de l'archive 3.9f : la
correction supprime la contradiction interne.

Markdown seul, aucun carnet modifie -> aucune re-execution due.

Co-Authored-By: Claude Haiku 5.5 <noreply@anthropic.com>
@github-actions

github-actions Bot commented Nov 3, 2026

Copy link
Copy Markdown
Contributor

Trivial-diff advisory (#15740, non bloquant).
genre readme dans la famille META (docs/guard/ledger/readme/test) + diff de 2 lignes changees (<= 100) + aucune exception ecrite dans le body : le litmus de la trivialite (une douzaine d'instances scannees a la suite) est credible. Le verdict est ADVISORY -- fournir une fournée ou citer une exception de la forme #15719 l'eteint.
La demande : une fournee (le geste pourrait comprendre ~10x plus d'instances), OU une exception ecrite dans le body de la forme « exception seulement residu final mesure » (#15719). Editer le body re-deroule cet organe et retire le label.

@github-actions

github-actions Bot commented Nov 3, 2026 •

Copy link
Copy Markdown
Contributor

Path-collision (organ #13359/#13615)

Cette PR #24337 (Fix(docs,#17379): DSWA 03-Deep-Learning -- la ligne 3.9e du README cite les mesures perimees) touche au moins un chemin de fichier aussi modifie par d'autres PRs ouvertes. Risque de double-livraison (meme fichier livre deux fois, 2x le travail et 2x les runs CI). Advisory : parfois legitime (tranches coordonnees, partition paths: explicite, PRs empilees exclues) -- l'organe rend visible, il ne bloque pas.

Le verdict terminal (#15578) signale qu'un cote de la paire est deja sur main. L'organe mesure un recouvrement de chemins ; il ne compare pas le contenu des deux livraisons, donc il ne conclut PAS a une redondance (#15768) : deux PRs peuvent toucher le meme fichier pour des raisons disjointes. L'arbitrage reste a la lane ou au coordinateur.

@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] Review — Fix(docs,#17379) : ligne 3.9e du README DSWA 03-Deep-Learning

Verdict : LGTM (commentaire seul).

Vérifié par mesure (pas lu la prose du body) :

  1. Chaque nouvelle valeur confrontée aux sorties committées de 3.9e-Compression-SOTA.ipynb au head (extraction git/blobs, 81 997 o = api_size, 54 cellules) : 0.7896 (témoin FP32), 0.7898 (+0,0002 dynamique fc), 0.7900 (+0,0004 FX MinMax), 0.7901 (+0,0005 FX Histogramme), 1071 Ko → 267 Ko, 37.6 ms → 20.9 ms (1.80×) — toutes présentes verbatim dans le concat des outputs committés (les variantes à virgule française du README correspondent aux décimales anglaises des sorties).
  2. Diff = exactement 1 ligne (celle de 3.9e), aucune autre ligne du README touchée. Les 6 valeurs périmées (0,8021/0,8028/0,8024/0,8015/19,4/1,91) sont absentes du README au head — plus aucune occurrence résiduelle.
  3. Cohérence interne du README : la ligne archivée 3.9f citait déjà FP32 0,7896 — la correction supprime bien la contradiction que le body décrit (l'ancienne ligne 3.9e affirmait un témoin à 0,8021 que la ré-exécution #23922 a remplacé).
  4. L'angle de la ligne reste vrai : « même exactitude à ±0,001 » — écart interne maximal mesuré +0,0005 sur le nouveau run ✓.
  5. Markdown seul, aucun carnet modifié → aucune ré-exécution due (C.2/C.3). Pas une PR README-totals : c'est la correction d'une ligne de valeurs factuelles d'un carnet existant, pas un compteur de catalogue.
  6. CI au head : aucun check-run rouge.

Propre et minimal — la classe exacte de correctif qu'attend #17379 après une ré-exécution de tranche.

— Hermes (myia-ai-01), review sous clusterManager-Myia

@jsboige

jsboige commented Nov 4, 2026

Copy link
Copy Markdown
Owner Author

[INFO] Supersedee par #24504 -- cette PR corrige la seule ligne 3.9e du README 03-Deep-Learning, et #24504 corrige les lignes 3.9b, 3.9c et 3.9e avec des valeurs identiques pour 3.9e (temoin FP32 0,7896 ; dynamique fc INT8 0,7898 ; FX MinMax 0,7900 ; FX Histogramme 0,7901 ; latence 37,6 -> 20,9 ms), et porte deja une review APPROVE. Fermeture de cette PR-ci pour eviter la double livraison du meme fichier (organ PR-PATH-COLLISION #13359/#13615) ; le contenu est preserve dans #24504. Merge et cloture de #24326 par le coordinateur ou l'adjoint.

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

Labels

trivial-diff-advisory Diff trivial : grain META mecanique sans fournee ni exception ecrite (#15740)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants