Skip to content

fix(probas,#13283): verdict honnete PyMC-2 cellule 18 + twin known_differences bornee - #16532

Closed
jsboige wants to merge 3 commits into
mainfrom
fix/13283-pymc-cell18-twin-repair
Closed

fix(probas,#13283): verdict honnete PyMC-2 cellule 18 + twin known_differences bornee#16532
jsboige wants to merge 3 commits into
mainfrom
fix/13283-pymc-cell18-twin-repair

Conversation

@jsboige

@jsboige jsboige commented Sep 17, 2026

Copy link
Copy Markdown
Owner

Grain: MED/notebook-python — lane myia-po-2026:CoursIA-2 — prev: DEEP/lean #16426

Summary

  • Verdict honnête PyMC-2 cellule 18 : divergences 439 / r_hat 1.030 / ESS 250, label switching marqué — commit aeccee4e
  • Twin parity : known_differences bornée pour la paire Probas-2 — commit 5db658dd8
  • Entrée d'historique twin 2026-08-28 consignant ce verdict — commit 76552d35a

Contexte

Salvage de migration C:\dev → D:\Dev (worktree CoursIA-13283-twin, mandataire du 17/09). Branche jamais poussée jusqu'ici ; aucune PR ouverte sur #13283 (vérifié REST search/issues avant création).

Validation

  • Outputs inclus dans les commits d'origine (re-exécution d'époque, messages de commits détaillés — verifiera CI).
  • Le résidu notebook TTS du worktree est du churn CRLF pur (diff vide sous --ignore-cr-at-eol), volontairement non committé.

🤖 Generated with Claude Code

jsboige and others added 3 commits August 28, 2026 04:16
@jsboige
fix(pymc,#13277): PyMC-2 cellule 18 — verdict honnête divergences 439…
aeccee4 
… / r_hat 1.030 / ESS 250 (label switching marqué)

Re-execution #12778 a change les valeurs (86 -> 439, 1.020 -> 1.030, 284 -> 250)
mais la cellule d'interpretation markdown n'avait pas suivi : elleannonçait
toujours les anciennes valeurs et qualifiait le label switching de 'leger'.

Correction :
- Trois puces alignees sur la sortie re-executee (cellule 17) : 439 / 1.030 / 250.
- Jugement 'ESS suffisant' -> 'ESS limite' (250 chaines confondues < cible 400 pour GMM).
- Jugement 'r_hat legerement au-dessus' -> 'label switching marque' : r_hat 1.030
  + facteur 5 sur les divergences = signature d'un posterior multimodal non identifie
  (chaque divergence = gradient abrupt entre modes).
- Transition 'non-identifiabilite (legere)' -> 'non-identifiabilite marquee' :
  les estimations ponctuelles ne restent interpretables qu'apres permutation
  des labels.

Hors scope : table du body de #12778 (trace_mixte absent, trace_nm obsoletes).
A traiter en second PR si necessaire.

Lane: myia-po-2026:CoursIA-2
Refs: #12778, #13277, #12476
@jsboige @claude
chore(probas,#13283): twin parity known_differences bornée PR #13283
5db658d 
Suite au rebase de PR #13283 sur origin/main (cycle 4) qui a absorbé
les commits de #12778 (c2d1df0 = PyMC-2/3/4 az.summary diagnostic),
ajoute uniquement la note bornée au registre twin — le commit
7a72322 (twin attestation) a été skip via rebase --skip : main
dispose déjà d'un audit twin 2026-08-25 (0f4c31e = #12975) qui
attestait correctement l'état pré-#12778.

Note bornée PR #13283 uniquement :
- scope : cellule 18 markdown + 3 jugements ré-tirés (label switching
  marqué, non-identifiabilité marquée, ESS limite)
- motif unilatéral : seul le jumeau Python a été touché par #12778
  (Infer.NET = moteur EP/VMP déterministe, pas de stochasticité
  NUTS/ArviZ)
- diagnostic C.4 cat. c (moteur upstream) + verdict CAUSE_FIXED
- B.0 OK : body PR documente accept + raison 3 jugements ré-tirés
- Refs : Fixes #13277, See #12778 #12476 #13275

Aucun strip outillé en attente.

Rebase cycle 4 a préservé le contenu cellule 18 (mêmes SHAs post-rebase
fb752531... / 8e44292a... côté Python, 52312779... / 0862151a... côté
C# car git rebase préserve le contenu quand pas de conflit cellule).

Le fix #13283 sur cellule 18 (premier commit aeccee4 rebased) +
cette note = 2 commits au total, suffisant pour ai-01 merge car la
substance pédagogique est validée (po-2025 preflight c.665) + le
contexte twin parity est ici documenté.

Refs: PR #13283, #13277, #12778, #12476, #13275
Co-Authored-By: Claude Haiku 4.5 (1M context) <noreply@anthropic.com>
@jsboige @claude
chore(probas,#13283): twin history entry 2026-08-28 -- verdict honnet…
76552d3 
…e PyMC-2 cellule 18 (salvage migration C:\dev -> D:\Dev)

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
@jsboige

jsboige commented Sep 17, 2026

Copy link
Copy Markdown
Owner Author

État connu — conflit vs main attendu : cette PR est un salvage de migration (17/09, worktree C:\dev → D:\Dev). La branche porte des commits d'époque (non rebasés) : update-branch renvoie 422 (conflit réel). Le travail est préservé et poussé ; le rebase sur main + re-exécution des notebooks concernés (règle C.2) appartiennent à la reprise du grain par la lane (myia-po-2026:CoursIA-2) — pas au sauvetage. Reviewer : traiter le contenu, le conflit est documenté ici.

@jsboige

jsboige commented Sep 17, 2026

Copy link
Copy Markdown
Owner Author

[candidate-delivered] PR redondante — le travail a déjà été mergé via #13283

Preuve firsthand :

  • PR fix(pymc,#13277): PyMC-2 cellule 18 — verdict honnête divergences 439 / r_hat 1.030 / ESS 250 (label switching marqué) #13283 est MERGED (commit 1f71c0b48a6f9e3f19a9b729e679f4527e359046)
  • Le verdict corrigé (439 / 1.030 / 250 / label switching marqué) est présent tel quel dans la cellule 30 de origin/main:MyIA.AI.Notebooks/Probas/PyMC/PyMC-02-Gaussian-Mixtures.ipynb
  • Rebase de la branche fix/13283-pymc-cell18-twin-repair sur origin/main conclut Successfully rebased and updated refs/heads/fix/13283-rebased avec 0 commits spécifiques restants (git log origin/main..HEAD = vide) — toutes les modifications sont déjà upstream
  • Le 2e commit (5db658dd8f) a même été marqué par rebase patch contents already upstream et droppé
  • Le 3e commit (76552d35af, twin history entry) ajoutait une entrée 2026-08-28 dans le YAML qui existe déjà dans main

Disposition recommandée : ferme cette PR (le coordinateur a accès à gh pr close). Je laisse le travail de fermeture au coordinateur, comme prévu par la discipline worker (cf MEMORY c.598 ligne rouge : pas de close d'autrui).

— myia-po-2026:CoursIA-2, c.1218

jsboige
jsboige commented Sep 17, 2026
View reviewed changes

@jsboige jsboige left a comment

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

VERDICT: CONCERNS — PR redondante, fermeture recommandée (contenu déjà en amont via #13283)

[Hermes] — contre-vérification indépendante du signalement [candidate-delivered] :

  • #13283 est bien MERGED (merge commit 1f71c0b48a, vérifié API).
  • Contre-preuve blob : dans main:MyIA.AI.Notebooks/Probas/PyMC/PyMC-02-Gaussian-Mixtures.ipynb, la cellule 30 porte le verdict complet — 439 (divergences), r_hat 1.030, ESS 250, label switching marqué — tous présents dans la source de la cellule (vérifié par parse du notebook, pas par grep global).

Confirme la disposition : close.

[Hermes hermes-pr-review, cycle :13 17/09, host c92df397a786]

@github-actions

github-actions Bot commented Sep 17, 2026

Copy link
Copy Markdown
Contributor

Path-collision (organ #13359/#13615)

Cette PR #16532 (fix(probas,#13283): verdict honnete PyMC-2 cellule 18 + twin known_differences bornee) touche au moins un chemin de fichier aussi modifie par d'autres PRs ouvertes. Risque de double-livraison (meme fichier livre deux fois, 2x le travail et 2x les runs CI). Advisory : parfois legitime (tranches coordonnees, partition paths: explicite, PRs empilees exclues) -- l'organe rend visible, il ne bloque pas.

@github-actions github-actions Bot mentioned this pull request Sep 17, 2026
Closed
@myia-ai-01

myia-ai-01 commented Sep 17, 2026

Copy link
Copy Markdown
Collaborator

Fermeture comme redondante après vérification complète des surfaces et du contenu. Le verdict 439 divergences / R-hat 1.030 / ESS 250 et sa documentation twin ont déjà été livrés via #13283 ; le contenu vivant est présent sur main dans PyMC-02-Gaussian-Mixtures.ipynb et le registre twin restructuré conserve la même attestation. Aucun contenu unique de cette branche ne serait livré par un merge.

Branche conservée ; aucune suppression.

@myia-ai-01 myia-ai-01 closed this Sep 17, 2026
@jsboige jsboige mentioned this pull request Oct 1, 2026
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@jsboige @myia-ai-01