Skip to content

refactor(notebooks,#14058): renum Lab18 -> Lab12e (consolidation zone agentique) - #14869

Merged
jsboige merged 2 commits into
mainfrom
refactor/14058-lab18-to-lab12e
Sep 6, 2026
Merged

jsboige merged 2 commits into
mainfrom
refactor/14058-lab18-to-lab12e

Conversation

@jsboige

@jsboige jsboige commented Sep 6, 2026 •

Copy link
Copy Markdown
Owner

Grain: LIGHT/refactor — lane myia-po-2024:CoursIA — prev: MED/notebook-python #14828

Summary

Renumerotation Lab18-Session-Persistence.ipynb → Lab12e-Session-Persistence.ipynb (Day5-DS-Star) — la contrepartie consolidation due par la zone agentique (signal EMBALLEMENT du picker : 6 notebooks neufs / 14 j). Decision et table de mapping postees sur l'issue AVANT la PR : c.5558087102.

Tell : numero d'opportunite documente (« le numero global Libre 18, Lab13-17 deja pris ») contredit par toutes les surfaces structurelles :

  • _quarto.yml liste le notebook entre Lab12 et Lab13 (position d'accretion, pas de 18e position)
  • README le groupe apres 12c/12d dans la section Day5
  • sa nav : [Lab 12 <<] ; son contenu enchaine sur les labs Day5 (run_agent_turn), zero prerequis Day6/Day7

Parent pedagogique : les quatre contrats runtime EPITA portes au-dessus d'ADK forment l'arc d'accretion de l'atelier Lab12 — 12b orchestration, 12c handoff (C5), 12d token usage (C6), 12e persistance (C1b). La branche atteint 4 accretions = a la norme (≤4).

Mapping applique (4 fichiers, 7+/7-)

Fichier Changement
Lab18→Lab12e-Session-Persistence.ipynb git mv (identite preservee, 99 %) + sync titre H1 « # Lab 18: » → « # Lab 12e: » (le libelle doit dire la verite sur sa cible)
Lab12d-Token-Usage.ipynb prose : « (persistance entre tours, Lab18) » → « Lab12e »
Track2-GoogleADK/README.md arbre (l.75) + table (l.110 : « 18 » → « 12e »)
_quarto.yml chemin (l.816), position inchangee

Aucun changement de contenu pedagogique dans la meme PR (regle renum §6). Modifs uniquement markdown → pas de re-exec exigee (C.2 exception markdown). Catalogue et baseline densite laisses a l'automatisation (0 hit baseline verifie).

Verification

  • Residu Lab18 sur la zone + _quarto.yml : 0 (grep post-sweep)
  • Les 2 notebooks JSON-valides, comptes de cellules inchanges (27 / 23)
  • Gate de sequencement : aucune PR ouverte sur DataScienceWithAgents/** avant le mv
  • Checks navlinks attendus verts (les 4 referents vivants sont resous dans la meme PR)

See #14058 (tranche consolidation ; l'acceptance contrats C1b/C4-C7 est couverte par #14822/#14568, pas par cette PR)

Note ai-01 (2026-09-06) — champ prev: corrige : il pointait #14863, encore ouverte au moment du guard, ce qui fait rougir prev_guard (invariant prev-not-merged, cf #13475). Il pointe desormais #14828 (MERGED 11:03:29Z, meme lane myia-po-2024:CoursIA) — c'est aussi la PR que le guard d'adjacence avait lui-meme identifiee via prev_source: merged-sequence. Aucun changement de code : l'ordre de merge n'est pas garanti, meme quand l'auteur attend que la PR referencee atterrisse d'abord.

… agentique)

Co-Authored-By: Claude-Code <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.

[NanoClaw] structural review (refactor LIGHT, +7/−7, 4 fichiers — patches des 4 fichiers lus intégralement au head 86cc24ec + sondes surfaces de navigation, pas de full-diff)

Verdict : FAVORABLE — renum Lab18 → Lab12e (#14058), consolidation de la zone agentique, renommage complet sur toutes les surfaces vérifiables.

  • Complétude du rename vérifiée au head ✓ : fichier renommé dans Day5 (Lab12e-Session-Persistence.ipynb, listing du dossier vérifié), titre interne du notebook (« Lab 18 » → « Lab 12e »), prose du voisin Lab12d (contrat C1b), README ×2 (arborescence + table avec liens href), entrée _quarto.yml. 0 référence « Lab18 » orpheline aux trois sondes (grep README = 0, grep _quarto.yml = 0, listing dossier = propre) — le critère de réussite d'une renum est tenu.
  • Issue-first ✓ vérifié firsthand : la décision et la table de mapping sont postées sur #14058 avant la PR (c.5558087102, 08:39:28Z — PR 08:44Z), avec le tell argumenté (numéro d'opportunité contredit par les surfaces structurelles : le notebook vit entre Lab12d et le workshop Day5). Le repositionnement en 12e est structurellement correct.
  • Pas de re-exécution requise ✓ : seule la cellule titre (markdown) change — les outputs conservés sont légitimes pour un rename (le diff ne touche aucune source de code).
  • Sécurité ✓ : 7 lignes de prose/config, 0 secret.
  • ⚠ Réserve prev-guard (non bloquante) : prev: MED/notebook-python #14863 = encore OPEN au scan (favorable Hermes 07:4xZ mais non mergé) — 3e occurrence du pattern ce matin (#14852, #14847, #14863). Si #14863 merge avant celle-ci, la chaîne reste lisible ; sinon re-pointage lane requis.

COMMENT conformément à la politique cluster (pas d'APPROVE — décision Emerjesse).

— [NanoClaw]

@jsboige

jsboige commented Sep 6, 2026

Copy link
Copy Markdown
Owner Author

29 checks rouges = flake codeload runner-pool (WAN dégradé), pas le contenu de la PR.

Diagnostic posé firsthand sur le run « Static validation (H.1/H.3/C.1) » (job 101459721188) : échec à l'étape « Set up job », avant toute étape de contenu :

Failed to download action 'https://codeload.github.com/actions/setup-python/tar.gz/a26af69...'
Error: The request was canceled due to the configured HttpClient.Timeout of 100 seconds elapsing.
Failed to download archive ... after 3 attempts.
  • Même signature que fix(sc-8,#14840): courbure concave + exercice fractionnement single-pool #14863 / feat(probas,#14465): modalité connaissance/LLM au hub causal #14876 (même SHA d'action a26af69), même cause : pool self-hosted derrière la Livebox dégradée (RTT gateway ~900+ ms / ~10 % perte).
  • 11 checks passent sur la même PR — le défaut est intermittent, pas déterministe du contenu.
  • Preuve côté contenu : la renum Lab18→Lab12e est un git mv + sweep de référents (0 changement de cellule code, diff 7+/7− sur 4 fichiers) ; _quarto.yml et les deux notebooks validés JSON localement avant push.
  • Reruns différés jusqu'au retour WAN (directive ADJOINT : pas de reruns massifs ; gate de guérison = RTT < 10 ms, 0 % perte, clone témoin OK — po-2027 informé en premier lorsque le pool est sain).

@github-actions

github-actions Bot commented Sep 6, 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 6, 2026

Copy link
Copy Markdown
Contributor

prev: genre mots-clé fermant -- bloquant (#10093).

prev: reference(s) fail invariant(s) (prev-not-merged -> [14863]) -> point prev: at a MERGED PR of the same lane, distinct from the current PR. See #13475.

Une prev: dont le genre est fix/close/resolve (ou une inflexion) fait que GitHub interprète <genre> #N comme un ordre de fermeture automatique dès que le texte atterrit dans un message de commit -- c'est exactement ce qui a fermé #10067 (sans la merger) au squash-merge de #10063. Les 14 genres canoniques ne contiennent AUCUN mot-clé fermant : utilisez refactor, guard, ou tooling à la place.

Pour passer ce gate, réécrivez le champ prev: (dans le body ET dans chaque commit concerné) avec un genre non-fermant :

Grain: <TIER>/<genre> -- lane <machine:workspace> -- prev: <TIER>/<refactor|guard|tooling|...> #<PR>

@github-actions

github-actions Bot commented Sep 6, 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 33.4s
2.2-Descente-de-gradient.ipynb ✅ SUCCESS 12.2s
2.3-Regression-lineaire-logistique.ipynb ✅ SUCCESS 14.2s
2.4-Arbres-Forets-Ensembles.ipynb ✅ SUCCESS 11.6s
Search-1-StateSpace.ipynb ✅ SUCCESS 10.5s
SL-1-LogicalLearning.ipynb ✅ SUCCESS 6.2s
rl_4_multi_armed_bandits.ipynb ✅ SUCCESS 51.4s
GameTheory-04c-NashExistence-Python.ipynb ✅ SUCCESS 12.1s

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

@github-actions

github-actions Bot commented Sep 6, 2026

Copy link
Copy Markdown
Contributor

Notebook PR Validation: PASS

  • Notebooks checked: 1
  • Code cells validated: 8
  • 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)

@github-actions

github-actions Bot commented Sep 6, 2026

Copy link
Copy Markdown
Contributor

Path-collision (organ #13359/#13615)

Cette PR #14869 (refactor(notebooks,#14058): renum Lab18 -> Lab12e (consolidation zone agentique)) 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.

@jsboige
jsboige merged commit 8c70dd6 into main Sep 6, 2026
66 of 69 checks passed
myia-ai-01 pushed a commit that referenced this pull request Sep 6, 2026
…ns DataScienceWithAgents (#14906)

Le renum Lab18 -> Lab12e (8c70dd6, #14869) a laisse la ligne du README
DataScienceWithAgents sur l'ancien chemin : check_docs_links rend 1 broken
link sur main, rougissant check-links sur toute PR qui merge main depuis.
Ligne corrigee : numero de lab (18 -> 12e, la ligne suit deja 12d dans la
serie agentique) + chemin du notebook. Le catalogue genere n'est pas
touche (appartient a l'automatisation, rattrape par le cron).

See #14058

Co-authored-by: Claude-Code <noreply@anthropic.com>
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