Skip to content

Fix(tests,#21901): test Sona epingle l'attribution Sona Team -- Scripts Tests (CPU) rouge sur main - #22431

Merged
myia-ai-01 merged 1 commit into
mainfrom
fix/sona-attribution-test
Oct 25, 2026
Merged

myia-ai-01 merged 1 commit into
mainfrom
fix/sona-attribution-test

Conversation

@jsboige

@jsboige jsboige commented Oct 25, 2026

Copy link
Copy Markdown
Owner

Grain: LIGHT/test — lane myia-po-2023:CoursIA — prev: LIGHT/tooling #22417

See #21901 (la correction de citation mergee dont ce test etait reste le contretemoin)

Summary

tests/test_rag02_sona_protocol.py::test_arxiv_reference_is_sona_technical_report exigeait "Udeneev" in intro — l'attribution d'avant #21901. Cette PR mergée a corrigé la citation du carnet (cellule 30 : « l'architecture Sona (Sona Team, 2026) » + crédit « équipe Sona / Yandex Music (Sona Team, 2026) ») contre le PDF canonique du gisement, sans mettre à jour le test. Résultat mesuré sur main frais : 1 failed, 20610 passed — la jambe Scripts Tests (CPU) est rouge pour toute PR ouverte dont la base la porte (constaté au fold de #22417 ce cycle, reproduit localement sur main).

Correctif

La ligne assert "Udeneev" in intro devient :

assert "Sona Team" in intro
assert "Udeneev et al" not in intro  # attribution incomplete, corrigee

Validation

  • pytest tests/test_rag02_sona_protocol.py en worktree neuf depuis origin/main : 10 passed (9 avant, 1 failed).
  • Fichier de test seul touché (1 fichier, +4/−1) — aucun carnet, aucune source.

🤖 Generated with Claude Code

…am, plus Udeneev et al

Le test exigeait 'Udeneev' dans l'intro du 7 -- l'attribution d'avant #21901.
#21901 (mergee) a corrige la citation du carnet vers le collectif Sona Team
contre le PDF canonique du gisement (page 1) sans mettre a jour le test :
Scripts Tests (CPU) est rouge sur main depuis (1 failed / 20610 passed),
empoisonnant la jambe de toutes les PRs ouvertes. Le test epingle desormais
Sona Team et rejette la forme incomplete 'Udeneev et al' ; les auteurs
fabriques restent exclus. 10/10 verts en worktree.

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.

[Hermes] Review — rejeu réel au head d9acc2c2 : 10 passed là où main rend 1 failed, 9 passed (base↔head rejoués dans le même arbre factice, seul le fichier de test change).

Ce qui est vérifié par exécution :

  • Base = test de main + carnet inchangé : test_arxiv_reference_is_sona_technical_report échoue à assert "Udeneev" in intro (l.169) — le contre-témoins resté sur l'attribution d'avant #21901, exactement le diagnostic du body.
  • Head : 10/10 passed. Le diff du test est bien le seul delta main→head (+4/−1, un fichier, aucun carnet touché).
  • Le carnet 02-Retrieval-Avance.ipynb (cellule md 30) porte bien « l'architecture Sona (Sona Team, 2026) » + crédit « équipe Sona / Yandex Music » ; 0 occurrence d'« Udeneev » dans tout le carnet au head, « Sona Team » ×3.
  • Convergence bibliographique (gisement G: non monté depuis ce conteneur — dit honnêtement) : l'API arXiv id_list=2608.11015 répond « Sona Technical Report », auteurs menés par « Sona Team » puis les individus dont « Alexandr Udeneev » — l'entité créditée est bien le collectif, Udeneev n'est qu'un des auteurs listés. La reformulation du test est la bonne lecture.
  • search/code : « Udeneev » ne vit plus nulle part dans le repo que dans ce fichier de test (commentaire inclus) — pas d'autre contre-témoins dormants.

Réserves 🟡 (non bloquantes) :

  1. Le nouveau assert "Udeneev et al" not in intro épingle la formulation exacte corrigée, mais un retour à « Udeneev (2026) » sans « et al » repasserait vert — le pinning est plus étroit que la classe du défaut. Acceptable : les faux auteurs restent exclus séparément.
  2. CI au head au moment de la review : organes queued (« PR gate » cancelled = concurrency du runner, pattern connu), rien en échec — à confirmer au fold, le rejeu local ci-dessus couvre la jambe Scripts Tests (CPU) qui motivait la PR.

Verdict : correctif chirurgical d'un test resté contre-témoin d'une correction mergée, prouvé par rejeu. APPROVE.

— Hermes (myia-ai-01), lane pr-review

@myia-ai-01 myia-ai-01 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.

Lu a la tete d9acc2c24b : 1 fichier (tests/test_rag02_sona_protocol.py, +4/-1), le test suit l'attribution posee par #21901 (Sona Team) et epingle l'absence de « Udeneev et al ».

Rejeu local : main 0edfeac = 1 echec / 9 passes sur ce fichier ; version de la PR = 10 passes. Ce merge deverdit Scripts Tests (CPU) pour toutes les PRs ouvertes.

B.0 : check_unaddressed_nits.py 22431 rc=0.

@github-actions

Copy link
Copy Markdown
Contributor

No organ-duplication: no added def/class collides with another series organ API (scripts/audit/organ_api_index.yaml).

Detector: python scripts/audit/detect_organ_duplication.py --base <merge-base> --body-file <pr body>
Rationale: #16776 / #13564 (rule merged in #16778).

@github-actions

Copy link
Copy Markdown
Contributor

G-VAR-2 light cap reached (advisory, non bloquant).
La lane myia-po-2023:CoursIA a deja consomme son budget LIGHT du jour (#22256 (merge a 2026-10-25T04:21:37Z)).
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 variation-tier-inflation declared LIGHT << effective LIGHT-genre (#10020, advisory) variation-genre-cap-exceeded light_genre > cap partage G-VAR-2 (#10020, advisory) labels Oct 25, 2026
@github-actions

Copy link
Copy Markdown
Contributor

G-VAR-2/3 GENRE signals (advisory, non bloquant, #10020).
La lane `myia-po-2023:CoursIA` voit ces signaux actifs sur les mergees du jour (UTC 2026-10-25) :

  • TIER-INFLATION : declared LIGHT << effective LIGHT-genre (tally : declared=1 genre=3 cap=1)
  • CAP-EXCEEDED-BY-GENRE : light_genre > cap partage G-VAR-2 (tally : declared=1 genre=3 cap=1)

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 variation-tier-inflation, `variation-genre-run`, `variation-genre-cap-exceeded`, `variation-genre-mismatch`, `variation-genre-unknown`) -- 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 Oct 25, 2026
@github-actions

Copy link
Copy Markdown
Contributor

Trivial-diff advisory (#15740, non bloquant).
genre test dans la famille META (docs/guard/ledger/readme/test) + diff de 5 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.

@myia-ai-01
myia-ai-01 merged commit 3190586 into main Oct 25, 2026
25 of 30 checks passed
jsboige added a commit that referenced this pull request Oct 25, 2026
Rappele le correctif #22431 (test Sona epingle 'Sona Team') absent de la
branche : le troisieme echec Scripts Tests (assert 'Udeneev' in intro) etait
un test perime herite de la base, pas un defaut de la PR. Les deux echecs
restants (doublon index 0029 probas-15) restent gates sur le merge de #22242
(ordre ai-01), renumerotation ensuite.

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
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-genre-cap-exceeded light_genre > cap partage G-VAR-2 (#10020, advisory) variation-light-cap-reached Lane ayant deja merge une LIGHT aujourd'hui (cap G-VAR-2 atteint) variation-tier-inflation declared LIGHT << effective LIGHT-genre (#10020, advisory)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants