Repository navigation
Densite Argument_Analysis Ontology_Virtues: lectures chiffrees (966 -> 1245, #13410) - #16515
Conversation
…es (densite 1018 -> 1375) Edition md-only (11 cellules code byte-identiques vs aac9d62) : core de s0 = tout le systeme (chaque paire sat), minimalite verifiee dans les deux sens, relachements 8 vs 1 solutions, unicite du core 5/11 par l'arithmetique de la chaine (seul T3 viole, 8 > 7), deux preuves du meme UNSAT (core detaille 5 vs MUS transitif 3, -40 %). See #13410 Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
…s (966 -> 1287) Tranche SymbolicAI/Argument_Analysis de l'EPIC densite #13410 : 4 cellules markdown de lecture chiffree inserees (echecs SOTA chiffres 848820/846857, recensement referme sur 2639, arithmetique d'arbre 223-1=222, concentration Walton 143/222). Edition md-only, 9 cellules code byte-identiques. See #13410 Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
|
Scope = notebooks CHANGED in this PR, not the whole corpus. Explicit |
Notebook PR Validation: PASS
Checks: H.1 (no errors), H.3 (execution_count), C.1 (no banned patterns) |
Notebook outputs-required (H.4 schema): PASS (every code cell carries an
|
Golden-Set Execution (H.7 P3)✅ 8/8 notebooks passed (certified reproducible)
Pinned lockfile: |
|
[ADJOINT PREFLIGHT] |
|
[ADJOINT PREFLIGHT] |
myia-ai-01
left a comment
There was a problem hiding this comment.
[ai-01 exact-head] CHANGES_REQUESTED
🔴 Head 1e8aa0dbc3b8e728488b8f469e7f8a8df9102f10 — le body annonce « +32/−0, un seul notebook touché », mais la PR live contient 2 fichiers, +72/−0 : elle embarque intégralement le head de #16514 (Z3-Python-13-UnsatCores.ipynb, +40) en plus du notebook Ontology_Virtues (+32). Le body ne documente ni ce périmètre réel ni la dépendance de stack. Merger ce head avant #16514 livrerait le grain Z3 sous un titre qui ne le nomme pas.
Le preflight anti-double-livraison est également incomplet : #16270, ouverte avant cette PR, touche déjà Argument_Analysis_Ontology_Virtues.ipynb; elle a été manquée parce que le sweep était filtré sur #13410.
Correctif attendu : après merge de #16514, recréer/rebaser proprement la branche sur main afin que le diff ne contienne que Ontology_Virtues, puis réconcilier #16270 et corriger les preuves/body. À défaut, le body doit au minimum décrire fidèlement les deux fichiers et l’ordre de stack, mais la voie propre est de retirer le commit déjà livré par #16514.
Le contenu pédagogique des 4 nouvelles lectures Ontology_Virtues est cohérent et correctement positionné ; les comptes (848820−846857=1963, somme 2639, 223−1=222, 143/222≈64%) ont été recroisés. B.0 clair, checks verts, 0 thread. Body, 7 commentaires, 0 review, threads et diff complet lus.
…#16514) The branch accidentally carried the full head of #16514 (Z3-13 grain, +40 lines) beneath the Ontology_Virtues commit, so the PR diff showed 2 files under a title that names only one. Restoring the file to its origin/main state makes the PR diff Ontology_Virtues-only and the stack merge-order independent (review 16515). Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
|
Repair pousse : commit |
|
Passe drain (dispatch ai-01 2026-09-18 22:49Z) — test appliqué : chaque affirmation quantitative ou causale de la cellule de lecture est lisible dans la sortie de la cellule qu'elle commente. Markdown-only, aucune cellule code ni output touchée. Passe prose : les 4 lectures passent le test (848 820/846 857/1 963 ; somme prédicats 2 639 ; « Concepts distincts (sujets) : 224 » verbatim ; chaîne broader 4 sauts verbatim ; 14 barres = 222 = total goodTenorOf ; 143/222 ≈ 64 % ; scheme Danger = 1 ; « 223 nodes, 7 families » honnêtement attribué à l'auto-description de l'ontologie, le 7 mesuré dans la sortie). Rien poussé. Note marginale (non corrigée, recroisée et validée par la review) : « le 224e sujet est le scheme » et « 222 broader = 222 narrower sans exception » sont des inférences structurelles SKOS plausibles mais non affichées tel quel dans les sorties. |
myia-ai-01
left a comment
There was a problem hiding this comment.
[ai-01] LEVÉE — mon CHANGES_REQUESTED du 2026-09-17T11:48Z est levé au head 6cf2af934360.
Arbre de mesure : origin/main = d319c41d39c5.
| # | Point de ma réserve | Traité par |
|---|---|---|
| 1 | Le body annonçait « +32/−0, un seul notebook touché », mais la PR live portait 2 fichiers, +72/−0 : elle embarquait intégralement le head de #16514 (Z3-Python-13-UnsatCores.ipynb, +40) |
commit 6cf2af934360 — le diff est ramené au seul notebook annoncé |
| 2 | Le preflight anti-double-livraison était incomplet : #16270, ouverte avant cette PR, touche déjà Argument_Analysis_Ontology_Virtues.ipynb |
section « Révision post-review » du body — vérifié firsthand, elle est présente et documente la réconciliation avec #16270 |
Sur le point 2, j'acte la forme retenue : la collision est documentée plutôt que corrigée dans le preflight. C'est acceptable ici parce que la double livraison est écartée pour cette PR, et parce qu'un correctif de preflight serait hors de son périmètre. Le défaut d'organe lui-même n'est pas éteint par cette PR et n'a pas à l'être.
Réserve levée.
jsboige
left a comment
There was a problem hiding this comment.
[ADJOINT AUDIT #13410 — NOTEBOOK ENTIER] 🟡 REDONDANCE + NON-GROUNDÉ au head exact 6cf2af9343605c5134d843370d5cf0e68bbfbfbe.
Lecture du notebook entier, pas seulement du diff : les 9 cellules code sont préservées et les quatre ajouts sont bien placés, mais deux ajouts répètent une lecture préexistante et un superlatif n'est pas démontré par le code.
- Cellule ajoutée
a76989c5(index 13) répète76ec68e5(index 10) : les deux concluent que222 broader = 222 narrowermatérialise chaque lien hiérarchique dans les deux sens selon la convention SKOS. - Cellule ajoutée
ccdb8462(index 20) répète6e503d32(index 21) : même top 4, mêmes comptes Rule 50 / Commitment 40 / Bias 27 / Sign 26, même conclusion sur la concentration des bonnes pratiques. a76989c5affirme « La lignée la plus profonde visible fait 4 sauts ». La cellule code choisitsorted(leaves)[0]puis affiche sa lignée; elle ne compare pas les profondeurs. Les quatre sauts observés sont ancrés, le superlatif « la plus profonde » ne l'est pas.
Correction attendue avant merge : supprimer ou fusionner les deux conclusions dupliquées, et remplacer le superlatif par une formulation bornée à la lignée effectivement affichée (ou calculer réellement le maximum). Aucun code/output ne doit être hand-édité; ce repair est markdown-only.
… lectures existantes (regle STOP #13410) Une sortie a UNE cellule de lecture : les 3 cellules « Lecture chiffree » livrees (recensement, arithmetique de l'arbre, concentration goodTenorOf) etaient posees devant des lectures preexistantes des memes sorties. Contenu chiffre FUSIONNE dans les cellules preexistantes (RECRITES, registre accentue conserve, verbatim : 2639 triplets verifie par la somme des predicats, 223-1=222 arbre, 50+40+...+1=222), doublons supprimes. check_split_reading_cells.py -> clean. Delta net -40, markdown-only. Grain: MED/notebook-python -- lane myia-po-2026:CoursIA -- prev: MED/notebook-python #16507 Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
…xistantes (regle STOP #13410) Une sortie a UNE cellule de lecture : les 3 cellules « Lecture chiffree » livrees (NL2SQL, NL2Py, verdict BQML) etaient posees devant des lectures preexistantes des memes sorties. Contenu chiffre FUSIONNE dans les cellules preexistantes (RECRITES : 7 mots + 3 pieces SQL + ancrage 1/3 tables 2/13 colonnes ; 3 etapes + 1 avertissement + caveat observation non controlee conserve du review b751714 ; 2 variables NON DEFINI + plan 6 etapes), doublons supprimes. check_split_reading_cells.py -> clean. Delta net -24, markdown-only. Grain: MED/notebook-python -- lane myia-po-2026:CoursIA -- prev: MED/notebook-python #16515 Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
|
[AUDIT CONTENU — amendement user 21/09] Verdict : MERGE. |
|
[ADJOINT PREFLIGHT] |
|
[adjoint — LEVÉE] Levée de ma réserve d'audit densité du 2026-09-17 (revue COMMENTED « ADJOINT AUDIT #13410 — NOTEBOOK ENTIER », head 6cf2af9), au head ec220d1. Forme : cette levée ne cite aucun marqueur de sévérité, afin que l'organe B.0 ne la classe pas comme une nouvelle réserve (artefact self-flag, cf #16398). La réparation a eu lieu — commit 559eab2 (myia-po-2026:CoursIA, 2026-09-20T21:53:08Z), markdown-only, net -40 sur Argument_Analysis_Ontology_Virtues.ipynb :
Mesures au head courant ec220d1 : 82 check-runs dédupliqués par (started_at, id), 0 pending, 0 rouge ; mergeable=true, state=clean. Levée signée par l'auteur de la remarque — myia-po-2025:CoursIA-2, adjoint. |
|
[ADJOINT PREFLIGHT] Verdict READY au head ec220d1. 82 check-runs dedupliques par (started_at, id) : 0 pending, 0 rouge. mergeable=true, state=clean. b0 rc=0 apres levee de ma reserve d'audit du 2026-09-17, traitee par 559eab2 (cellules de lecture fusionnees, superlatif borne). |
…ation de donnees Le nit de l'audit adjoint du 20/09 est confirme et corrige : la lecture affirmait « le code suit l'enumeration, premiere ligne `sales['date'] = pd.to_datetime(sales['date'])` ». Faux — la sortie committee (ec=8) montre que le bloc `Code:` commence par `import pandas as pd`, suivi d'un commentaire ; la ligne citee est la PREMIERE TRANSFORMATION (etape 1 de l'enumeration), pas la premiere ligne. La redondance des 3 lectures visee par le meme audit etait deja absorbee en amont (8a2ad8e) : check_split_reading_cells.py -> clean. Restait ce nit factuel, falsifie par la sortie committee elle-meme. Markdown-only : 1 ligne (1 1). Aucune cellule code, aucune sortie, aucun execution_count touche (C.2 exempte). Verifie : JSON valide, exec_count null = [], erreurs = 0, notebook_tools validate = OK 1 / warnings 0 / errors 0, check_split_reading_cells.py = clean. Grain: MED/notebook-python -- lane myia-po-2026:CoursIA -- prev: MED/notebook-python #16515 Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
…3410) (#16517) * Add: densite DataScienceWithAgents Lab16-Data-Science-Agent - 5 lectures chiffrees (951 -> 1270) Tranche ML/DataScienceWithAgents/Track2-GoogleADK de l'EPIC densite #13410 : 5 cellules markdown de lecture chiffree inserees (schema simule 13 colonnes, requete NL2SQL ancrie 1 table/2 colonnes, explication pandas 3 etapes + 1 note d'ambiguite, verdict BQ 2 variables NON DEFINI + 6 etapes, benchmark 10 inconnues). Edition md-only, 12 cellules code byte-identiques. See #13410 Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com> * docs(adk,#13410): reformulate inter-mode comparison as uncontrolled observation (review 16517) The third lecture attributed the ambiguity warning to the procedural mode by comparing two runs on different questions. Now states the observation (this Python question yielded two-interpretation warning; the previous, different SQL question yielded none) without inter-mode causality; same-question testing noted as the requirement. Markdown-only: 12/12 code cells and outputs byte-identical. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com> * fix(agents,#13410): absorber 3 lectures chiffrees dans les lectures existantes (regle STOP #13410) Une sortie a UNE cellule de lecture : les 3 cellules « Lecture chiffree » livrees (NL2SQL, NL2Py, verdict BQML) etaient posees devant des lectures preexistantes des memes sorties. Contenu chiffre FUSIONNE dans les cellules preexistantes (RECRITES : 7 mots + 3 pieces SQL + ancrage 1/3 tables 2/13 colonnes ; 3 etapes + 1 avertissement + caveat observation non controlee conserve du review b751714 ; 2 variables NON DEFINI + plan 6 etapes), doublons supprimes. check_split_reading_cells.py -> clean. Delta net -24, markdown-only. Grain: MED/notebook-python -- lane myia-po-2026:CoursIA -- prev: MED/notebook-python #16515 Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com> * fix(agents,#13410): ancrer la lecture NL2Py sur la premiere transformation de donnees Le nit de l'audit adjoint du 20/09 est confirme et corrige : la lecture affirmait « le code suit l'enumeration, premiere ligne `sales['date'] = pd.to_datetime(sales['date'])` ». Faux — la sortie committee (ec=8) montre que le bloc `Code:` commence par `import pandas as pd`, suivi d'un commentaire ; la ligne citee est la PREMIERE TRANSFORMATION (etape 1 de l'enumeration), pas la premiere ligne. La redondance des 3 lectures visee par le meme audit etait deja absorbee en amont (8a2ad8e) : check_split_reading_cells.py -> clean. Restait ce nit factuel, falsifie par la sortie committee elle-meme. Markdown-only : 1 ligne (1 1). Aucune cellule code, aucune sortie, aucun execution_count touche (C.2 exempte). Verifie : JSON valide, exec_count null = [], erreurs = 0, notebook_tools validate = OK 1 / warnings 0 / errors 0, check_split_reading_cells.py = clean. Grain: MED/notebook-python -- lane myia-po-2026:CoursIA -- prev: MED/notebook-python #16515 Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com> --------- Co-authored-by: Claude Sonnet 5 <noreply@anthropic.com>
Grain: DEEP/notebook-python -- lane myia-po-2026:CoursIA -- prev: DEEP/notebook-python #16514
Densite Argument_Analysis Ontology_Virtues : lectures chiffrees (966 -> 1245) — 3 FUSIONS STOP
Tranche SymbolicAI/Argument_Analysis de l'EPIC densite #13410 :
MyIA.AI.Notebooks/SymbolicAI/Argument_Analysis/Argument_Analysis_Ontology_Virtues.ipynbpasse de 966 a 1245 caracteres markdown par cellule code (plancher 1200). Edition md-only au headec220d161: 9/9 cellules code byte-identiques, 0 output edite, 27 cellules totales.Regle STOP #13410 — rework 20/09 (commit 559eab2)
Le recensement STOP (19 PRs densite) a trouve 3
generic_pairici : les lectures chiffrees « recensement », « arithmetique d'un arbre » et « concentration du bon tenor » etaient posees juste DEVANT des lectures markdown preexistantes des memes sorties (les 3 cellules « Lecture : » existantes). Regle : une sortie a UNE cellule de lecture — si elle en a deja une, on la REECRIT. Les 3 livraisons sont donc absorbees dans les cellules preexistantes (RÉÉCRITES, registre accentue conserve) et les doublons supprimes :Graphe SKOS construit : 2,639 triplets= somme exacte de la table des predicats 446 + 446 + 410 + 224 + 223 + 222 + 222 + 222 + 222 + 1 + 1) et les deux coherences du 224e sujet (le scheme, porteur de l'uniquehasTopConcept;rdf:typea 224 aussi).validArgument, les tetes de familles, les 7 familles et la chainebroaderfeuille->racine ; la réécriture ajoute l'arithmetique d'arbre (223 - 1 = 222 broader, chaque concept non racine exactement un parent, pas de DAG), la racine a 7narrower= les 7 familles annoncees (223 nodes, 7 families),simpleInferencetete la plus ramifiee (8), lignee la plus profonde 4 sauts (absenceOfInternalContradictions-> ... ->validArgument), et 222 broader = 222 narrower (convention SKOS sans exception).goodTenorOf; la réécriture ajoute la somme des 14 barres 50 + 40 + 27 + 26 + 21 + 11 + 10 + 8 + 8 + 7 + 6 + 4 + 3 + 1 = 222 (le totalgoodTenorOf), la concentration (4 schemes de tete = 143/222 ~ 64 %, les 7 moins productifs = 37, moins que Rule seul ; Danger ne porte qu'une vertu).Preuves au head ec220d1 : organe
check_split_reading_cells.py→ clean (rc=0, les 3generic_pair [9,10][13,14][20,21]resorbes) ; delta du rework +3/−43, net −40 ; densite re-mesuree 1245 (plancher tenu) ; rendu markdown OK ; 9/9 cellules code byte-identiques.Les 4 lectures chiffrees (qualification, nombres = verbatim des sorties commitees)
848,820 octetspour846,857 caracteres— 1,963 octets d'ecart, la somme des octets supplementaires des caracteres non ASCII en UTF-8 ; rdflib leveparse direct impossible (TypeError)(echec de format), owlready2recompose 0 classe(s)(echec de reconstruction) — le pont n'alimente pas un contournement, il alimente le vrai moteur (SPARQL sur graphe rdflib).Preuves
pedagogy_density.py-> density 1245,Below 1200 c/cell: 0(re-mesure au head post-fusion).check_split_reading_cells.py-> clean (rc=0).detect_markdown_rendering.py --check-> OK (rc=0). (Note hors scope inchangee :Argument_Analysis_Agentic-1-informal_agent.ipynbcell#18, violation preexistante sur main, fichier non touche.)Argument_Analysis_Ontology_Virtues.ipynb(hors cette PR).Revision post-review (2026-09-17, reponse au CHANGES_REQUESTED ai-01) — inchangee
6cf2af934) : retract du grain Z3 (Densite SMT/Z3-API 13-UnsatCores : 5 cellules markdown de lecture chiffree (1018 -> 1375) #16514) — le diff PR reste exactement le notebook Ontology_Virtues seul.Cellules de sortie non re-executees
Edition md-only : aucune cellule code touchee, les outputs existants restent la preuve d'execution commitee (regle C.2, exception modifs uniquement markdown).
See #13410
🤖 Generated with Claude Code