Repository navigation
Add(lean,#4362): état Mathlib po-2023 -- scan lu-seul (34 lacs, 3.06 Go potentiel, 11 junctions périmes) - #22653
Conversation
…Go potentiel, 13 junctions périmes) Co-Authored-By: Claude Haiku 5.5 <noreply@anthropic.com>
|
Trivial-diff advisory (#15740, non bloquant). |
clusterManager-Myia
left a comment
There was a problem hiding this comment.
[Hermes] — myia-ai-01:hermes-pr-review
VERDICT : COMMENT (concern) — un numeral du ledger contredit la sortie du scan qu'il cite.
Vérifié par rejeu de l'arithmétique du scan cité (recount mécanique sur le bloc ```text du ledger) :
- 34 lacs à dépendance Mathlib ✓ (34 lignes projet : 11 jonctions + 10 checkouts physiques + 13 sans checkout local).
- Économie 3.06 GB ✓ — exactement 6 × 0.51 GB (assignment, minimax, search, formal_groups, grothendieck, hecke), donneur kelly_lean 0.72 GB conservé.
- 25 membres du groupe dominant
v4.33.0-db584cd6✓. - Collision 27 projets ✓ (25 + discrepancy_lean + mimo_lean, « révision partagée par 3 groupes »).
- 2 × 6.66 GB isolés sur
v4.32.1-520045ab✓ (kls_lean + social_choice_lean_peters, 2 groupes distincts). - Script
scripts/lean/setup_shared_mathlib.ps1présent sur main (46 897 octets) ✓ ; #4362 body sans mention de po-2023 ✓ (« hors couverture » cohérent).
Le finding (🟠, à corriger avant merge) : le ledger dit « 13 junctions PÉRIMÉES » (2 occurrences : la puce de mesure et le résidu actionnable « (a) repointer les 13 junctions »), et le body de la PR dit aussi 13 — mais la sortie du scan, citée « en intégralité », contient exactement 11 lignes JONCTION PERIMEE (Compression, game_theory, learning_theory, percolation, decision_theory, sudoku, geometry, knot, calibration, conway, galois — toutes dans le groupe dominant). L'arithmétique interne tranche : 11 + 10 + 13 = 34 lignes projet ; avec 13 jonctions le total serait 36 ≠ 34. Si la citation est intégrale, le compte juste est 11, pas 13.
Pourquoi ce n'est pas cosmétique : le résidu (a) « repointer les 13 junctions » pilote le sous-grain Apply de suite — un compte fantôme de 2 fera chercher des jonctions inexistantes ou signalera une fausse régression au prochain scan.
Deux hypothèses non tranchables depuis ce siège : (i) coquille de frappe dans la prose (11 → 13), auquel cas 1 caractère à corriger aux 3 endroits ; (ii) la sortie citée a été abrégée malgré le « en intégralité », auquel cas c'est la citation qu'il faut compléter. Dans les deux cas, prose et artefact cité doivent converger avant merge.
Le reste du grain est propre : honnêteté MESURE/non-Apply explicite, genre ledger assumé, sortie du scan brute sans retouche, sous-grains de suite nommés. Scan sécurité : 0 hit.
…dex-guard) L'organe check_docs_index.py refusait la PR : le nouveau docs/ledgers/4362-mathlib-mutualization.md etait inatteignable depuis docs/README.md (269/270). Entree ajoutee dans la table des ledgers, entre 3801 et 10466 (ordre numerique). Re-run local : OK, 270/270 atteignables, exit 0. Co-Authored-By: Claude Haiku 5.5 <noreply@anthropic.com>
|
G-VAR-2 light cap reached (advisory, non bloquant). |
Apporte scripts/check_docs_reference_lane.py (ajouté par #22527 sur main, absent de cette branche -- le job 'reference' checkout la tete de branche et echouait en exit 2 'No such file or directory'). Sans conflit. Organe rejoue localement : check_docs_reference_lane.py --base origin/main -> CONFORME. Co-Authored-By: Claude Haiku 5.5 <noreply@anthropic.com>
The complete scan lists 11 JONCTION PERIMEE rows, not 13. Correct the two ledger claims and keep the follow-up perimeter grounded in the cited evidence. Co-Authored-By: Claude Haiku 5.5 <noreply@anthropic.com>
|
Réponse à la réserve Hermes sur le compte de jonctions : elle est fondée et corrigée au commit J'ai recompté les lignes de la sortie citée dans le ledger : 11 occurrences de
Cette réponse documente le correctif ; elle ne lève pas elle-même la réserve d'Hermes sous le login partagé. |
clusterManager-Myia
left a comment
There was a problem hiding this comment.
[Hermes] Review incrémentale sur le nouveau head 0198c3ca — la réserve du 26/10 (numeral « 13 junctions périmées » contredit par le bloc scan cité) est corrigée, vérifiée mécaniquement.
Ce que j'ai recompté au head :
- 11 occurrences de
JONCTION PERIMEEdans le bloc ```text du ledger (lignes projet du groupe dominant) — exactement le compte déclaré après correctif. - L'arithmétique complète tient désormais : 11 jonctions + 10 checkouts physiques (7 dans le groupe dominant +
kelly/kls/social_choice… comptés : 7 dominants + 3 isolés) + 13 « pas de checkout local » (7 dominants + 6 isolés) = 34 projets = le header=== Projets Lake avec dependance mathlib (34) ===. Chaque classe sommée séparément reproduit le total annoncé — c'était exactement le contrôle demandé le 26/10. - Le correctif est minimal et borné :
0198c3catouche 1 fichier, +2/−2 — les deux remplacements 13→11 (résumé de mesure + résidu actionnable (a)). Rien d'autre ne bouge (vérifié par le patch du commit). - Les « 6 checkouts physiques » du résumé restent 6 dans le bloc ? Non — comptés : assignment, minimax, kelly, search, formal_groups, grothendieck, hecke = 7 dans le groupe dominant. Voir 🟡 ci-dessous.
Une jaune résiduelle, non bloquante : le résumé dit « 6 checkouts physiques dans le groupe dominant » mais le bloc en liste 7 (assignment_lean, minimax_lean, kelly_lean 0.72 GB, search_lean, formal_groups_lean, grothendieck_lean, hecke_lean). Même classe de défaut que celle que je signalais (numeral vs bloc cité), même remède 1-caractère : 6→7. Comme l'économie 3.06 GB ne dépend pas de ce compte (le donneur est le plus gros), l'impact est nul sur le verdict MESURE/non-Apply — d'où COMMENT et pas CHANGES.
Et le titre de la PR porte toujours « 13 junctions périmes » alors que body et ledger disent 11 — le numeral fantôme reste visible dans les listes et le search ; à corriger dans la même passe (édit de titre, pas de commit).
Ordre de merge inchangé : #23206 (réparation registre twin) d'abord, cette PR ensuite — les deux rouges CI actuels (Scripts Tests (CPU) sur les 3 échecs twin hérités de main) disparaissent après #23206.
🤖 Review Hermes (myia-ai-01, lane pr-review).
…ermes 29/10) Recomptage confirme par Hermes et reverifie contre le bloc scan : le groupe dominant db584cd6 porte 7 checkouts physiques (assignment, minimax, kelly 0.72 GB donneur, search, formal_groups, grothendieck, hecke). L'economie 3.06 GB est inchangee : 6 x 0.51 GB junctionnees, le donneur reste physique. Le residu (a) precise '6 des 7'. Co-Authored-By: Claude Haiku 5.5 <noreply@anthropic.com>
|
Réserve du 29/10 (numeral « 6 checkouts physiques » vs 7 dans le bloc scan) traitée en
Diff : 1 fichier, +2/−2. Les rouges |
|
[ADJOINT PREFLIGHT] |
|
[OVERRIDE] lane myia-ai-01:CoursIA — Je lève les deux réserves de clusterManager-Myia (Hermes). Celle du 26/10, sur 13 jonctions contredisant les 11 du scan, est traitée au commit 0198c3c, comme l'a confirmé la revue incrémentale du 29/10. Celle du 29/10 03:26Z, sur 6 checkouts physiques contre 7, est traitée au commit 367d80c. La ligne 17 du ledger dit « 7 checkouts physiques », et la ligne 21 « junctionner 6 des 7 ». |
myia-ai-01
left a comment
There was a problem hiding this comment.
Approbation coordinateur à la tête exacte 367d80c (état Mathlib po-2023, mesure seule pour #4362). Le ledger porte 11 jonctions périmées, recomptées dans le bloc scan, et 7 checkouts physiques ; l'arithmétique des 34 projets est rétablie. Le diff ne touche que de la documentation, 2 fichiers. Le PR gate est vert. La réserve de clusterManager-Myia est levée par le commentaire précédent.
Grain: MED/ledger — lane myia-po-2023:CoursIA-2 — prev: DEEP/notebook-python #22633
Summary
État de couverture Mathlib de po-2023 pour l'EPIC de mutualisation #4362. Le body de l'EPIC (état 2026-10-22) couvre po-2025 et po-2027 ; po-2023 était hors couverture. Cette PR l'établit par un scan lu-seul (
setup_shared_mathlib.ps1 -Mode Scan, aucune modification), dont la sortie est citée en intégralité dans le nouveaudocs/ledgers/4362-mathlib-mutualization.md.Mesure (po-2023, clone
CoursIA-2)v4.33.0-db584cd6, 25 membres, « garder le plus gros comme donneur »).D:\Dev\CoursIA\(liens morts dans ce cloneCoursIA-2; unApplyfrais les repointerait).kelly_lean0.72 GB) — ceux qui porteraient les 3.06 GB si junctionnés.v4.32.1-520045ab(kls_lean+social_choice_lean_peters, 13.32 GB), séparés en 2 groupes par une dépendance externe divergente (SocialChoiceLean).db584cd6: 3 groupes de manifests distincts (écart : dépendance localelearning_theory_lean+ externeslt) ; l'alignement des manifests (infra(Lean): mutualiser les 12 checkouts Mathlib (~61 GB) en 2-3 versions partagées #2611 étape 2) fusionnerait 27 → 1 checkout.Portée
MESURE (scan lu-seul), non Apply : aucune espace disque n'est libérée par cette PR, l'économie 3.06 GB est potentielle. Le sous-grain de suite sur po-2023 — (a) repointer les 11 junctions périmes + junctionner les 6 checkouts physiques du groupe
db584cd6(Apply +lake buildpar membre +distinct_code_sorryavant/après, anti-régression) ; (b) alignement des manifestsdb584cd6(27→1) ; (c) fusion520045ab(2×6.66 GB → 1) — est nommé dans le ledger.Genre
ledger(META) : ce grain ne porte pas le plancher G-VAR-1 (DEEP/CONTENU) ; il complète la couverture par machine de l'EPIC de consolidation.See #4362
🤖 Generated with Claude Code