Skip to content

PR #24460 -- cloture formelle de la reserve de revue (traitement en code fait) #24688

Open

Description

@jsboige

Suivi de la clôture formelle de la réserve de revue posée sur la PR #24460.

Ce qui est déjà fait (traitement en code)

La réserve porte sur trois points du carnet MyIA.AI.Notebooks/GenAI/Texte/10_LocalLlama.ipynb :

  1. le cliquet « une sortie = une lecture » : la PR ajoutait une seconde cellule de lecture pour la sortie de la cellule de code 25 ;
  2. le mélange de deux plateformes dans la même section (OpenAI / OpenRouter) ;
  3. la formule « la section précédente », qui visait une section située deux cellules plus haut.

Les trois sont traités au commit dfc5c8168b8cb35c71bb87b19ab2c6d429736f5d (tête de la PR) : le contenu de la lecture ajoutée est fusionné dans la lecture unique de la sortie (cellule 23), qui devient l'unique lecture de cette sortie.

Vérifications firsthand (rejouées sur la tête dfc5c8168b8cb) :

  • cliquet relu : python scripts/notebook_tools/check_split_reading_cells.py "MyIA.AI.Notebooks/GenAI/Texte/10_LocalLlama.ipynb" --head dfc5c8168b8cb35c71bb87b19ab2c6d429736f5d --base-ref origin/main --json → base_total: 1, head_total: 1, delta: 0, added: [], regressed: false ;
  • note 2 : la cellule 23 énonce explicitement que les limites citées valent pour le parcours OpenRouter et que le run committé tape l'API OpenAI ;
  • note 3 : plus aucune occurrence de « précédent » / « précédente » dans la cellule 23 (base et tête : 0) ;
  • le cliquet de la base (origin/main) reste inchangé : la PR n'ajoute aucun finding.

Ce qui reste à faire (hors de portée de la lane)

La levée formelle de la réserve : soit la persona de relecture réexamine la tête et referme sa propre réserve, soit le coordinateur pose une levée nommée.

Une lane worker ne peut pas lever une réserve de tiers : B.0 réserve l'arbitrage écrit au coordinateur (règle pr-review-discipline.md, et la borne d'auteur de l'organe scripts/check_unaddressed_nits.py).

Merci de garder cette issue ouverte jusqu'au merge de la PR #24460 — elle est le support du report explicite de la clôture.

Lane : myia-ai-01:CoursIA-2.

Activity

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