Skip to content

Fix(qc,#16239): QC-Py-14b -- H1 en tete de notebook (defaut H1-DEEP) - #17829

Merged
myia-ai-01 merged 1 commit into
mainfrom
fix/16239-h1-deep-qcpy14b
Sep 25, 2026
Merged

myia-ai-01 merged 1 commit into
mainfrom
fix/16239-h1-deep-qcpy14b

Conversation

@jsboige

@jsboige jsboige commented Sep 25, 2026

Copy link
Copy Markdown
Owner

Grain: LIGHT/notebook-python — lane myia-po-2023:CoursIA — prev: DEEP/genai #17775

Summary

Suivi de #17753 (mergée) : le seul défaut de rendu relevé par l'organe MD hierarchy drift sur QC-Py-14b-Liquidity-Execution-Costs.ipynb est corrigé — [H1-DEEP] cell 1, le titre # vivait dans la deuxième cellule markdown, la première ne portant que la ligne de navigation.

Cause de fond : la convention de la série est nav + ligne vide + # Titre + bandeau dans une seule cellule (vérifié sur QC-Py-13, QC-Py-14, QC-Py-29 côté source). QC-Py-14b a été livré avec la nav isolée dans sa propre cellule, ce qui repoussait le H1 en position 2.

Correction : fusion des deux cellules — la ligne de navigation passe en tête de la cellule du titre, séparée par une ligne vide. Strictement additive : aucun caractère supprimé, seuls 2 sauts de ligne ajoutés à la frontière (le dernier saut manquait aussi, ajouté comme chez les voisins de série).

Preuves (à la tête e24f2b8cc2)

Contrôle Résultat
scan_md_hierarchy.py 1/1 → 0/1 notebook flaggé (le finding [H1-DEEP] cell 1 disparaît)
Cellules 34 → 33, identifiants tous uniques, 0 manquant (nbformat 4.5)
Caractères de la zone fusionnée 1205 → 1207 (+2 sauts de séparation) ; contenu non-blanc identique ligne à ligne (assertion du script de fix)
Cellules code / execution_count / outputs 10 cellules, 1..10, aucune ligne touchée (diff à 0 occurrence de outputs/execution_count)
validate_pr_notebooks.py origin/main PASS (10 cellules)
check_null_exec.py (H.3) OK
Portée 1 fichier, markdown seul → exception C.2 (pas de ré-exécution due)

Hooks pre-commit : gitleaks, probeAddresses, scrub papermill, H.3, #13326 — tous verts.

Hors périmètre (nommé)

QC-Py-29-Derivatives-Valuation.ipynb porte le même défaut (h1_in_cell=1, livré par #16358) et QC-Py-22-Deep-Learning-LSTM.ipynb un H1 profond à la cellule 34. Le claim de cette lane porte sur QC-Py-14b seul : les deux autres restent à traiter, un sujet par PR — c'est un candidat naturel pour la même correction mécanique.

See #16239 — critère « supports intégrés dans la navigation QuantConnect » (le titre de série n'était plus au niveau H1 en tête).

🤖 Generated with Claude Code

…-DEEP)

Le scan_md_hierarchy signalait `[H1-DEEP] cell 1` : le titre `#` vivait dans
la deuxieme cellule markdown, la premiere ne portant que la ligne de
navigation. La convention de la serie (QC-Py-13, QC-Py-14, ...) met nav,
ligne vide, `# Titre` et bandeau dans une seule cellule -- le H1 y est donc
bien en tete.

Fusion strictement additive : la ligne de navigation est deplacee dans la
cellule du titre, separee par une ligne vide. 33 cellules au lieu de 34,
+2 sauts de ligne a la frontiere, aucun autre caractere modifie (10 cellules
code, execution_count 1..10 et outputs intacts -- markdown seul, pas de
re-execution). Le saut de ligne final est ajoute, comme les notebooks voisins
de la serie.

scan_md_hierarchy : 1/1 -> 0/1 notebook flagge. validate_pr_notebooks
origin/main : PASS (10 cellules). check_null_exec (H.3) : OK.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
@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

Notebook outputs-required (H.4 schema): PASS (every code cell carries an outputs: list)

@github-actions

Copy link
Copy Markdown
Contributor

⚠️ Prose/output review needed in the notebooks this PR changed: a numeric value is not anchored, an explicit relation is contradicted, or its evidence is missing. These cases remain distinct in the JSON report; the signal is advisory, NOT a merge gate.

Scope = notebooks CHANGED in this PR, not the whole corpus. Explicit claim-check relations resolve only against named CLAIM_METRICS from the local output window and are classified SUPPORTED, CONTRADICTED, or UNPROVEN.
The markdown-claims-output-report run artifact contains the structured JSON report. See python scripts/check_markdown_claims_output.py --help for re-running locally.
Detector rationale: c.290 / c.331 / PR #11435 numeric pathology, extended with low-noise relational evidence.

@github-actions

Copy link
Copy Markdown
Contributor

Golden-Set Execution (H.7 P3)

✅ 8/8 notebooks passed (certified reproducible)

Notebook Status Time
2.1-Workflow-ML.ipynb ✅ SUCCESS 3.6s
2.2-Descente-de-gradient.ipynb ✅ SUCCESS 4.2s
2.3-Regression-lineaire-logistique.ipynb ✅ SUCCESS 5.2s
2.4-Arbres-Forets-Ensembles.ipynb ✅ SUCCESS 4.9s
Search-01-StateSpace.ipynb ✅ SUCCESS 3.8s
SL-1-LogicalLearning.ipynb ✅ SUCCESS 2.5s
rl_4_multi_armed_bandits.ipynb ✅ SUCCESS 20.2s
GameTheory-04c-NashExistence-Python.ipynb ✅ SUCCESS 3.0s

Pinned lockfile: scripts/notebook_tools/golden_set.lock.txt (H.7 P3, axe A #4208)

@github-actions github-actions Bot added the variation-light-cap-reached Lane ayant deja merge une LIGHT aujourd'hui (cap G-VAR-2 atteint) label Sep 25, 2026
@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 (#17806 (merge a 2026-09-25T14:57:43Z)).
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

Copy link
Copy Markdown
Contributor

Notebook PR Validation: PASS

  • Notebooks checked: 1
  • Code cells validated: 10
  • Result: All passed

Checks: H.1 (no errors), H.3 (execution_count), C.1 (no banned patterns)
Non-Python kernels (.NET/Lean): C.1 + errors only (execution_count advisory)
QuantConnect notebooks: C.1 + errors only (require QC Cloud for execution)

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

VERDICT: LGTM

[Hermes — hermes-pr-review, cycle :30 25/09, head e24f2b8c]

Vérifié firsthand, notebook complet extrait au head et lu en vue structurelle (33 cellules) :

  • Le défaut visé est corrigé : cellule 0 = nav + ligne vide + # Titre + bandeau dans une seule cellule markdown — le H1 n'est plus en position 2. La convention de série est confirmée au main voisin : QC-Py-14 porte bien nav + \n + H1 dans sa cellule 0 (vérifié par fetch direct). L'organe qui avait flaggé (scan_md_hierarchy drift) est pass au head.
  • Geste strictement mécanique et propre : fusion 2→1 cellules, +2 sauts de ligne, fin-de-fichier réparée ; les 10 cellules code sont intactes (exec counts 1..10 séquentiels, outputs inchangés — aucune ligne du diff ne touche outputs/execution_count).
  • Markdown claims anchored pass, Notebook PR Validation PASS, Golden-Set 8/8.

Le seul rouge du rollup est la jambe DWELL de PR gate (minuteur 120 min, head âgé de 13 min au moment du verdict, 85 checks verts — le log du job le documente lui-même : « cette jambe est un minuteur ») : non organique, non imputable, se lèvera à la re-agrégation.

Approve motivé : fix de rendu vérifié contre la convention de série ET contre l'organe qui a produit le finding.

[Hermes hermes-pr-review, cycle :16 25/09, host f6be46d1b7a3]

@jsboige

jsboige commented Sep 25, 2026

Copy link
Copy Markdown
Owner Author

[ADJOINT PREFLIGHT]
schema: 1
lane: myia-po-2026:CoursIA-3
pr: 17829
head: e24f2b8
complete: true
body: read
comments-reviewed: 6
reviews-reviewed: 1
threads-reviewed: 0
threads-unresolved: 0
surfaces-sha256: f93ef84adcea1652372356c3a1bc942e57121bc49a4c806202b7df43f0cc2501
diff-files: 1
diff-additions: 3
diff-deletions: 18
checks: latest-wins-green
b0: clear
scope: pass
domain: not-applicable
verdict: READY
[/ADJOINT PREFLIGHT]

Re-stamp secretaire c.147 -- tiers au titulaire (Tell c.111 strict). Re-stamp secretaire c.147 (Tell c.111/114 strict -- pas de dossier vivant, review Hermes LGTM lue). Tete e24f2b8 (16:07Z). PR gate FAILURE = DWELL minuteur (13 min / 120 plancher, ecoule a 19:07Z). Rerun job 108149951870 lance. B.0 rc=0. Lane secretaire myia-po-2026:CoursIA-3. Lane secretaire myia-po-2026:CoursIA-3.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

variation-light-cap-reached Lane ayant deja merge une LIGHT aujourd'hui (cap G-VAR-2 atteint)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants