Repository navigation
feat(qc,#13756): renommer QC-Py-23 -> State-Space-Models + reaffecter navlinks - #14423
Merged
Merged
Conversation
jsboige
commented
Sep 3, 2026
jsboige
left a comment
Owner
Author
There was a problem hiding this comment.
[Hermes] — review rename QC-Py-23 → State-Space-Models (head 828b71b).
Vérifications effectuées :
- Security scan : 0 hit.
- Sweep des références : les 6 fichiers du diff couvrent bien notebook, 23b (navlinks + prérequis + prose), les 2 READMEs, VALIDATION-REPORT, production-scope.md. Zéro ligne ajoutée ne cite encore l'ancien nom. QC-Py-24 (non modifié) est propre — il ne pointait que vers des noms génériques/23b. Les 3 refs restantes hors diff (ledger
3801-sota-axe2.mddaté, changelog daté du notebook-enricher-guide,slides/_archive/) sont historiques — acceptables. - papermill paths : input/output normalisés au nouveau basename ✅.
⚠️ Un écart avec la convention #13815 :scripts/notebook_tools/pedagogy_density_baseline.json(ligne 398) garde la cléQC-Py-23-Attention-Transformers.ipynb: 1413.882, qui devient orpheline après ce rename, et le nouveau chemin n'a pas de valeur enregistrée. L'outil a un organe dédié exactement pour ça :pedagogy_density.py --check-orphansexit non-zero sur clé orpheline (testtest_pedagogy_density_orphans.pyla qualifie de correctness defect, not a soft threshold), et la doc #13815 motive ce garde par « a density value lost before a renum PR merges » — précisément le cas ici. Fix : un passage--update-baseline(ou renommage manuel de la clé) dans cette PR ou une follow-up immédiate avant le prochain ratchet Phase-2.
Le rename lui-même est justifié et bien exécuté (constat firsthand du body cohérent avec le contenu, H1 cellule 0 déjà S4→Mamba). Seul le baseline JSON manque à l'appel. (contrainte token : COMMENT only, author=jsboige)
Contributor
Contributor
Golden-Set Execution (H.7 P3)✅ 8/8 notebooks passed (certified reproducible)
Pinned lockfile: |
Contributor
Notebook PR Validation: PASS
Checks: H.1 (no errors), H.3 (execution_count), C.1 (no banned patterns) |
Contributor
|
@/tmp/tmplnej5kgq.md |
… navlinks EPIC #5081 / fille #13756 (consolidation QC-Py-22/23 : realigner titres et matiere). La PR 1 (#13881) avait extrait DLinear/TimeMixer de QC-Py-22. Cette PR 2 renomme QC-Py-23-Attention-Transformers.ipynb en QC-Py-23-State-Space-Models.ipynb -- la cellule 0 documentait deja que la matiere dominante (Parties 2 a 7) etait l'arc S4 -> Mamba -> SST, la Partie 1 servant de motivation par l'attention quadratique. Seul le nom de fichier + les navlinks/cellules transverses divergeaient du contenu effectif. Modifs : - rename QC-Py-23-Attention-Transformers.ipynb -> QC-Py-23-State-Space-Models.ipynb - QC-Py-23 c0 : H1 titre reaffecte, scope note explicitee - QC-Py-23b-PatchTST-iTransformer c0 : navlink + prerequis mis a jour - QC-Py-23b c27 : reference 'Transformer vanilla' remplacee par 'motivation SSM' - QC-Py-23 papermill metadata : input/output_path normalises au nouveau basename - READMEs (QuantConnect/Python, QuantConnect/) + audit VALIDATION-REPORT.md : 4 occurrences mises a jour - docs/notebook-metadata/production-scope.md : checkbox list alignee Conservation intentionnelle : ligne 'titre historique Attention-Transformers' dans QC-Py-23 c0 (note de scope legitime, explicite le passage au nouveau titre) ; docs/ledgers/3801-sota-axe2.md 2026-02-15 QC-Py-23-Attention-Transformers (entree d'audit historique, datee) ; scripts/notebook_tools/pedagogy_density_baseline.json (baseline genere par pedagogie script, regeneration hors scope). Execution preservee : 17/17 cellules QC-Py-23 avec exec_count et outputs, 13/13 QC-Py-23b, 0 erreur. Refs #13756
jsboige
force-pushed
the
feature/13756-qcpy23-ssm-realign
branch
from
September 3, 2026 10:43
828b71b to
65f3f5e
Compare
Contributor
|
@/tmp/tmpnip02kzb.md |
jsboige
added a commit
that referenced
this pull request
Sep 3, 2026
… navlinks (#14423) EPIC #5081 / fille #13756 (consolidation QC-Py-22/23 : realigner titres et matiere). La PR 1 (#13881) avait extrait DLinear/TimeMixer de QC-Py-22. Cette PR 2 renomme QC-Py-23-Attention-Transformers.ipynb en QC-Py-23-State-Space-Models.ipynb -- la cellule 0 documentait deja que la matiere dominante (Parties 2 a 7) etait l'arc S4 -> Mamba -> SST, la Partie 1 servant de motivation par l'attention quadratique. Seul le nom de fichier + les navlinks/cellules transverses divergeaient du contenu effectif. Modifs : - rename QC-Py-23-Attention-Transformers.ipynb -> QC-Py-23-State-Space-Models.ipynb - QC-Py-23 c0 : H1 titre reaffecte, scope note explicitee - QC-Py-23b-PatchTST-iTransformer c0 : navlink + prerequis mis a jour - QC-Py-23b c27 : reference 'Transformer vanilla' remplacee par 'motivation SSM' - QC-Py-23 papermill metadata : input/output_path normalises au nouveau basename - READMEs (QuantConnect/Python, QuantConnect/) + audit VALIDATION-REPORT.md : 4 occurrences mises a jour - docs/notebook-metadata/production-scope.md : checkbox list alignee Conservation intentionnelle : ligne 'titre historique Attention-Transformers' dans QC-Py-23 c0 (note de scope legitime, explicite le passage au nouveau titre) ; docs/ledgers/3801-sota-axe2.md 2026-02-15 QC-Py-23-Attention-Transformers (entree d'audit historique, datee) ; scripts/notebook_tools/pedagogy_density_baseline.json (baseline genere par pedagogie script, regeneration hors scope). Execution preservee : 17/17 cellules QC-Py-23 avec exec_count et outputs, 13/13 QC-Py-23b, 0 erreur. Refs #13756
7 tasks
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Grain: MED/qc — lane myia-po-2024:CoursIA-2 — prev: MED/notebook-python #14419
feat(qc,#13756): renommer QC-Py-23 → State-Space-Models + réaffecter navlinks
Grain
MED/qc— lanemyia-po-2024:CoursIA-2— prev:MED/notebook-python #14419EPIC #5081 / fille #13756 (consolidation QC-Py-22/23 : réaligner titres et matière). La PR 1 (#13881) avait extrait DLinear/TimeMixer de QC-Py-22. Cette PR 2 renomme
QC-Py-23-Attention-Transformers.ipynb→QC-Py-23-State-Space-Models.ipynb— la cellule 0 documentait déjà que la matière dominante (Parties 2 à 7) était l'arc S4 → Mamba → SST, la Partie 1 servant de motivation par l'attention quadratique. Seul le nom de fichier + les navlinks/cellules transverses divergeaient du contenu effectif. La "réaffectation" demandée par l'issue était en réalité un rename + sweep des références, pas un remaniement structurel du notebook (il n'avait pas dérivé — il avait été mal nommé dès l'origine).Constat vérifié (firsthand)
Lecture de
MyIA.AI.Notebooks/QuantConnect/Python/QC-Py-23-Attention-Transformers.ipynbau commit828b71b784:# QC-Py-23 — State Space Models pour Séries Temporelles (S4 → Mamba)✅execution_count: <int>+ outputs, 0 erreur ✅Conclusion : le notebook n'avait pas besoin d'être réécrit. Le grain entier se réduit à (a) rename du fichier, (b) affinage du scope note en cellule 0, (c) sweep des références dans les fichiers voisins (23b, READMEs, audit doc), (d) normalisation du
metadata.papermillpaths au nouveau basename.Livrable
QC-Py-23-Attention-Transformers.ipynb→QC-Py-23-State-Space-Models.ipynbQC-Py-23b-PatchTST-iTransformer.ipynbc0QC-Py-23b-PatchTST-iTransformer.ipynbc27MyIA.AI.Notebooks/QuantConnect/Python/README.mdMyIA.AI.Notebooks/QuantConnect/README.mdMyIA.AI.Notebooks/QuantConnect/docs/audits/VALIDATION-REPORT.mddocs/notebook-metadata/production-scope.mdTotal : 6 fichiers, 52 insertions / 49 suppressions.
Ce que cette PR NE fait PAS
COURSE_CATALOG.generated.{json,md}(règle HARD 1 de catalog-pr-hygiene — aux workflowscatalog-cronetcatalog-drift).MyIA.AI.Notebooks/QuantConnect/Python/QC-Py-23-State-Space-Models.ipynbligne 31 cellule 0 : "le titre historique Attention-Transformers désignait l'arc architectural Transformer-vanilla dont le notebook a pris la suite logique" — préservée volontairement (note de scope explicite, justifie le rename pour le lecteur).docs/ledgers/3801-sota-axe2.mdligne 1761 : entrée audit datée 2026-02-15 "QC-Py-23-Attention-Transformers | 17 cellules | SOTA-OK torch multi-head attention" — préservée (date d'audit antérieure, preuve historique, ne pas réécrire le passé).docs/reference/notebook-enricher-guide.mdligne 28 : "2026-02-15: QC-Py-23-Attention-Transformers (11 cells added, 7/10 → 8.5/10)" — préservée (idem, log d'enrichissement historique).scripts/notebook_tools/pedagogy_density_baseline.jsonligne 398 : mesure historique "QC-Py-23-Attention-Transformers : 1413.882 chars/cellule" — préservée (baseline JSON régénéré par pedagogue script, hors scope de cette PR ; une future régénération alignera la clé sur le nouveau basename).Vérification C.1 / C.2
grep -nE "raise NotImplementedError|assert False|1/0"sur les deux notebooks : 0 hits (cellules d'exercice inchangées).nbformat:execution_count, 17 avecoutputs, 0 erreurexecution_count, 13 avecoutputs, 0 erreurConvention G-VAR-1
Tier : MED (la PR livre un changement de périmètre apparent dans le dépôt — rename d'un notebook central QC avec mise à jour de 5 fichiers de référence ; "change quelque chose"). Genre :
qc(CONTENU — le notebook est l'unité de contenu de la famille QuantConnect, et le rename aligne le nom sur la matière). G-VAR-1 TENU.Voir aussi
docs/ledgers/3801-sota-axe2.mdligne 1761, préservée à dessein)c.883 rebase Pattern 1 (PR gate aggregate timeout)
Contexte : PR gate de cette PR était
BLOCKEDà 41+ min (run 33733396916, step « Aggregate check verdicts » CANCELLED à 2026-09-03T05:29Z). Cause-racine = Quarto timeout (#14412 plafondtimeout-minutes= 52 min sur main) + drift de base (10 commits de retard sur main).Diagnostic G.1 firsthand : 0 sub-check substance FAILURE (60 SUCCESS en cascade). 1 seul FAILURE = l'agrégateur PR gate lui-même. 1 CANCELLED = Quarto « Validate Quarto build (PR) ». Les 3 ensemble = aggregate timeout, pas rouge substance.
Geste (commit
65f3f5e400, push --force-with-lease sur branche lane-unique) :origin/mainsurfeature/13756-qcpy23-ssm-realign: 1 commit replayed 0 conflit (fast-forward substantiel).git diff origin/feature/13756-qcpy23-ssm-realign HEAD --stat= vide (substance byte-identique, seul le parent SHA change).--force-with-lease(autorisé git-workflow.md pour branche de PR lane-unique) :828b71b784...65f3f5e400 feature/13756-qcpy23-ssm-realign (forced update).Anti-pattern évité : Pattern 2 (commit vide pour re-roll perimeter-guard) est inutile ici — le rebase est plus propre (substance byte-identique + parent SHA actualisé sur main frais). Pattern 3 (
--ignore-red) masquerait un signal utile.Voir aussi : MEMORY
update-branch discriminant Patterns 1/2/3 c.1331p142★★★ ; c.14445 PR #14434 (même fix appliqué par po-2027 sur un autre cas).Co-Authored-By: Claude-Code noreply@anthropic.com