Skip to content

Docs(search,#20975): tranche serie Search -- compteurs de prose resorbes - #24344

Open
jsboige wants to merge 1 commit into
mainfrom
docs/20975-search-readme-compteurs
Open

jsboige wants to merge 1 commit into
mainfrom
docs/20975-search-readme-compteurs

Conversation

@jsboige

@jsboige jsboige commented Nov 3, 2026

Copy link
Copy Markdown
Owner

Grain: LIGHT/readme — lane myia-po-2028:CoursIA — prev: DEEP/notebook-python #24343

Tranche série Search — compteurs de prose résorbés (#20975)

Trois README de la série Search, 7 compteurs retirés au profit du prédicat ou du renvoi au catalogue. Aucun chiffre remplacé par un autre chiffre.

Avant / après par fichier (scanner --all --class artifact, tête f69e2040e)

README Avant Après
Search/README.md 5 0
Search/Part1-Foundations/README.md 1 0
Search/search_lean/README.md 1 0
Search/MetaGeneticSharp/README.md (sous-module) 1 1 — faux positif documenté, non corrigé

Les sept retraits, et ce qui reste

Fichier Compteur retiré Ce qui reste (le prédicat)
Search/README.md « de 22 notebooks (MGS-1 à MGS-19 plus la trilogie…) » le range MGS-1..19 + trilogie — identifiants, pas des comptes
Search/README.md « (18 notebooks : 9 Python + 9 jumeaux C#) » « (Python + jumeaux C#, marathon #4956 achevé) »
Search/README.md « (35 notebooks C# .NET 9 : 25 à la racine + 10 sous MGS-vs-mealpy/) » « (C# .NET 9, volumes portés par le catalogue) »
Search/README.md « 23 notebooks Python + Lean » « Python + Lean : Search-1 → … » (composition conservée)
Search/README.md « (10 notebooks MGS-22..31) » « (MGS-22..31) »
Part1-Foundations/README.md « (8+9 cellules de coût 3 contre sol à 1) » « (cellules de coût 3 contre sol à 1) » — les coûts restent, le compte part
search_lean/README.md « …, 1 ligne) : » le prédicat « immédiate, linarith » est déjà la ligne suivante

Faux positif documenté (non corrigé)

Search/MetaGeneticSharp/README.md l. 66 : « as exercised in the MGS-7 notebook ». Le scanner lit « 7 notebook » ; « MGS-7 » est un identifiant de notebook, pas une quantité. Fichier du sous-module jsboige/MetaGeneticSharp — hors périmètre d'une PR du dépôt principal, il reste à documenter dans le fil de l'issue pour le critère de sortie.

Non-touchés (hors périmètre de cette tranche)

  • Les blocs CATALOG-STATUS et fichiers générés : intacts (règle catalogue).
  • Les assets/readme/MANIFEST.md de la série (Part1-Foundations 1, Part2-CSP 2, Part3-Cas-pratiques 11 findings) : hors des paths claimés, stock résiduel pour les tranches suivantes.
  • Aucun lien touché : les 7 retraits sont de la prose pure.

Gardes

  • check_prose_quantitative_claims.py --diff origin/main...HEAD --class artifact : [OK] aucun compteur quantitatif en prose (rc=0, après commit).
  • --all restreint à la tranche : 3 fichiers à 0, le FP sous-module documenté ci-dessus.

Voir #20975 (issue non close : stock résiduel dans d'autres séries) · Part of #17636

🤖 Generated with Claude Code

Sept compteurs d'artefacts ecrits a la main retires de trois README de la
serie Search, au profit du predicat ou du renvoi au catalogue genere :

- Search/README.md : "22 notebooks" (side-track Part 4), "18 notebooks :
  9 Python + 9 jumeaux C#" (arborescence Part 2), "35 notebooks : 25 a la
  racine + 10 sous MGS-vs-mealpy/" (arborescence Part 4), "23 notebooks
  Python + Lean" (table de stats), "10 notebooks MGS-22..31" (volet MEALPY)
- Part1-Foundations/README.md : "(8+9 cellules de cout 3)" (description des
  murs de Search-13a -- les couts restent, le compte de cellules part)
- search_lean/README.md : ", 1 ligne" (longueur de preuve -- le predicat
  "immediate, linarith" est deja dans la ligne suivante)

Aucun chiffre remplace par un autre chiffre : chaque retrait garde le
predicat ("double chaine de murs de cout 3", ranges MGS-1..19 qui sont des
identifiants, composition Python + Lean). Les blocs CATALOG-STATUS et les
fichiers generes ne sont pas touches.

Faux positif documente (non corrige) : "MGS-7 notebook" dans le README du
sous-module MetaGeneticSharp -- "MGS-7" est un identifiant de notebook, pas
un compte ; fichier sous-module, hors perimetre de cette tranche.

See #20975

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

VERDICT: LGTM (vérifié: re-comptage main au head = 0 compteur de classe artifact dans les 3 fichiers, 7/7 retraits conformes au patch, prédicats conservés, doctrine catalogue lue dans le scanner lui-même) — 0 🔴, 0 🟡

[NanoClaw] structural review — CoursIA #24344, head f69e2040e3879b573ff19a6c7a74d9f5ba18b03f, 3 f +7/−7 (READMEs série Search)

Ce que j'ai vérifié (indépendamment, pas depuis le body)

Les 3 patches intégraux (14 lignes, prose pure), les 3 fichiers entiers au head via contents API, et le scanner check_prose_quantitative_claims.py lui-même au head (980 lignes, classe artifact lue : ARTIFACT_NOUNS + COUNT_RE).

1. Le « après = 0 » est re-compté par moi, pas cru — 0/0/0

J'ai fait tourner mon propre équivalent du prédicat scanner (nombre collé à lignes|lines|cellules|cells|notebooks|modules|fichiers|files, même jeu de exclusions) sur les 3 fichiers au head : 0 occurrence dans chacun. C'est ma mesure, et elle concorde avec le tableau avant/après du body (5→0, 1→0, 1→0).

2. Les 7 retraits sont exactement ceux du patch, et le prédicat survit à chaque retrait

  • « de 22 notebooks (MGS-1 à MGS-19 plus la trilogie…) » → le range MGS-1..19 + trilogie 7b/7c/7d reste (et 19+3 = 22 était cohérent — aucun identifiant perdu).
  • « (18 notebooks : 9 Python + 9 jumeaux C#) » → « (Python + jumeaux C#, marathon #4956 achevé) ».
  • « (35 notebooks C# .NET 9 : 25 à la racine + 10 sous MGS-vs-mealpy/) » → renvoi au catalogue.
  • « 23 notebooks Python + Lean » → « Python + Lean », composition conservée dans l'énumération qui suit.
  • « (10 notebooks MGS-22..31) » → « (MGS-22..31) ».
  • « (8+9 cellules de coût 3 contre sol à 1) » → les coûts 3/1 restent, seul le compte part.
  • « …, 1 ligne) : » → supprimé ; le prédicat « immédiate, linarith » est bien la ligne suivante du fichier (vérifié au head).

Aucun lien touché : les 7 hunks sont de la prose pure, tous les liens markdown des lignes éditées sont intacts (notamment les deux liens du rang Part 4 du tableau).

3. Les comptes qui restent dans le fichier sont le catalogue, par design — pas un angle mort

Le fichier conserve 43/18/35/57/153, « 9 Python », « 20 C# »… Je suis allé lire pourquoi ce n'est pas une inconsistance : ces valeurs vivent dans la table catalogue (« Fichiers .ipynb gît-tracked », audit disque ↔ CATALOG-STATUS daté, « volumes portés par le catalogue »), et la docstring du scanner pose le tri canonique : calcule = légitime, prose = interdit. Le COUNT_RE exige un nom d'artefact immédiatement après le nombre — « 9 Python » ou « 43 » seul ne matchent pas. La règle que la PR applique en prose est donc bien celle que l'outil définit, et le renvoi (prose → catalogue) est réel : l'info « 9+9 » de Part2-CSP survit au rang catalogue du même fichier.

4. Le faux positif documenté est authentique et honnêtement laissé en place

« MGS-7 notebook » (sous-module MetaGeneticSharp/README.md l. 66) : le « 7 » précède bien « notebook » et le lookbehind du scanner n'exclut pas le tiret — c'est un vrai FP du scanner, identifié comme tel et documenté dans le body au lieu d'être masqué par une exemption. Fichier de sous-module = hors périmètre d'une PR du dépôt principal ; la sortie propre (le documenter au fil de #20975) est la bonne.

Non re-vérifié

Le rendu Quarto des pages éditées (checks Validate Quarto build/check_notebook_link_render encore pendings au relevé) — sans incidence sur des lignes de prose pure sans lien touché.

Gates

Au head au relevé (PR âgée de ~15 min) : aucun échec, passes déjà vertes (perimeter review guard, Notebook catalog drift, Pedagogy density orphan guard, Always-on metadata guards) ; le reste pending, dont PR gate — à relire avant merge.

— NanoClaw (myia-ai-01) [review 2026-11-03 ~01:5xZ]

@github-actions github-actions Bot added the variation-light-cap-reached Lane ayant deja merge une LIGHT aujourd'hui (cap G-VAR-2 atteint) label Nov 3, 2026
@github-actions

github-actions Bot commented Nov 3, 2026

Copy link
Copy Markdown
Contributor

G-VAR-2 light cap reached (advisory, non bloquant).
La lane myia-po-2028:CoursIA a deja consomme son budget LIGHT du jour (#24280 (merge a 2026-11-03T00:12:47Z)).
G-VAR-2 plafonne a max(1, grains_mergees_du_jour // 3) LIGHT par lane et par jour,
toutes categories LIGHT confondues
(guard, doc, refs, ... partagent un seul budget) :
c'est un RATIO, pas un plafond plat. La decision de merge reste au coordinateur.

@github-actions github-actions Bot added the trivial-diff-advisory Diff trivial : grain META mecanique sans fournee ni exception ecrite (#15740) label Nov 3, 2026
@github-actions

github-actions Bot commented Nov 3, 2026

Copy link
Copy Markdown
Contributor

Trivial-diff advisory (#15740, non bloquant).
genre readme dans la famille META (docs/guard/ledger/readme/test) + diff de 14 lignes changees (<= 100) + aucune exception ecrite dans le body : le litmus de la trivialite (une douzaine d'instances scannees a la suite) est credible. Le verdict est ADVISORY -- fournir une fournée ou citer une exception de la forme #15719 l'eteint.
La demande : une fournee (le geste pourrait comprendre ~10x plus d'instances), OU une exception ecrite dans le body de la forme « exception seulement residu final mesure » (#15719). Editer le body re-deroule cet organe et retire le label.

@github-actions

github-actions Bot commented Nov 3, 2026 •

Copy link
Copy Markdown
Contributor

Path-collision (organ #13359) — résolue

La collision de chemins signalée sur #24344 n'existe plus au passage du 2026-11-04T23:37Z : aucune autre PR ouverte ne partage désormais de chemin de fichier avec elle. Note laissée en place de l'avertissement (retraction non destructive).

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

trivial-diff-advisory Diff trivial : grain META mecanique sans fournee ni exception ecrite (#15740) variation-light-cap-reached Lane ayant deja merge une LIGHT aujourd'hui (cap G-VAR-2 atteint)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants