Publish strict index ETF inputs to both private vaults - #44
Merged
Merged
Conversation
Co-Authored-By: Codex <noreply@openai.com>
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
1 participant
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.
The strict index ETF runner needs raw execution prices, market status, calendar and corporate-action inputs that the existing learning CSV does not provide. Add a local JSON importer that retains source bytes and license evidence, validates through the actual CN consumer, and publishes the immutable manifest to both existing private vaults.
Keep synthetic input as the default. A historical declaration cannot grant trusted provenance or promotion authority; the owning research job must independently approve the exact input root. Missing fields, unsafe paths, changed content or an incomplete second vault fail without claiming successful publication. Existing acquisition and publication paths remain unchanged.
Pin the published CN and QPK revisions in package metadata and the lockfile. A fresh installation without source overrides verified both exact installed commits and passed all 145 repository tests with networking denied. Dependency checks, Ruff, diff checks and wheel/sdist builds passed. Independent review covered actual consumer readback and recovery after the second vault fails. No real market data was acquired and no model, shadow or trading run was started.