Skip to content

TV-01/TV-02 -- les constantes LOC des carnets voisins sont figees en dur dans des cellules de code et derivent (383->388, 194->200, 431->436) #24421

Open

Description

@jsboige

Les tables « LOC » de la serie Texte/TransformerVariants figent, dans des cellules de code, le nombre de lignes de carnets voisins. Ces constantes derivent des que les carnets evoluent, et le carnet ne peut pas se voir deriver : la valeur pour lui-meme est un litteral en dur, la ou la colonne d'en face est mesuree a l'execution.

Constats verifies firsthand sur origin/main (mesure refaite independamment, convention calibree sur la seule ligne encore exacte du tableau).

Ce qui est fige, ou, et la derive mesuree

Carnet Cellule Constante figee Valeur reelle Derive
TV-01-Attention-Variants-SOTA.ipynb 24 (code) "383" pour TV-00b 388 +5
TV-01-Attention-Variants-SOTA.ipynb 24 (code) "182" pour TV-00a 182 0
TV-01-Attention-Variants-SOTA.ipynb 24 (code) "152" pour TV-01 (lui-meme) 152 0
TV-02-MoE-SOTA.ipynb 33 (code) LOC_34C = 431 (3.4c) 436 +5
TV-02-MoE-SOTA.ipynb 33 (code) "194" pour TV-02 (lui-meme) 200 +6

Convention de comptage (celle qu'enonce la table elle-meme) : lignes de cellules de code, non vides, ne commencant pas par #. Elle est calibree par la ligne TV-01 = 152, qui compte exactement 152 lignes de code sous cette convention — les deux valeurs de calibration (182 pour TV-00a, 152 pour TV-01) tombent juste, la convention est donc la bonne.

Les extraits verbatim :

  • TV-01, cellule 24 : ("TV-00b attention (commite)", ..., "383"),
  • TV-02, cellule 33 : LOC_34C = 431 # LOC effectives du 3.4c-MoE-from-scratch (constantes commitees, PR #16152) et ("LOC", f"{LOC_34C} (3.4c)", "194"),

Pourquoi la correction n'est pas un retouche de prose

Les valeurs sont des litteraux dans des cellules de code. Les changer touche la source d'une cellule executable : la re-execution devient due (C.2/C.3), et elle exige l'environnement des deux carnets — poids Mistral-7B en 4 bits pour TV-01, OLMoE pour TV-02, transformers 5.12.1. Aucun env local de la flotte ne le fournit a ce jour (coursia-sae 5.13.0, coursia-ml-training 5.14.1). Le report est donc route, pas contourne : aucune valeur n'est reecrite a la main sans sortie correspondante.

Correction proposee (deux niveaux, le second est celui qui tient)

  1. Recalcul a l'execution — remplacer les litteraux par la lecture du carnet voisin sur disque au moment de l'execution (le meme mecanisme que LOC_INDUSTRIE, qui mesure deja le fichier source du modele). Les constantes ne derivent plus : la table lit ce qu'elle decrit.
  2. Valeurs au commit de reference — a defaut, recalculer les cinq valeurs au commit de reference et citer ce commit dans le commentaire de la cellule, comme le fait deja LOC_34C pour la PR Add: TV-00c notebook MoE from scratch (routeur top-k, capacite, alpha_load) - See #16058 #16152.

Le niveau 1 est le bon : il supprime la classe de defaut au lieu de la deplacer.

Perimetre

MyIA.AI.Notebooks/GenAI/Texte/TransformerVariants/TV-01-Attention-Variants-SOTA.ipynb (cellule 24) et TV-02-MoE-SOTA.ipynb (cellule 33). Aucun autre carnet. 0 secret.

Origine

Finding de la campagne d'audit #17073, depose sur la partition GenAI #19451 (commentaire de finding, classe stale-claim), et signale en revue sur la PR #24403 (correction de prose TV-01/TV-02, serie Texte/TransformerVariants). La correction de prose de #24403 est livree ; le present ticket porte le reliquat qui exige une re-execution GPU.

Part of #19451

Activity

  1. added
    auditAutomated quality audit findings
    on Nov 3, 2026
  2. jsboige commented on Nov 3, 2026

    @jsboige
    OwnerAuthor

    [DELIVERED] lane myia-ai-01:CoursIA-2 -- PR #24451

    Le niveau 1 est livre : les volumes de code sont relus sur le disque au moment de l'execution (organe tv/loc.py de la serie), la re-execution du bloc B est faite sur le device 2 (TV-01 NF4, TV-02 bf16), et la prose qui citait le run precedent est realignee sur la sortie reelle.

  3. added a commit that references this issue on Nov 5, 2026
  4. jsboige commented on Nov 6, 2026

    @jsboige
    OwnerAuthor

    [CLAIMED] lane myia-ai-01:CoursIA-2 -- paths: MyIA.AI.Notebooks/GenAI/Texte/TransformerVariants/TV-01-Attention-Variants-SOTA.ipynb, MyIA.AI.Notebooks/GenAI/Texte/TransformerVariants/TV-02-MoE-SOTA.ipynb -- les constantes LOC figees dans des cellules de code : la valeur de TV-01/TV-02 pour eux-memes doit etre MESUREE a l'execution, comme la colonne d'en face

  5. jsboige commented on Nov 6, 2026

    @jsboige
    OwnerAuthor

    [INFO] candidate-delivered lane myia-ai-01:CoursIA-2 -- la correction est deja livree et mergee : PR #24451, commit 3f4525f822341 (mergee 2026-11-05T08:38:49Z).

    Le carnet lit desormais les volumes de code SUR DISQUE au moment de l'execution, via l'organe MyIA.AI.Notebooks/GenAI/Texte/TransformerVariants/tv/loc.py : plus aucun litteral dans les cellules de code. Mesure a la tete mergee (base 0374bf0f19c2d) :

    • TV-00a = 182 · TV-00b = 388 · TV-01 = 152 · TV-02 = 200 · 3.4c = 436.

    Les cinq valeurs mesurees par la regle de portage de l'issue tombent exactement sur les valeurs reelles (TV-00b 383 -> 388, TV-02 194 -> 200, 3.4c 431 -> 436) : la derive constatee est corrigee, et la table ne peut plus deriver.

    Ce que je n'ai pas fait : rien n'a ete re-execute dans ce cycle, et aucune cellule n'a ete touchee par ma lane. La PR #24451 avait deja produit les sorties re-executees du bloc B.

    La cloture de l'issue revient au coordinateur ou a l'adjoint (le droit de cloture d'une issue de partition est reserve) -- cette note donne la preuve par criterion.

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

Metadata

Metadata

Assignees

No one assigned

    Labels

    auditAutomated quality audit findings

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions