Skip to content

Fix field evaluation for live OHLCV bars - #21

Merged
Mopati123 merged 1 commit into
masterfrom
codex/fix-field-evaluation-live-ohlcv
May 8, 2026
Merged

Fix field evaluation for live OHLCV bars#21
Mopati123 merged 1 commit into
masterfrom
codex/fix-field-evaluation-live-ohlcv

Conversation

@Mopati123

Copy link
Copy Markdown
Owner

Summary

  • Allows the field Hamiltonian tensor to consume live canonical OHLCV bar lists as well as historical dict-style OHLCV arrays.
  • Adds a regression test for the live-bar-list shape that blocked the first post-merge MT5 canary.

Why

After PR #20 merged, the first controlled MT5 canary stopped before broker execution because stage_field_evaluation passed market_state["ohlcv"] as a list of canonical bar dictionaries, while compute_maxwell_tensor only supported dict-style arrays and called .get() on the list.

Validation

  • git diff --check
  • python -m pytest tests/rootfile/test_lawful_collapse_universe.py -q -> 13 passed
  • python -m pytest tests/rootfile/test_mt5_position_close_tracker.py -q -> 3 passed
  • python -m pytest tests/rootfile -q -> 143 passed
  • python -m pytest tests -q -> 236 passed, 9 warnings

Controlled MT5 canary after fix

  • Ran one MT5-only demo canary from this branch after safety preflight.
  • Ticket 357844399 was created: SELL EURUSD_r, 0.01 lot, entry 1.17449, SL 1.17649, TP 1.17049.
  • PPO pending state persisted with key 357844399, proving the entry durability path is active.
  • Runner was stopped immediately after ticket observation.
  • Position remains open, so settlement/PPO close feedback must wait for SL/TP/manual close.

Safety

  • No Deriv or --mode both path was used.
  • No raw logs/ or data/ artifacts are committed.
  • kluster review was attempted, but the kluster.ai trial has ended and no verification output is available.

@Mopati123
Mopati123 merged commit cf7cc81 into master May 8, 2026
@Mopati123
Mopati123 deleted the codex/fix-field-evaluation-live-ohlcv branch May 12, 2026 15:08
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.

2 participants