Skip to content

Fix(genai-image,#17926): examples -- chaine de navigation history -> literature -> science rattachee - #20620

Open
jsboige wants to merge 2 commits into
mainfrom
claude/17926-image-examples-nav
Open

jsboige wants to merge 2 commits into
mainfrom
claude/17926-image-examples-nav

Conversation

@jsboige

@jsboige jsboige commented Oct 14, 2026

Copy link
Copy Markdown
Owner

See #17926

Summary

  • La série MyIA.AI.Notebooks/GenAI/Image/examples/ comptait une chaîne mutualisée (literature-visual <-> science-diagrams) que rien ne reliait à l'entrée history-geography : check_notebook_nav_chain rendait un finding independent_chain sur cette série.
  • Correction par markdown seul : la ligne de navigation de la cellule 0 de deux carnets reçoit le lien manquant, au format canonique de GenAI/Image/ (Index | << Précédent | Suivant >>).
  • Ordre pédagogique retenu : history-geography -> literature-visual -> science-diagrams. C'est l'ordre des tableaux de GenAI/Image/README.md et de examples/README.md, celui des en-têtes, et le lien literature -> science déjà présent.
  • Aucune cellule de code touchée, pas de ré-exécution, aucun *_en.ipynb touché.

Changes

Fichier Cellule Avant Après
examples/history-geography.ipynb 0 (markdown) **Navigation** : [Index](../README.md) **Navigation** : [Index](../README.md) | [Suivant >>](literature-visual.ipynb)
examples/literature-visual.ipynb 0 (markdown) **Navigation** : [Index](../README.md) | [Suivant >>](science-diagrams.ipynb) **Navigation** : [Index](../README.md) | [<< Précédent](history-geography.ipynb) | [Suivant >>](science-diagrams.ipynb)

science-diagrams.ipynb n'est pas modifié : il porte déjà [<< Précédent](literature-visual.ipynb). Les README ne sont pas touchés : un lien README ne crée pas d'arête pour le détecteur (build_graph ne lit que les cellules markdown des carnets).

Verdicts

# Constat (sortie du détecteur) Verdict Preuve
A independent_chain sur GenAI/Image/examples CONFIRMED, corrigé Avant : entries = [history-geography], chain_starts = 2, independent_chains = 2 ; history-geography n'avait aucune arête sortante intra-série. Après : independent_chains = 0, unreachable = [].
B Autres findings de GenAI/Image (orphan_entry sur deux *_en.ipynb de 01-Foundation) hors périmètre Traductions dérivées, non modifiables à la main ; présents dans le baseline.

Forme de la série après correction (point d'attention pour le relecteur). Chaque carnet a désormais un lien entrant intra-série : le détecteur ne trouve plus d'entrée et classe la série comme chaîne bouclée (wrapped), jugée depuis history-geography, qui atteint les trois carnets : zéro finding pour la série. C'est la forme mutualisée (#17277 : Suivant et Précédent sur chaque paire), déjà utilisée dans GenAI/Image/01-Foundation et 04-Applications. Une variante à sens unique (sans << Précédent dans literature) garderait une entrée unique mais casserait cette forme : choix laissé à l'arbitrage du mainteneur.

Validation

$ python scripts/notebook_tools/check_notebook_nav_chain.py --json   # avant / après, série GenAI/Image/examples
avant : independent_chain present (entries=[history-geography], independent_chains=2)
après : independent_chains=0, unreachable=[], wrapped=True ; 0 NEW finding pour la série
$ python scripts/notebook_tools/check_notebook_navlinks.py <les deux carnets>
0 lien casse
nbformat.validate : OK sur les deux carnets
$ python scripts/notebook_tools/check_activity_leak.py --diff origin/main...HEAD
[OK] aucun passage d'activite dense ajoute

Le baseline partagé scripts/tests/baseline_nb_nav_chain.json n'est pas modifié : l'entrée résolue y apparaîtra en INFO « résolu » au --check ; sa mise à jour relève de l'outillage du baseline. Les vérifications de liens du worktree de travail (checkout partiel) ont signalé des link_404 vers GenAI/Video et SymbolicAI/Lean, dossiers absents de ce checkout mais suivis par git : artefact du checkout, sans rapport avec cette PR.

Review Checklist

  • 1. Scope réel : deux lignes de navigation, cellule 0 de deux carnets.
  • 2. Validation post-fix : détecteur, navlinks, nbformat, activity-leak relancés après le commit.
  • 3. Cohérence pédagogique : ordre des tables README et des en-têtes.
  • 4. Exécution réelle : N/A, markdown seul (C.2).
  • 5. Regression check : aucun nouveau finding de navigation pour la série ; aucun lien cassé.

Hors flotte : session cloud du mainteneur (exemption #17713), pas de lane ni de tag Grain.

🤖 Generated with Claude Code

https://claude.ai/code/session_01LRbWZZHE9jZscr8ya3niWA


Generated by Claude Code

…ependante)

La serie comptait deux chaines mutualisees (literature-visual et
science-diagrams, liees entre elles) sans aucun lien depuis l'entree
history-geography : check_notebook_nav_chain rendait independent_chain.

Ajout de la ligne Navigation canonique (Index | << Precedent | Suivant >>)
dans la cellule 0 markdown des deux carnets, au format de GenAI/Image
(01-Foundation, 04-Applications). Ordre pedagogique history-geography ->
literature-visual -> science-diagrams : celui des tableaux du README, des
en-tetes (memes prerequis, durees 30/30/35 min) et du lien literature ->
science deja present.

Verdicts :
- independent_chain sur examples : CONFIRMED (avant : 2 chaines
  independantes, 0 noeud atteint depuis history-geography) -> corrige,
  apres : basis = history-geography, 0 unreachable, 0 NEW finding.
- link_404 de 04-Applications/04-5-MiniMax-Cloud-Image : hors de cette
  serie, non traite dans ce commit.

Markdown seul : aucune cellule de code touchee, pas de re-execution.
Aucun fichier *_en.ipynb touche.

See #17926

Co-Authored-By: Claude Haiku 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01LRbWZZHE9jZscr8ya3niWA
@github-actions

Copy link
Copy Markdown
Contributor

Notebook outputs-required (H.4 schema): PASS (every code cell carries an outputs: list)

@github-actions github-actions Bot added the consecutive-code-cells Modified notebook has >=2 consecutive code cells (#12797) label Oct 14, 2026
@github-actions

Copy link
Copy Markdown
Contributor

✅ No prose/output mismatch detected in the notebooks this PR changed.

Scope = notebooks CHANGED in this PR, not the whole corpus. Explicit claim-check relations resolve only against named CLAIM_METRICS from the local output window and are classified SUPPORTED, CONTRADICTED, or UNPROVEN.
The markdown-claims-output-report run artifact contains the structured JSON report. See python scripts/check_markdown_claims_output.py --help for re-running locally.
Detector rationale: c.290 / c.331 / PR #11435 numeric pathology, extended with low-noise relational evidence.

@github-actions

Copy link
Copy Markdown
Contributor

✅ No unanchored measurement claim detected in the notebooks this PR changed.

Scope = notebooks CHANGED in this PR, not the whole corpus. The stale-claim-report run artifact holds the structured JSON.
Rationale: the sibling detector above only compares a claim to the outputs of the cells that PRECEDE it; a claim written in a cell that precedes its code (App-5-Timetabling c.2/c.4) is invisible to it, and a value imported from a twin notebook is never produced locally. See python scripts/check_stale_claims.py --help.

@github-actions

Copy link
Copy Markdown
Contributor

✅ No factual mislabel detected in the notebooks this PR changed (entity counts and tuple formulas checked against nearby committed streams).

Scope = notebooks CHANGED in this PR, not the whole corpus. The factual-mislabel-report run artifact holds the structured JSON.
Rationale: pure ABSENCE of a claimed value is the sibling stale-claim detector's job; this one only reports CONTRADICTIONS between an adjacent code cell's stream and the markdown that describes it. See python scripts/check_factual_mislabel.py --help.

@github-actions

Copy link
Copy Markdown
Contributor

Golden-Set Execution (H.7 P3)

✅ 9/9 notebooks passed (certified reproducible)

Notebook Status Time
2.1-Workflow-ML.ipynb ✅ SUCCESS 4.1s
2.2-Descente-de-gradient.ipynb ✅ SUCCESS 13.2s
2.3-Regression-lineaire-logistique.ipynb ✅ SUCCESS 4.7s
2.4-Arbres-Forets-Ensembles.ipynb ✅ SUCCESS 4.0s
Search-01-StateSpace.ipynb ✅ SUCCESS 3.1s
SL-1-LogicalLearning.ipynb ✅ SUCCESS 2.0s
RL-04-Bandits-Manchots-Python.ipynb ✅ SUCCESS 16.6s
GameTheory-04c-NashExistence-Python.ipynb ✅ SUCCESS 2.6s
GameTheory-13d-Optimistic-CFR-Python.ipynb ✅ SUCCESS 9.0s

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

@github-actions github-actions Bot added the variation-tag-missing PR sans tag Grain: <TIER>/<GENRE> (variation-protocol) label Oct 14, 2026
@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 commented Oct 14, 2026

Copy link
Copy Markdown
Owner Author

Check rouge Always-on guards -- 16 organes, 1 checkout sur la tête e41d16771a : il ne vient pas de cette PR.

Hors flotte : session cloud du mainteneur (exemption #17713).


Generated by Claude Code

@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.

VERDICT: LGTM

[Hermes] Review — head e41d16771a5 (promotion tier-âgé v1.1 : cr 00:43Z, 21 h, 0 review, 0 commentaire non-auteur).

Vérifications exécutées :

  • Diff markdown-only exact : 2 lignes de navigation (cellule 0 de history-geography et literature-visual), aucun code/output touché. Scan sécurité : 0 hit.
  • Chaîne re-derivée au head, cellule par cellule : history-geography → Suivant literature-visual ✓ ; literature-visual → Précédent history-geography + Suivant science-diagrams ✓ ; science-diagrams (non modifié, déjà sur main) → Précédent literature-visual ✓. Bidirectionnelle et fermée sur les deux extrémités — le finding independent_chain de check_notebook_nav_chain est mécaniquement levé.
  • Cibles vivantes : les 3 carnets existent au head dans examples/ (listing vérifié), format Index | << Précédent | Suivant >> conforme au canon GenAI/Image/, pas de lien mort.
  • Ordre pédagogique : history → literature → science = l'ordre des tableaux de #17926, déclaré dans le corps de PR.

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

@github-actions

Copy link
Copy Markdown
Contributor

Notebook PR Validation: PASS

  • Notebooks checked: 2
  • Code cells validated: 24
  • 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 added the pr-gate-missing PR gate absent du rollup: contexte requis jamais rapporte, PR bloquee, checks verts (#10928) label Oct 15, 2026
@github-actions

Copy link
Copy Markdown
Contributor

PR gate absent du rollup (advisory, #10928)

PR gate est absent du rollup de cette PR et la cause n'est pas determinee : les mesures suivantes ont ete faites, aucune ne tranche.

  • mergeable_state = blocked (pas dirty) ;
  • aucun evenement base_ref_changed dans la timeline ;
  • le sujet du commit de tete ne porte pas le token [skip ci] ;
  • auteur : jsboige (pas une PR bot).

Un remede au hasard coute un commit sans effet (issue #14477 : la prescription est fonction de la cause). Signaler ce cas sur le dashboard de coordination pour investigation manuelle -- c'est le cas non identifie #10902 qui reste en suspens.

Cause mesuree : mergeable_state=blocked, pas de base_ref_changed, sujet sans [skip ci], auteur jsboige

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

consecutive-code-cells Modified notebook has >=2 consecutive code cells (#12797) pr-gate-missing PR gate absent du rollup: contexte requis jamais rapporte, PR bloquee, checks verts (#10928) variation-tag-missing PR sans tag Grain: <TIER>/<GENRE> (variation-protocol)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants