[codex] add cross-provider analysis showcase#20
Draft
hudsonaikins wants to merge 1 commit into
Draft
Conversation
📚 Documentation Status✅ Code changes detected
This comment is automatically generated by the documentation workflow. |
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.
Summary
This PR tightens the SDK's cross-provider data story and gives the examples directory a clearer proof that the analysis stack works on standardized external market data.
On the data side, it fixes Kalshi sports parsing so ticker dates and matchup extraction resolve correctly for NBA and related helpers, and it expands the Polymarket US adapter so public market reads work without credentials where the upstream public Retail Market API allows it. That means the SDK can now pull public Kalshi and Polymarket snapshots through a more consistent normalized interface, instead of treating Polymarket reads as auth-only.
On the examples side, it adds a progressive showcase set:
buy_yes,buy_no, andholddecisions on one standardized streamThe deterministic replay matters because live public feeds are not reliable as a proof artifact for analysis quality. Live markets can stay flat, move too little, or expose incomplete public quote depth, which makes the pipeline look inert even when the strategy layer is correct. The replay gives us a reproducible artifact that demonstrates normalization, stream construction, signal generation, and risk sizing under controlled conditions.
Root Cause
Before this change, two gaps made the SDK harder to demonstrate clearly:
First, Kalshi sports parsing had a date interpretation bug and some brittle title parsing, which meant derived game metadata could be wrong even when the underlying market pull succeeded.
Second, the Polymarket US integration required credentials up front even for public market data, and the examples directory did not yet show a strong cross-provider narrative from raw pulls through analysis decisions.
Fix
The PR:
Validation
I validated the change set with targeted checks focused on the touched modules and examples:
The deterministic replay run produced the expected mix of
buy_yes,buy_no, andholdoutputs and wrote inspectable CSV artifacts underdata/deterministic_cross_provider/.