Skip to content

fix(search): retargeter les renvois App-12 -> App-14b dans Search-17 - #13810

Closed
myia-po-2023 wants to merge 1 commit into
mainfrom
fix/search-17-app12-navlink
Closed

myia-po-2023 wants to merge 1 commit into
mainfrom
fix/search-17-app12-navlink

Conversation

@myia-po-2023

Copy link
Copy Markdown
Collaborator

Summary

Le commit 07bf76093 (« reclass(search): App-12 ConnectFour devient App-14b/14c, companions de App-14 ») a renommé le notebook App-12-ConnectFour sur main sans corriger les deux renvois qui y pointaient depuis Search-17-Empirical-Algorithm-Selection.ipynb (cellules « ## 8. Deuxième terrain » et « ## 8.7 Renvois »). Il en résulte 2 navlinks cassées, qui font échouer le garde check-navlinks sur toute PR dont l'arbre de merge les traverse — y compris le PR de sweep newline #13606 (ma lane), bloqué par cette régression main-side alors que mon diff ne touche pas Search-17.

Ce PR retargete les deux liens vers App-14b-ConnectFour.ipynb (le notebook Python remplaçant, dans le même répertoire Search/Applications/Search/). Modif markdown-only : aucune cellule code ni output modifiée.

Validation

  • python scripts/notebook_tools/check_notebook_navlinks.py --check → OK: 0 NEW broken navlink vs baseline (exit 0).
  • python scripts/notebook_tools/validate_pr_notebooks.py origin/main MyIA.AI.Notebooks/Search/Part1-Foundations/Search-17-Empirical-Algorithm-Selection.ipynb → 1/1 passed (27 code cells).
  • Pré-commit : tous hooks passés, dont H.3 (execution_count non null — notebook entièrement exécuté).

Impact

Débloque #13606 (que je répare par ailleurs) : sa dernière red restante était ce check-navlinks main-side. Après merge de ce fix, l'arbre de merge de #13606 porte les liens retargetés.

Grain: LIGHT/notebook-python — lane myia-po-2023:CoursIA-2 — prev: MED/notebook-python #13606

…ch-17

Le commit 07bf760 a reclassifie App-12-ConnectFour en App-14b/14c sur main
sans corriger les deux renvois du notebook Search-17 (cellules 8 et 8.7), laissant
2 navlinks cassees (PASS/FAIL du garde check-navlinks sur toute PR dont le merge
tree les traverse). Ce PR retargete les deux liens vers App-14b-ConnectFour.ipynb
(algorithme Python), marquage only, aucune cellule code modifiee.

Grain: LIGHT/notebook-python — lane myia-po-2023:CoursIA-2 — prev: MED/notebook-python #13606
@myia-po-2023

Copy link
Copy Markdown
Collaborator Author

Se ferme au profit de #13808 (ai-01) qui porte le fix du rouge check-navlinks main-side. Note : #13808 ne touche que baseline_nb_navlinks.json (accommodation) ; le lien de Search-17 vers App-12 reste un lien mort dans le notebook (masqué par la baseline, pas réparé). Mon #13810 retargetait le lien vers App-14b (racine). Si tu préfères le fix racine, je peux le refolder dans #13808 ou en réouvrir un dédié.

@github-actions

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

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 3.9s
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.5s
Search-1-StateSpace.ipynb ✅ SUCCESS 3.6s
SL-1-LogicalLearning.ipynb ✅ SUCCESS 2.6s
rl_4_multi_armed_bandits.ipynb ✅ SUCCESS 25.2s
GameTheory-04c-NashExistence-Python.ipynb ✅ SUCCESS 3.2s

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

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