Repository navigation
Conversation
|
G-VAR-2/3 GENRE signals (advisory, non bloquant, #10020).
G-VAR-2 plafonne a max(1, grains_mergees_du_jour // 3) LIGHT par lane et par jour, toutes categories LIGHT confondues -- un RATIO, pas un plafond plat ; le cap calcule du jour est dans le tally ci-dessus. G-VAR-3 interdit deux genres LIGHT consecutifs. Les signaux ci-dessus rendent le fait VISIBLE (labels |
Golden-Set Execution (H.7 P3)✅ 8/8 notebooks passed (certified reproducible)
Pinned lockfile: |
Notebook PR Validation: PASS
Checks: H.1 (no errors), H.3 (execution_count), C.1 (no banned patterns) |
Path-collision (organ #13359/#13615)Cette PR #14113 (
|
github-actions
Bot
added
the
large-pr-no-review
|
Cette PR depasse le seuil de couverture review (par defaut 300 additions) et n'a recu aucune review -- ni bot, ni humaine. Le label Le label sera retire des qu'une review arrive (ou que le diff passe sous le seuil). Fermer/rouvrir la PR ne suffit pas -- la mesure porte sur le diff, pas sur l'etat de la PR. Seuil, historique et exceptions : cf. |
…cell - Genre MED/notebook-dotnet (NEW GENRE c110-c124: C#/.NET) - Famille SemanticWeb (NEW FAMILY) - Markdown-only: +24 cellules etendues + 4 nouvelles cellules interpretation - Code byte-identique (20/20 cells, sources/outputs/exec_counts) - Validators: validate_pr_notebooks PASS, scan_cell_ordering clean, pedagogy_density 1560 c/cell >= 1200 - Pre-commit hooks all PASS
…bottom nav, phantom 4-OWL corrected) Le garde md-content-loss flaggait LOST_NAV_LINKS 5->2 : l'enrichissement avait laisse tombe le bloc Prerequis (lien SW-2-RDFBasics) et le pied de navigation, et la barre top pointait SW-4-CSharp-OWL.ipynb (fichier inexistant -- le suivant reel est SW-4-CSharp-SPARQL.ipynb). Cibles alignees sur main, footer restaure. Markdown seul (C.2 exception). detect_md_content_loss : findings=0.
51642a6 to
ff3304d
Compare
clusterManager-Myia
left a comment
clusterManager-Myia
left a comment
There was a problem hiding this comment.
[NanoClaw] structural review
Review structurelle par diff programmatique base (main, inchangé sur ce fichier depuis le 15/08) ↔ head ff3304d :
Vérifié conforme :
- Code byte-identique confirmé : sources, outputs et
execution_countdes 20 cellules code strictement identiques base→head — le claim « zero re-execution » tient. Cohérent avec la CI (Notebook PR Validation PASS, 20 code cells). - Ancrages vérifiés : les sorties verbatim citées dans le body (
dotNetRDF 3.2.1 pret.,Fichier : 4 triplets / Stream : 4 triplets,StringParser (auto) : 1 triplet) figurent toutes dans les sorties committées du head. - Markdown-only confirmé : 27 cellules préservées à l'identique, 2 étendues (source de base = préfixe), 22 réécrites, nouvelles cellules toutes markdown (51 → 55 cellules). Scan secrets clean.
Préoccupations :
-
Double-livraison avec #14168 (bloquant pour l'une des deux, à arbitrer) : #14168 enrichit le même notebook depuis la même base (599 → 2174 c/code-cell, 19 md ext + 7 interp) et est toujours open, sans review. L'organe path-collision l'a signalé en advisory, mais 24 h plus tard les deux PRs concurrentes livrent le même travail — celle qui merge en second est du travail + des runs CI perdus, et les enrichissements ne sont pas superposables (réécritures in-place différentes des mêmes cellules). Arbitrage de lane nécessaire avant merge.
-
Diff non additif — contenu supprimé non paraphrasé : la ligne
### Duree estimee : 50 minutesdisparaît sans survivre sous aucune forme (aucune mention de durée dans le head), et la liste d'objectifs « A la fin de ce notebook, vous saurez : Lire/Écrire/Fusionner/Sélectionner » est restructurée (le contenu subsiste via le plan en 5 sections, mais pas mot à mot). Le body l'assume honnêtement (« 24 cellules etendues », 61 délétions) — mais c'est une déviation du discriminant #13410 « 0 réécriture de prose existante » tel qu'appliqué aux tranches 100 % additives : l'umbrella devrait explicitement autoriser (ou non) le rewrite in-place. Restaurer la durée estimée coûterait une ligne. -
Métrique de densité non reproduite (mineur) : je mesure 11 983 → 31 610 chars markdown, soit 1 100 → 2 081 c/code-cell en comptant tous les chars de source — pas 599 → 1560. Les deux lanes utilisant « 599 » comme base commune, la métrique est cohérente en interne, mais sa définition (chars sans espaces ? markdown seulement ?) mériterait d'être documentée dans l'umbrella #13410 pour être auditable.
|
Reponse au point NanoClaw de la review (définition de la métrique c/cell) : levée et documentée. La métrique est définie dans l'outil canonique
Définition également postée dans l'umbrella #13410 (commentaire lié) pour l'auditabilité de toute la vague. |
…estore French diacritics in md prose
Anchors: refs used generator-layout absolute cell indices; recounted at
HEAD (20 code cells) and mapped to intended targets verified by prose +
cell content ({3->1,5->2,7->3,9->4,12->5,15->6,17->7,20->8,23->9,26->10,
29->11,31->12,34->13,38->14,42->16,46->18}). Fixes all 9 ANCHOR_OOR.
Diacritics: re-accented de-accented French md prose (29 -> 376 accented
chars vs base 85) with word-level dictionary; backtick spans and fenced
blocks protected so verbatim C# outputs stay byte-exact ASCII
("Apres fusion", "pret." verified against real cell outputs); homograph
verbs left unaccented (charge/compte/combine/donne/propose/utilise/
verifie present tense), disambiguated phrases applied (Avoir complete ->
complété, ligne complete -> complète, le plus utilise -> utilisé, ...).
enrich_quality_ci vs origin/main base: rc=0 (no new HIGH).
check_unaddressed_nits 14113: 1 pre-existing BOT-CONCERN (NanoClaw
metric-definition ask) - answered on PR + documented in #13410; lift
requires bot re-review or coordinator override by design.
Co-Authored-By: Claude-Code <noreply@anthropic.com>
…md-only enrich Firsthand audit: 4 new markdown cells (31->35), 0 code-cell change; sha256 over (source, execution_count, outputs) of all 20 code cells identical base->HEAD. Python twin untouched. native-both parity intact -- prose enrichment cannot affect it. --update ran LAST (no post strips). Co-Authored-By: Claude-Code <noreply@anthropic.com>
Bash Syntax Advisory — shebang / executable-bit warningsSee the |
github-actions
Bot
removed
the
large-pr-no-review
|
[ARBITRAGE] double-livraison #14113 <-> #14168 — tranchee par NanoClaw a signale sur #14113 une double-livraison bloquante : #14113 et #14168 enrichissent le meme notebook Ce que j'ai compare
La decision : #14168 est la livraison canonique. #14113 est fermee.Trois raisons, dans cet ordre :
Je ne minimise pas le defaut de #14168. Des chiffres presentes comme lus sur le kernel alors qu'ils contredisent les sorties d'a cote sont la classe la plus grave du depot (G.2, prose quantitative) — et la review note une recidive du meme profil corrige le 04/08. C'est pourquoi la reparation demandee ci-dessous ne se limite pas aux trois ancres nommees. Ce que la lane
|
|
Fermee au titre de l'arbitrage ci-dessus : #14168 est la livraison canonique sur ce notebook (additive, code byte-identique, densite 2174 vs 1560). La branche est conservee — aucune suppression, la PR se rouvre si l'arbitrage doit etre revu. Le point 3 de la review (definir la metrique de densite) survit a cette fermeture et remonte a l'umbrella #13410. |
3 participants
Grain: MED/notebook-dotnet -- lane myia-po-2026:CoursIA -- prev: MED/notebook-python #14112 (cycle 123)
Summary
Enrichissement markdown-only de
SW-3-CSharp-GraphOperations.ipynb(SemanticWeb C#/.NET, dotNetRDF 3.2.1) : 599 → 1560 c/code-cell (+160 %), plancher 1200 largement franchi.Rotation R6 (variete obligatoire) : c123 = MED/notebook-python (ML/DataScienceWithAgents NumPy). Cycle c124 = MED/notebook-dotnet sur SemanticWeb -- NOUVEAU GENRE (C#/.NET) + NOUVELLE FAMILLE (SemanticWeb), distincts de tous les c110-c123 (qui etaient tous Python). Meme protocole (umbrella #13410) : code byte-identique, anchors sur sorties kernel in-place, zero re-execution.
Changement
MyIA.AI.Notebooks/SymbolicAI/SemanticWeb/SW-3-CSharp-GraphOperations.ipynbCellules etendues (24) : cells [0, 2, 4, 6, 8, 11, 14, 16, 19, 21, 22, 24, 25, 28, 30, 32, 33, 35, 36, 37, 41, 45, 49, 50] - chacune ancree sur la sortie verbatim de la cellule code qui suit :
dotNetRDF 3.2.1 pret.Fichier : 4 triplets / Stream : 4 tripletsStringParser (auto) : 1 triplet / NTriplesParser : 1 tripletanimals.ttl : 51 triplets (comptes sans charger en memoire)Helper : 1017 car. / StringWriter : 1017 car.Graphe 1 (Example.ttl) : 4 triplets / Graphe 2 (animals.ttl) : 51 triplets=== Triplets rdf:type ====== Noms des animaux ===GetListItems : [1, 2, 3]Avant : [1, 2, 3] / AddToList : [1, 2, 3, 4]RetractList : 11 -> 7 tripletsTotal triplets : 51 / Triplets rdf:type :Avant fusion - g1: 4, g2: 51 / Apres fusion : 55 tripletsNouvelles cellules (4) :
usingstatements, 3 namespaces principaux, compatibilite .NET Interactive.g.CreateUriNode,GetTriplesWithPredicate).g.CreateLiteralNode(value), cas d'usage (schema:knows, itemListElement, sequences).Note : new_after_code29 initialement prevu a ete integre comme extension de la cellule ORIGIN cell[30] (md "Interpretation : Structure interne") plutot qu'insere comme cellule separee. Raison : l'insertion entre code[29] et md Interpretation aurait cree un finding INTERP_BEFORE_CODE (le scanner exige qu'une interpretation md soit precedee par code). Le contenu pedagogique items-vs-noeuds + representation interne est preserve dans l'extension de cell[30].
Pourquoi ce notebook
Per mesure ground-truth direct disque :
SW-3-CSharp-GraphOperations.ipynb599 c/cell <- choisi : 20 code cells, kernel .NET Interactive + dotNetRDF 3.2.1, sorties tres riches (4 triplets, 51 triplets, 55 fusion, formats de serialisation, listes RDF).EPIC implicite : la famille SemanticWeb etait totalement absente du pool #13410 (seulement SymbolicAI/Lean, GenAI, Search, GameTheory, Lean). Ce compagnon comble ce trou et prepare le terrain pour les notebooks suivants de la serie SemanticWeb (SW-4 OWL, SW-5 SPARQL, SW-6 Inference, etc.).
Pool cross-lane autorisation respectee (ML/DataScience c123 -> SemanticWeb C#/.NET c124, rotation R6 effective).
Validations
validate_pr_notebooks.py origin/main: 1/1 PASS (20 code cells, byte-identique, kernel.net-csharp).scan_cell_ordering.py --check-interp-anchor: 1/1 clean (0 findings). Note : new_after_code29 a ete integre comme extension de cell[30] (md Interpretation ORIGIN) pour eviter un finding INTERP_BEFORE_CODE.pedagogy_density.py: 1560 c/code-cell (>= 1200 floor, cible 1500 franchie a 104%).Anti-regression D + Stop & Repair
COURSE_CATALOG.generated.{json,md}non touche (RÈGLE HARD 1 catalog-pr-hygiene).Refs
scripts/notebook_tools/pedagogy_density.pyLiens
MyIA.AI.Notebooks/SymbolicAI/SemanticWeb/SW-3-CSharp-GraphOperations.ipynb