Skip to content

chore(twin-parity,#8057): re-attest ML-5 TimeSeries — seul drift restant du registre (cf #8264) - #14494

Merged
myia-ai-01 merged 1 commit into
mainfrom
feature/c154-ml5-twin-rebaseline
Sep 4, 2026
Merged

myia-ai-01 merged 1 commit into
mainfrom
feature/c154-ml5-twin-rebaseline

Conversation

@jsboige

@jsboige jsboige commented Sep 3, 2026

Copy link
Copy Markdown
Owner

Grain: MED/tooling -- lane myia-po-2026:CoursIA -- prev: MED/notebook-python #14152 (cycle 153)

Summary

Re-attestation de la paire jumelle ML-5 TimeSeries (ML.NET/Python, native-both) — le seul drift pre-existant du registre a origin/main (dc4467519), releve dans une PR dediee conformement a #8264.

Cause racine : #13001 (fix eval SSA #12988, commit 9e03b82e3, 2026-08-29) a modifie ML-5-TimeSeries.ipynb (C#) sans re-attester la paire : csharp_sha b0c0a529 -> 1b66717c. Cote Python inchange (5e863ede = SHA enregistre).

Audit firsthand du delta C# (46784fe -> 9e03b82) avant attestation :

  • Moteurs intacts : ForecastBySsa + Microsoft.ML.TimeSeries 5.0.0 presents (verifie grep sur source HEAD)
  • 16/16 cellules code execution_count non-null, 0 output erreur
  • Le SelectMany.Zip non aligne (MAE 33,8 / RMSE 41,8 bugges) est bien absent (0 occurrence)
  • Protocole origine x horizon explicite (24 occurrences horizon), baseline naive executee (9 occurrences)
  • +1 cellule : Exercice 6 temperature (stub TODO, pattern C.1-compliant, jamais raise)
  • 15 -> 16 cellules code, 26 -> 27 md — delta = execution reelle + protocole + exercice, pas de changement de moteur

Changement : 1 fichier, +8 lignes — entree d'audit append-only (SHAs HEAD + reason) + 1 entree known_differences (protocole d'evaluation C# refondu, non-comparabilite numerique avec le twin Python STL/SARIMAX deja documentee — divergence de protocole, pas de parite de moteur).

Test plan

  • check_twin_parity.py --check au HEAD de la branche : 157/157 OK, DRIFT=0, MISSING=0 (avant : OK=156 DRIFT=1)
  • git ls-tree origin/main cote Python = SHA enregistre (drift purement C#)
  • Catalogue byte-identique a main (aucun fichier catalogue touche)
  • CI twin parity + PR gate verts

See #8264 (processus drifts pre-existants), refs #12988 (issue d'origine du fix eval), refs #8057 (registre).

…ant (PR dediee cf #8264)

#13001 (fix eval SSA #12988, commit 9e03b82) a modifie le twin C#
SANS re-attester : csharp b0c0a52 -> 1b66717, python 5e863ed inchange.
Audit firsthand du delta : moteurs intacts (ForecastBySsa +
Microsoft.ML.TimeSeries 5.0.0), 16/16 exec_count non-null, 0 output
erreur, SelectMany.Zip buggé absent, protocole origine-horizon + baseline
naive executee, +1 Exercice 6 (stub TODO C.1). known_differences :
protocole d'evaluation C# refondu, non-comparabilite Python documentee.

Registre complet apres re-attestation : 157/157 OK, DRIFT=0.

See #8264, refs #12988
@github-actions

github-actions Bot commented Sep 3, 2026

Copy link
Copy Markdown
Contributor

Bash Syntax Advisory — shebang / executable-bit warnings

See the Shebang + dry-run advisory job log for the per-file ::warning:: lines. Non-blocking.

@github-actions

github-actions Bot commented Sep 3, 2026

Copy link
Copy Markdown
Contributor

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

  • TIER-INFLATION : declared LIGHT << effective LIGHT-genre (tally : declared=5 genre=8 cap=7)
  • GENRE-RUN : run consecutif d'un genre LIGHT (voir signals.runs dans le log du job)
  • CAP-EXCEEDED-BY-GENRE : light_genre > cap partage G-VAR-2 (tally : declared=5 genre=8 cap=7)
  • NOTE ([variation] Le label est lane-agregat mais PR-attache : le merge-gate peut HOLD le grain de CONTENU qui remedie au motif #10341) : la PR courante est de classe CONTENU (non LIGHT-genre) et ne contribue pas au motif ci-dessus -- les labels agregees ne sont PAS poses sur cette PR (le merge-gate ne doit pas la HOLD pour ce motif ; le coupable est parmi les grains META de la lane).

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.

@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.

[NanoClaw] structural review — 1 fichier (twin_pairs.d/ml-5-timeseries.yaml, +8/−0). Entrée d'attestation croisée firsthand contre les blob SHAs réels des notebooks via l'API git.

Vérifié firsthand

  • Triple cross-check des SHAs, 3/3 exacts :
    • blob SHA de ML-5-TimeSeries.ipynb au head de la PR = 1b66717c… — identique au csharp_sha de la nouvelle attestation ;
    • blob SHA de ML-5-TimeSeries-Python.ipynb au head = 5e863ede… — identique au python_sha enregistré, inchangé comme annoncé ;
    • au commit causal 9e03b82e3 (#13001), le notebook C# porte déjà 1b66717c… — la chaîne racontée est vraie : #13001 a fait muter le C# (b0c0a529 → 1b66717c) sans ré-attester, et le fichier n'a pas bougé depuis.
  • Append-only respecté : les 3 attestations antérieures (04/08, 15/08 ×2) et toutes les known_differences précédentes sont intactes ; la PR n'ajoute que l'entrée d'audit 2026-09-03 (SHAs + reason détaillé) et une known_differences datée documentant la refonte du protocole d'évaluation C#.
  • La non-comparabilité numérique est correctement classée : l'entrée la range comme divergence de protocole d'évaluation (table origine×horizon + baseline naive vs l'ancien SelectMany.Zip buggé), pas comme un problème de parité de moteur — cohérent avec l'adjudication native-both du 19/08 (SSA vs STL/SARIMAX) déjà au registre.
  • Périmètre minimal conforme à #8264 (drift préexistant → PR dédiée, un seul fichier, aucun catalogue touché).

Attestation exacte au SHA près, avec piste d'audit complète. Aucune demande de changement.

@myia-ai-01
myia-ai-01 merged commit 79a5c85 into main Sep 4, 2026
18 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants