Skip to content

docs: plan de migration vers la méthode d'achat sur pullback (Martin Luk) - #110

Open
laurentvv wants to merge 1 commit into
mainfrom
docs/plan-migration-pullback
Open

laurentvv wants to merge 1 commit into
mainfrom
docs/plan-migration-pullback

Conversation

@laurentvv

Copy link
Copy Markdown
Owner

Résumé

Décision de l'utilisateur du 2026-10-12 : migration complète du projet vers la méthode d'achat sur pullback de Martin Luk. Cette PR ne contient aucun code : elle pose le plan, le contrat et les portes de passage.

  • Plan : docs/PLAN_MIGRATION_PULLBACK_2026-10-12.md
    • les quatre étapes traduites en règles mécaniques journalières : screener momentum (+30 % sur 1, 3 ou 6 mois, top 25 %), consolidation qui se resserre, support (EMA 9/21/50, AVWAP, anciens plus hauts), achat stop au plus haut du jour de setup, stop ≤ 3 %, 0,5 % de risque par trade, ventes à 3R et 5R, suivi à l'EMA 9 ;
    • univers : actions US disponibles sur T212 (6 898 _US_EQ sur 12 168 actions, relevé du 2026-10-12) ;
    • banc portefeuille multi-actions (gaps payés, pire cas intrajournalier, change 0,15 %, PFU 30 %), contrôles « entrées au hasard » et « momentum seul », mesure du biais du survivant, walk-forward 2015-2020 / 2021+ ;
    • porte de passage : pas de code d'exécution tant que la règle ne bat pas le buy & hold Nasdaq-100 EUR hors échantillon (§2.1, à valider) ;
    • LLM et TimesFM : classement et veto des candidats (LLM vision en « chartiste », thèmes et catalyseurs, TimesFM en variable de classement), journal A/B ; ils ne créent jamais de trade.
  • memory-bank : nouveau contrat (29 critères, remplace celui du 2026-08-19), features F-42 → F-51, progress et log. Ce commit inclut aussi les entrées d'audit du 2026-10-12 restées non commitées dans le checkout.
  • AGENTS.md : bandeau « migration en cours » pour les autres agents ; l'ancien plan du 2026-09-29 est marqué remplacé.

Effets sur le reste

Avant de merger

Après le merge, dans le checkout PROD, faire git checkout -- memory-bank/log.md memory-bank/progress.md avant le git pull : leur contenu est repris dans cette PR.

🤖 Generated with Claude Code

…n utilisateur du 2026-10-12)

Migration complète décidée par l'utilisateur : méthode de Martin Luk traduite en règles
mécaniques journalières sur les actions US disponibles sur Trading 212 (6 898 `_US_EQ`
relevées le 2026-10-12). LLM et TimesFM deviennent classement et veto des candidats.

- docs/PLAN_MIGRATION_PULLBACK_2026-10-12.md : règles, banc portefeuille, portes de passage,
  rôle de l'IA (journal A/B), exécution T212, phases et PR.
- memory-bank : nouveau contrat (29 critères, remplace celui du 2026-08-19), features
  F-42 → F-51, progress et log (y compris les entrées d'audit du 2026-10-12 restées locales).
- AGENTS.md : bandeau de migration ; ancien plan du 2026-09-29 marqué remplacé.

Aucun code modifié. Le scheduler démo continue de tourner sur l'ancien pipeline.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
### 1.1 Univers (« tout ce qui est facilement disponible sur T212 »)

- Source : `GET /equity/metadata/instruments` (relevé du 2026-10-12, compte démo) : 18 473 instruments, dont
**12 168 actions** : 6 898 en USD (`XXX_US_EQ`), 3 140 en EUR, 1 401 en GBX, 542 en CAD, 134 en CHF.

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

WARNING: Currency breakdown doesn't sum to stated total

The text states "12 168 actions" broken down as: 6 898 (USD) + 3 140 (EUR) + 1 401 (GBX) + 542 (CAD) + 134 (CHF) = 12 115. There's a discrepancy of 53 actions. Either the total is wrong, the breakdown is missing a currency, or one of the numbers has a typo.


Reply with @kilocode-bot fix it to have Kilo Code address this issue.

Comment thread AGENTS.md
@@ -4,6 +4,8 @@ This file gives future AI agents (Kilo, Codex, Copilot, etc.) the minimum contex

**Project in one paragraph** — Python 3.12 trading-decision pipeline. Multi-model ensemble (Scikit-Learn, TimesFM 3.0, TensorTrade PPO, NexusAI-Client unified cloud LLMs for text + vision + oil fundamentals + web research, sentiment, Vincent Ganne model) orchestrated by `main.py`. Outputs to `trading_journal.csv`, with optional Trading 212 demo/live execution via `--t212`. Virtualenv managed by `uv`. See `README.md` for the full feature list.

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

SUGGESTION: Main project description describes legacy architecture

The opening paragraph still describes the old ensemble (Scikit-Learn, TimesFM 3.0, TensorTrade PPO, NexusAI-Client LLMs, sentiment, Vincent Ganne model) without clarifying this is the current/legacy pipeline being replaced. The migration banner (line 7) warns about the change, but a new agent reading only the first paragraph would get an outdated picture. Consider adding a note like "(current legacy pipeline — being replaced per migration plan)" or updating the paragraph to reflect the target architecture.


Reply with @kilocode-bot fix it to have Kilo Code address this issue.

@kilo-code-bot

kilo-code-bot Bot commented Oct 12, 2026 •

Copy link
Copy Markdown

Code Review Summary

Status: 2 Issues Found | Recommendation: Address before merge

Overview

Severity Count
CRITICAL 0
WARNING 1
SUGGESTION 1
Issue Details (click to expand)

WARNING

File Line Issue
docs/PLAN_MIGRATION_PULLBACK_2026-10-12.md 36 Currency breakdown doesn't sum to stated total (12,115 vs 12,168)

SUGGESTION

File Line Issue
AGENTS.md 5 Main project description describes legacy architecture without clarification
Files Reviewed (7 files)
  • AGENTS.md - 1 issue
  • docs/PLAN_AMELIORATION_PASSAGE_REEL_2026-09-29.md - No issues
  • docs/PLAN_MIGRATION_PULLBACK_2026-10-12.md - 1 issue
  • memory-bank/contract.md - No issues
  • memory-bank/feature_list.json - No issues
  • memory-bank/log.md - No issues
  • memory-bank/progress.md - No issues

Fix these issues in Kilo Cloud


Reviewed by free · Input: 253.8K · Output: 8.4K · Cached: 380.2K

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