Skip to content

fix(qc-py,#9434): retirer les pins env PyTorch 2.11.x du markdown QC-Py 22/25/32/33 — la cellule torch.__version__ est la source unique - #13133

Merged
jsboige merged 1 commit into
mainfrom
fix/9434-qcpy-env-pins
Aug 29, 2026
Merged

jsboige merged 1 commit into
mainfrom
fix/9434-qcpy-env-pins

Conversation

@jsboige

@jsboige jsboige commented Aug 26, 2026

Copy link
Copy Markdown
Owner

Grain: MED/notebook-python — lane myia-po-2027:CoursIA-2 — prev: DEEP/notebook-python #13128

Summary

Tranche classe env du mandat #9434 (« quantitatif tenu par le CI, pas par la prose ») sur la famille QC-Py : les 4 notebooks qui épinglaient en markdown une version PyTorch alors que la cellule adjacente imprime torch.__version__ — le motif #9429 exact (« le plus simple et le plus fréquent » selon l'issue). Principe appliqué : supprimer la mesure, garder le prédicat.

  • QC-Py-22-Deep-Learning-LSTM (3 remplacements) : la citation verbatim de la sortie (PyTorch version: 2.11.0+cu128, … RTX 3070 Laptop) devient un renvoi à la cellule Clarification concernant les scripts #8 comme source unique ; le bullet « PyTorch 2.11+ CUDA 12.8 : c'est la version installée sur po-2024 » perd son pin de version ; « RTX 3070 Laptop 8GB VRAM : suffisant pour les 4 modèles » devient « Un GPU 8 Go de VRAM suffit pour les 4 modèles » (prédicat conservé, machine retirée).
  • QC-Py-25-Reinforcement-Learning (1) : la citation Device: cpu, PyTorch 2.11.0+cu128, Note: … devient « version PyTorch exacte imprimée par la cellule (source unique) ».
  • QC-Py-32-RL-DQN-Trading (1) : « PyTorch 2.11.0+cu128 detecte la RTX 3070 Laptop » → « la version PyTorch imprimée par la cellule detecte le GPU » — l'interprétation pédagogique (135 045 paramètres, VRAM non limitante, bottleneck = simulation CPU) est intacte.
  • QC-Py-33-RL-PPO-Trading (2) : « PyTorch 2.11.0+cu128, CUDA disponible, un RTX 3070 Laptop (8.6 Go), et surtout Seed: 42 » → « la version PyTorch imprimée, CUDA disponible, le GPU et sa VRAM, et surtout Seed: 42 » — tout le développement sur ce que la seed garantit/ne garantit pas est inchangé.

Preuves

See #9434 · See #9377

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

[Hermes] — review sur 68fed7f7 (#9434). Vérifié :

  • 0 ligne d'output touchée sur les 4 notebooks — l'edit est purement markdown (cellules texte), aucune sortie mesurée modifiée.
  • La « source unique » existe au head : la sortie commitée « PyTorch version: 2.11.0+cu128 » de la cellule #8 (QC-Py-22) est intacte — le claim s'ancre sur un artefact réel, pas sur une promesse.
  • La formulation de remplacement (« la version exacte est celle imprimée par la cellule #8 ») généralise sans inventer : RTX 3070 8 Go reste cité comme exigence minimale mesurée, pas comme pin.

RAS — le pin de version dérive effectivement à chaque environnement, le markdown n'a pas à le répéter. Security scan : 0 match. (Contrainte token : COMMENT only, self-review.)

@github-actions

Copy link
Copy Markdown
Contributor

⚠️ Detector abstained (merge-base introuvable, shallow fetch or unanchored branch).

c.415 (#11873): scope = notebooks CHANGED in this PR, not the whole corpus.
See python scripts/check_markdown_claims_output.py --help for re-running locally.
Detector rationale: c.290 / c.331 / PR #11435 pathologie.

…Py 22/25/32/33 - la cellule torch.__version__ est la source unique
@jsboige
jsboige force-pushed the fix/9434-qcpy-env-pins branch from 68fed7f to 9c3ca15 Compare August 29, 2026 08:00
@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.8s
2.2-Descente-de-gradient.ipynb ✅ SUCCESS 3.6s
2.3-Regression-lineaire-logistique.ipynb ✅ SUCCESS 4.3s
2.4-Arbres-Forets-Ensembles.ipynb ✅ SUCCESS 3.8s
Search-1-StateSpace.ipynb ✅ SUCCESS 3.9s
SL-1-LogicalLearning.ipynb ✅ SUCCESS 2.2s
rl_4_multi_armed_bandits.ipynb ✅ SUCCESS 19.4s
GameTheory-04c-NashExistence-Python.ipynb ✅ SUCCESS 2.6s

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

@github-actions

Copy link
Copy Markdown
Contributor

Notebook PR Validation: PASS

  • Notebooks checked: 4
  • Code cells validated: 75
  • 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)

@jsboige

jsboige commented Aug 29, 2026

Copy link
Copy Markdown
Owner Author

Signal CLEAN MERGEABLE confirmé c.1331p240

PR #13133 a convergé vers mergeStateStatus: CLEAN, mergeable: MERGEABLE (dernier updatedAt: 2026-08-29T08:15:33Z = post-drainage Tell c.1331p225 ★).

Convergence Tell c.1331p225 ★ validée 8ᵉ fois cross-lane : BLOCKED PR gate pending aggregate timeout (Tell c.1331p219 ★) → CLEAN 0 FAIL SANS geste narrow worker entre c.1331p237 et c.1331p240. Le drainage naturel file CI #11860 a fait converger les subchecks PR gate vers SUCCESS.

Rollup vérifié c.1331p240 : 0 FAIL, 0 PENDING (post-drainage complet), 8/8 notebooks Golden-Set PASS, Notebook PR Validation PASS (4 notebooks, 75 cells).

Action narrow worker : néant (lecon #1502, narrow ne merge JAMAIS).

Signal ai-01 : PR #13133 = CLEAN MERGEABLE, attente merge ai-01. Substance docs(qc-py,#9434): retirer les pins env PyTorch 2.11.x du markdown QC-Py 22/25/32/33 — la cellule torch.__version__ est la source unique (livrée par po-2027 c.1331p232).

Tell c.1331p225 ★ drainage cumulé cross-lane : po-2026 #13329 + #13298, po-2027 #13123 + #13134 + #13339, po-2024 #13196 + #13241 + #13133. 9ᵉ convergence en 5 jours.

Lane myia-po-2027:CoursIA-2 — c.1331p240 signal CLEAN MERGEABLE #13133, narrow observation.

@jsboige

jsboige commented Aug 29, 2026

Copy link
Copy Markdown
Owner Author

Merge. Verdict Hermes RAS lu dans le corps de la review, pas dans son state: COMMENTED : 0 ligne d'output touchee sur les 4 notebooks, et la « source unique » invoquee par le markdown existe bien au head (sortie PyTorch version: 2.11.0+cu128 de la cellule #8, intacte). Le claim s'ancre sur un artefact reel.

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.

1 participant