Skip to content

feat(notebooks,#13934): brancher-plusieurs-providers — 6 lectures ancrees aux sorties (1004->1394) - #14048

Merged
myia-ai-01 merged 1 commit into
mainfrom
feature/aew-brancher-providers-enrich
Sep 1, 2026
Merged

myia-ai-01 merged 1 commit into
mainfrom
feature/aew-brancher-providers-enrich

Conversation

@jsboige

@jsboige jsboige commented Sep 1, 2026

Copy link
Copy Markdown
Owner

Grain: MED/notebook-python — lane myia-po-2027:CoursIA — prev: MED/notebook-dotnet #14043 (substance distincte : notebook Python d'API multi-provider AI Engine, et non GameTheory-05 C#)

Sujet

brancher-plusieurs-providers-par-l-api.ipynb sortait à 1004 c/cell (seuil 1200) et portait un run de 4 cellules code consécutives ([13]-[16]) sans aucune interprétation entre elles, plus les cellules [7]/[8]. Les sorties committées — 1 modèle qwen3.6-35b-a3b (8192/4096 tokens), poignée provider=custom, env inexistant False « Environment not found. », ai_envs 2 entrées après déclaration, bascule default_env vllm-secours puis remise vllm-local, purge — n'étaient lues par aucune cellule markdown.

Ajout de 6 cellules markdown « Lecture » ADDITIVES, chacune ancrée aux valeurs committées qu'aucune autre cellule ne lit :

# Position (après) Contenu ancré (valeurs committées)
1 cell. 7 (catalogue ai/models) 1 seul modèle qwen3.6-35b-a3b, features=['completion'], contexte 8192/4096, prix 0/0, tags ['core','chat'] — le catalogue comme source de vérité
2 cell. 8 (poignée ai/test_connection) success=True, provider=custom, 1 modèle ; env inventé → False « Environment not found. » — l'API valide l'envId, échec explicite
3 cell. 13 (déclarer vllm-secours) recopie copy.deepcopy de vllm-local, update=True, ai_envs → 2 entrées — read-modify-write (contre-mesure du piège §4)
4 cell. 14 (interroger le secours) même modèle ['qwen3.6-35b-a3b'], connect True « Found 1 models. » — enveloppe de repli, même backend
5 cell. 15 (basculer l'usage chat) default_env vllm-local → vllm-secours → vllm-local, model suit l'env — inversion maîtrisée
6 cell. 16 (purger le secours) envs final=['vllm-local'], default_env=vllm-local, module_embeddings=True, connexion OK — réversibilité du read-modify-write, instance sans état résiduel

Validation

  • Code byte-identique : 12/12 cellules code inchangées (source + outputs + execution_count + metadata comparés JSON-normalisés contre origin/main, sans text=True — pas de faux mojibake cp1252). Enrichissement markdown-only, exempt de re-exécution (C.2/C.3 exception modifs uniquement markdown) ; pre-commit H.3 passé (cellules code conservent execution_count + outputs).
  • Densité : pedagogy_density.py 1004 → 1394 c/cell (seuil 1200 atteint, « 0 below ») ; 26 → 32 cellules ; 6 insertions, 0 deletions.
  • detect_consecutive_code_cells.py : Run >= 2 = 0 — le run de 4 consécutives [13]-[16] est corrigé par cette PR (défaut pré-existant signalé, résolu en ajoutant l'interprétation manquante entre chaque cellule).
  • Guards verts : detect_markdown_rendering.py 0 violation · detect_code_in_markdown_cells.py 0 nouveau vs baseline (2) · scan_cell_ordering.py 0 finding · check_interp_positioning.py 0 finding.
  • Pas de twin pair (aucune entrée twin_pairs.d ne référence ce notebook) ; catalogue intact (catalog-pr-hygiene).
  • Choix du grain : picker sans candidat CONTENU non-VOID ; scan famille AI-Engine-WordPress à densité < seuil → les items basse-densité y sont déjà interprétés (auditer-la-conformite-visuelle : 843 c/cell = artefact 9273/11, interprétation complète) → pivot vers ce notebook, seul à détenir un vrai gap (run de 4 consécutives + sorties non lues). REFUS documenté de l'item 2 (auditer) sur l'issue Enrichissement AI-Engine-WordPress : 1 notebook restant sous la norme 1200 (parler-au-chatbot — po-2026) #13934, commentaire 5493162598.

See #13934

…rees aux sorties (1004->1394)

Enrichissement markdown-only ADDITIF : 6 cellules 'Lecture' lisent les sorties
commitees qu'aucune cellule markdown ne lisait (catalogue qwen3.6-35b-a3b
8192/4096, poignee custom + env inexistant, ai_envs 2 entrees, bascule
default_env vllm-secours puis vllm-local, purge reversible). Corrige le run de
4 cellules code consecutives [13]-[16]. Code byte-identique (12/12), zero
re-execution (C.2/C.3 exception markdown). See #13934.
@github-actions

github-actions Bot commented Sep 1, 2026

Copy link
Copy Markdown
Contributor

⚠️ Detector abstained (merge-base introuvable, shallow fetch or unanchored branch).

c.415 (#11873): scope = notebooks CHANGED in this PR, not the whole corpus.
See python scripts/check_markdown_claims_output.py --help for re-running locally.
Detector rationale: c.290 / c.331 / PR #11435 pathologie.

@github-actions

github-actions Bot commented Sep 1, 2026

Copy link
Copy Markdown
Contributor

Golden-Set Execution (H.7 P3)

✅ 8/8 notebooks passed (certified reproducible)

Notebook Status Time
2.1-Workflow-ML.ipynb ✅ SUCCESS 4.0s
2.2-Descente-de-gradient.ipynb ✅ SUCCESS 4.0s
2.3-Regression-lineaire-logistique.ipynb ✅ SUCCESS 4.8s
2.4-Arbres-Forets-Ensembles.ipynb ✅ SUCCESS 4.6s
Search-1-StateSpace.ipynb ✅ SUCCESS 3.6s
SL-1-LogicalLearning.ipynb ✅ SUCCESS 2.6s
rl_4_multi_armed_bandits.ipynb ✅ SUCCESS 24.2s
GameTheory-04c-NashExistence-Python.ipynb ✅ SUCCESS 3.2s

Pinned lockfile: scripts/notebook_tools/golden_set.lock.txt (H.7 P3, axe A #4208)

@github-actions

github-actions Bot commented Sep 1, 2026

Copy link
Copy Markdown
Contributor

Notebook PR Validation: PASS

  • Notebooks checked: 1
  • Code cells validated: 12
  • Result: All passed

Checks: H.1 (no errors), H.3 (execution_count), C.1 (no banned patterns)
Non-Python kernels (.NET/Lean): C.1 + errors only (execution_count advisory)
QuantConnect notebooks: C.1 + errors only (require QC Cloud for execution)

@myia-ai-01
myia-ai-01 merged commit 9cc12cc into main Sep 1, 2026
60 of 61 checks passed
@jsboige
jsboige deleted the feature/aew-brancher-providers-enrich branch September 2, 2026 13:13
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.

2 participants