Skip to content

Match rounded money for close-first entries and opening margin calls - #196

Merged
luisleo526 merged 2 commits into
mainfrom
round12/candidate-x
Sep 6, 2026
Merged

Match rounded money for close-first entries and opening margin calls#196
luisleo526 merged 2 commits into
mainfrom
round12/candidate-x

Conversation

@luisleo526

Copy link
Copy Markdown
Collaborator

Apply TradingView's rounded signal-cost check when an opposite entry follows a same-bar close. Use rounded required money at the existing opening-margin checkpoints, including before a priced exit. The pinned May 28 trim becomes 62.32 instead of 62.28; the June 9 opening now includes the missing one-unit margin call.

Validation on engine c63be8880c49576300b2a53d1889a2e1c34dfb29 with codegen 3fd97fe28abd7b191cb376d9f5db4e056fcfae4b:

  • Cloud Run: 72/72 cases, all 4,190 probes measured. Demete and willow EURUSD convert from strong to excellent, with 100% canonical matching and zero canonical trade-count gap. Their PnL P90 remains 0.0005 and 0.0006.
  • Gate pineforge-pr-gate-xsrx6: PASS, zero regressions or tolerances across 704 hard probes, target two-band score +2. No tier drops or new moderate; result is 4,147 excellent / 42 strong / 1 moderate.
  • 202 C++ unit tests and the C ABI check passed. Independent Grok review of this exact combined commit: GREEN, P0/P1/P2 = 0.

Taro EURUSD remains strong and unresolved: its canonical match rate changes from 97.6% to 97.5% and PnL P90 from 0.0009 to 0.001, while the canonical trade-count gap improves from 5 to 3. These mixed metrics are retained explicitly.

Candidate snapshot: 1fccc6441684a1a75911ec16e8f9e0477b0b6a1213788280f8735be56eacf92a (788,867 bytes), compared with baseline engine ff4a6b897b66a97d78a7c5aa8ca4b02b6804bd94.

@luisleo526
luisleo526 merged commit c63be88 into main Sep 6, 2026
8 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.

1 participant