Repository navigation
Conversation
Co-Authored-By: Codex <noreply@openai.com>
This branch has not been deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Adds local-only dual-basis equity replay, paired-bundle export, and review-bound manifest generation/validation. Each basis and gross/1x/2x cost scenario maintains independent cash/holdings; records include window-opening state and scenario-specific fills/turnover. Metadata, input/code identities and complete object references are bound to review records; null/unknown declarations and synthetic dispatch manifests are rejected.
Validation: 50 focused synthetic tests, full-repository Ruff 0.15.9, and an end-to-end synthetic run on exchange-calendars 4.13.2 XNYS sessions. Four aliases sharing one synthetic adapter produce 8 structurally valid windows; omitting inception leaves 4. Real comparisons remain 0/8. No cloud input reads/dispatch, broker execution, deployment, credential or permission changes.
Draft for the batch's unified review. This is a restricted equity engine, not a qualified replay of the four real strategies. Real strategy/plugin adapters, historical inputs, enabled option joint accounting, non-linear/minimum-fee costs, full producer qualification and reviewed cloud identities/read access remain missing. Unsupported modes stop; existing consumer cost-return ordering stays enforced, so crossing independent return paths also stop. Review records bind bytes but do not authenticate a reviewer or authorize dispatch. No trading-diff PASS or live readiness is claimed.