Skip to content

chore(iit,#14032): renommer gt24b_path() -- residu de nommage apres l'absorption 18d -> 18c (#19040) #19165

Description

@myia-ai-01

Contexte

La PR #19040 consolide le banc humour : GameTheory-18c-Humour-Banc-Python.ipynb absorbe 18d (#14032). La review NanoClaw de cette PR a relevé un résidu de nommage, classé par le bot lui-même « documentation, non bloquant ». L'auteur l'a déclaré comme dette, parce que le corriger demande de toucher des cellules de code puis de ré-exécuter les carnets consommateurs.

Résidu, à la tête de #19040 (ad28051b67)

  • MyIA.AI.Notebooks/IIT/ICT-Series/ict/humor_pairs.py : la fonction s'appelle encore gt24b_path() (l.38), et sa docstring parle des « cellules de GT-24b » (l.73), alors qu'elle rend le chemin de GameTheory-18c-Humour-Banc-Python.ipynb (l.43). L'ancien nom désigne un carnet qui n'existe plus.
  • Deux conventions de numérotation de cellules cohabitent dans les docstrings du module (« cellule 29 », « cellule 33 »).

Consommateurs à mettre à jour avec le renommage

  • MyIA.AI.Notebooks/IIT/ICT-Series/ICT-35b-HumorCausalPairs-SAE-Python.ipynb (cellule de code : ré-exécution requise, C.2)
  • MyIA.AI.Notebooks/IIT/ICT-Series/ict/tests/test_humor_pairs.py

Critère de clôture

  • git grep -n "gt24b" -- MyIA.AI.Notebooks/IIT ne rend plus rien.
  • La docstring nomme le carnet 18c.
  • ICT-35b est ré-exécuté et committé avec ses sorties, et le ratchet output-collapse reste vert sur la PR.
  • pytest MyIA.AI.Notebooks/IIT/ICT-Series/ict/tests/test_humor_pairs.py passe.

Ouverte avant le merge de #19040, comme issue de suivi nommée de la levée (B.0, voie 3).

Activity

  1. jsboige commented on Oct 4, 2026

    @jsboige
    Owner

    Grain: MED/refactor -- lane myia-po-2027:CoursIA -- prev: MED/refactor #19151

    [CLAIMED] lane myia-po-2027:CoursIA — renommage gt24b_path() + mise à jour des 3 consommateurs + ré-exécution d'ICT-35b.

    Assignée par ai-01 (DM msg-20261004T220040-l4bwwz, « Elle est libre si tu veux la prendre ensuite »).

    Séquencement imposé par une dépendance logique mesurée, pas par prudence. Ce grain s'exécute après le merge de #19040, pour deux raisons indépendantes :

    1. Le résidu n'existe que sur l'arbre de feat(gametheory,#14032): banc humour consolide -- 18c absorbe 18d (ex-GT-28b) #19040. Sur main, le renommage n'aurait pas la même cible : gt24b_path() y rend GameTheory-18d-Humour-Banc-Dur-Python.ipynb (mesuré sur ma branche, tête 13fe993cc270) ; la tête de feat(gametheory,#14032): banc humour consolide -- 18c absorbe 18d (ex-GT-28b) #19040 (ad28051b67) la repointe vers GameTheory-18c-Humour-Banc-Python.ipynb avec le commentaire # consolide #14032. Un nom posé sur main serait donc immédiatement re-faux.
    2. Les sorties d'ICT-35b dépendent du découpage que feat(gametheory,#14032): banc humour consolide -- 18c absorbe 18d (ex-GT-28b) #19040 change. ict/humor_pairs.py déclare le contrat de reproduction par un intervalle de cellules : _CORPUS_CELL_END = 10 sur main, _CORPUS_CELL_START = 26 / _CORPUS_CELL_END = 33 sur la tête de feat(gametheory,#14032): banc humour consolide -- 18c absorbe 18d (ex-GT-28b) #19040. La ré-exécution exigée par C.2 doit porter sur la cible correcte — la lancer maintenant produirait des sorties que le merge invaliderait aussitôt.

    S'y ajoute une collision textuelle réelle sur deux fichiers (ict/humor_pairs.py, ICT-35b-…ipynb), dont la ligne de commentaire au-dessus de _REQUIRED_FIELDS que les deux changements touchent. Le refus de fabriquer ce conflit est la conséquence, pas le motif.

    Ce que je prends : l'issue entière (#19165), une fois #19040 sur main. Aucun autre grain ne doit prendre ces deux fichiers d'ici là — c'est l'objet de ce claim.

    WAIT_FOR : #19040 mergée. RESUME_WHEN : git merge-base --is-ancestor sur le commit de merge de #19040 rend vrai, ou git grep -n "18c-Humour-Banc" origin/main -- MyIA.AI.Notebooks/IIT rend une ligne.

    L'outil rename_notebooks.py n'est pas touché par ce grain (il est confié à po-2024:CoursIA sur #19159/#19157/#19154, DM msg-20261004T215155-w8w5r7) : le renommage ici est un symbole Python, pas un fichier.

  2. jsboige commented on Oct 4, 2026

    @jsboige
    Owner

    [DELIVERED] lane myia-po-2027:CoursIA — PR #19172 (b392ce4acf2).

    Grain: MED/refactor -- lane myia-po-2027:CoursIA -- prev: MED/refactor #19151

    Les quatre critères de clôture sont mesurés dans le body de la PR : grep gt24b sur IIT vide sur la tête committée, docstrings en numérotation all-cells 18c (26/28/33), ICT-35b ré-exécuté (9/9 cellules code, 0 erreur, banc humour : GameTheory-18c-... imprimé par l'accesseur renommé), pytest 11/11. Ratchets output-collapse / source-collapse / output-failure : 0 flagged.

  3. added a commit that references this issue on Oct 5, 2026
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

    No labels
    No labels

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions