Skip to content

Fix official backtests without feature pins - #91

Merged
pjy008008 merged 1 commit into
developfrom
codex/allow-empty-official-features
Aug 9, 2026
Merged

Fix official backtests without feature pins#91
pjy008008 merged 1 commit into
developfrom
codex/allow-empty-official-features

Conversation

@pjy008008

Copy link
Copy Markdown
Contributor

What changed

  • allow official backtest requests with zero feature materialization pins
  • add a regression test for strategies that require only market-bar inputs

Root cause

The Backend correctly emitted featureMaterializations: [] for strategies with no precomputed feature requirements. The Backtest consumer schema required at least one item, rejected the request as CONTRACT_VIOLATION, and moved it to the request DLQ while the run remained queued.

User impact

Basic strategies that do not require feature materializations can now pass request intake and proceed to execution.

Verification

  • reproduced the failure before the fix: official_backtest_request.featureMaterializations: [] should be non-empty
  • ruff check src tests
  • full Docker-free pytest suite

Product authority instruction

Authority: user:pjy008008

이거 수정해줘 PR생성하고 CI통과하면 merge시켜줘.

@pjy008008
pjy008008 merged commit 62db062 into develop Aug 9, 2026
9 checks passed
@pjy008008
pjy008008 deleted the codex/allow-empty-official-features branch August 9, 2026 20:58
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