Skip to content

fix: validate lifecycle backtest execution results - #194

Open
Pigbibi wants to merge 2 commits into
mainfrom
codex/qsl-20260923-b1-crypto-audit
Open

Pigbibi wants to merge 2 commits into
mainfrom
codex/qsl-20260923-b1-crypto-audit

Conversation

@Pigbibi

@Pigbibi Pigbibi commented Sep 23, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Pass supported lifecycle parameters into the executed strategy configuration and reject unsupported fields.
  • Record the observed return count and actual return window.
  • Reject malformed observed returns before producing ordinary metrics while preserving leading warmup NaNs.

Validation

  • Python 3.12 CI dependency lock (requirements-lock.txt): targeted unittest discovery passed (7 runner tests, 11 metric tests).
  • Existing uv.lock frozen environment: targeted pytest passed (18 tests).
  • git diff --check passed.

No dependency or lockfile changes.

Pigbibi and others added 2 commits September 23, 2026 23:25
Co-Authored-By: Codex <noreply@openai.com>
Co-Authored-By: Codex <noreply@openai.com>
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