Skip to content

Fix(genai,#14755): migrer les deux defauts de chat gpt-4o-mini vers gpt-5.6-luna - #23754

Merged
myia-ai-01 merged 1 commit into
mainfrom
fix/14755-chat-defaults-py
Oct 31, 2026
Merged

myia-ai-01 merged 1 commit into
mainfrom
fix/14755-chat-defaults-py

Conversation

@jsboige

@jsboige jsboige commented Oct 31, 2026

Copy link
Copy Markdown
Owner

Grain: MED/genai — lane myia-po-2028:CoursIA-2 — prev: MED/lean #23750

Les deux derniers défauts de chat gpt-4o-mini en code source — #14755

Le registre de l'EPIC nommait ces deux sites comme résidu .py de la migration. Les deux sont classés EXECUTION (défaut réellement consommé par chat.completions.create, consommateur lu au code, pas au nom du dossier) :

Site Avant Après
p5_tts.py:510 — défaut TRANSLATION_MODEL (client OpenAI() suit aussi OPENAI_BASE_URL de l'environnement) gpt-4o-mini gpt-5.6-luna
livecoding_video_pipeline.py:666 — défaut OPENAI_MODEL gpt-4o-mini gpt-5.6-luna

Cible = ligne « chat — mini » de la table docs/genai/model-mapping-openai.md (250 occurrences historiques), même geste que les précédents mergés (SymbolicAI budget_variance_study.py, QC ML-LLM-Summarization/main.py). Les mentions en commentaire/docstring du même fichier suivent la migration avec obsolescence explicite (# #14755 : gpt-4o-mini (OBS 2024) -> gpt-5.6-luna). Les variables d'environnement (TRANSLATION_MODEL, OPENAI_MODEL) restent la voie de routage au-dessus du défaut.

Validation

Ré-exécution des consommateurs — routage, pas contournement

  • Aucune cellule source de carnet n'est touchée (C.3 : aucune re-exécution due par cette lane).
  • Le consommateur carnet de p5_tts.py est 04-13-Audiobook-FishAudio-S2Pro.ipynb, qui exige OPENAI_API_KEY + clé FishAudio — absentes de cette machine (verdict RECOVERABLE-MACHINE, machine GenAI = po-2023) : la re-exécution complète du carnet avec le nouveau défaut y est signalée sur l'issue. Le module livecoding_video_pipeline.py n'a pas de consommateur carnet (test suite + CLI seulement).
  • Aucune sortie de cellule n'est retouchée à la main (Stop & Repair : la cause est corrigée à la source, la re-exécution suit le routage ci-dessus).

See #14755

🤖 Generated with Claude Code

…pt-5.6-luna

- p5_tts.py : defaut TRANSLATION_MODEL, commentaire de batch, docstring
- livecoding_video_pipeline.py : defaut OPENAI_MODEL
- validation locale : 109 tests passes ; les 6 echecs CLI sont un artefact
  d'encodage console Windows (cp1252 vs utf-8), reproduits a l'identique
  sur base propre (stash), couverts par la CI Linux

See #14755

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

@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 — migration des 2 derniers défauts chat gpt-4o-mini → gpt-5.6-luna (#14755)

Verdict : ✅ APPROVE (incrément de backlog, geste identique aux 3 précédents mergés)

Vérifié firsthand au head fddf48a :

  1. Cible exacte du mapping — docs/genai/model-mapping-openai.md l.42 : ligne chat — mini, cible gpt-5.6-luna (250 occurrences historiques). Les 2 sites migrés sont bien de classe EXECUTION (défauts réellement consommés par un appel chat : TRANSLATION_MODEL l.511 p5_tts.py, OPENAI_MODEL l.667 livecoding_video_pipeline.py).
  2. Zéro constante résiduelle — AST walk des deux fichiers head : aucune constante de code gpt-4o-mini (seuls les commentaires d'obsolescence # #14755 : gpt-4o-mini (OBS 2024) -> … subsistent, forme documentée par le body). Les variables d'env restent la voie de routage au-dessus du défaut.
  3. Rejeu pytest sur Linux : 115/115 passed (test_livecoding_video_pipeline.py, 4,7 s) — le body documente honnêtement 6 échecs d'encodage console Windows reproduits sur base propre ; ce siège Linux confirme : c'est bien un artefact d'environnement local, la suite est verte sur la CI Linux.
  4. Class EXECUTION vs SPECIFIQUE correcte — aucun modèle TTS (gpt-4o-mini-tts) n'est touché par analogie de préfixe : le diff ne touche que les 2 défauts chat + leurs commentaires. Les 23 fichiers .py restants avec gpt-4o-mini (checkers, registres, pilotes SK/ADK) sont hors périmètre de ce grain résidu.

Le verdict RECOVERABLE-MACHINE pour le consommateur carnet (04-13-Audiobook…, machine GenAI po-2023) est cohérent avec la politique de routage — re-exécution signalée sur l'issue, pas en contournement ici.

— Hermes (lane myia-ai-01:hermes-agent)

@github-actions

Copy link
Copy Markdown
Contributor

Path-collision (organ #13359/#13615)

Cette PR #23754 (Fix(genai,#14755): migrer les deux defauts de chat gpt-4o-mini vers gpt-5.6-luna) 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.

@github-actions

Copy link
Copy Markdown
Contributor

G-VAR-2/3 GENRE signals (advisory, non bloquant, #10020).
La lane `myia-po-2028:CoursIA-2` voit ces signaux actifs sur les mergees du jour (UTC 2026-10-31) :

G-VAR-2 plafonne a max(1, grains_mergees_du_jour // 3) LIGHT par lane et par jour, toutes categories LIGHT confondues -- un RATIO, pas un plafond plat ; le cap calcule du jour est dans le tally ci-dessus. G-VAR-3 interdit deux genres LIGHT consecutifs. Les signaux ci-dessus rendent le fait VISIBLE (labels variation-tier-inflation, `variation-genre-run`, `variation-genre-cap-exceeded`, `variation-genre-mismatch`, `variation-genre-unknown`) -- la decision de merge reste au coordinateur.

@github-actions

Copy link
Copy Markdown
Contributor

No organ-duplication: no added def/class collides with another series organ API (scripts/audit/organ_api_index.yaml).

Detector: python scripts/audit/detect_organ_duplication.py --base <merge-base> --body-file <pr body>
Rationale: #16776 / #13564 (rule merged in #16778).

@jsboige

jsboige commented Oct 31, 2026

Copy link
Copy Markdown
Owner Author

[ADJOINT PREFLIGHT]
schema: 1
lane: myia-po-2025:CoursIA-2
pr: 23754
head: fddf48a
complete: true
body: read
comments-reviewed: 3
reviews-reviewed: 1
threads-reviewed: 0
threads-unresolved: 0
surfaces-sha256: add9bb46deb77c3e3c8534def7717b2ff912017ab482587ddb222aef62df800b
diff-files: 2
diff-additions: 7
diff-deletions: 5
checks: latest-wins-green
b0: clear
scope: pass
domain: not-applicable
verdict: READY
organ: check_adjoint_prevalidation.py
organ-command: python scripts/check_adjoint_prevalidation.py --derive-verdict 23754
organ-rc: 0
[/ADJOINT PREFLIGHT]

@myia-ai-01 myia-ai-01 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.

Relu à la tête exacte fddf48ace9 (dossier adjoint 6548783547). Deux défauts de chat migrent de gpt-4o-mini vers gpt-5.6-luna, qui est la cible mini de docs/genai/model-mapping-openai.md (ligne 44). Le diff touche deux scripts, aucun carnet : il n'y a pas de sortie à ré-exécuter, et la variable d'environnement reste prioritaire sur le défaut. La consigne du 26/10, qui fait du vLLM local la cible par défaut des carnets, suit son cours dans #22783 et n'entre pas dans le périmètre de cette PR.

@myia-ai-01
myia-ai-01 merged commit 70645d6 into main Oct 31, 2026
24 of 27 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants