Skip to content

fix(data): accept global open interest rows#82

Merged
cesarenaldi merged 1 commit into
mainfrom
fix/open-interest-global-market
Jun 9, 2026
Merged

fix(data): accept global open interest rows#82
cesarenaldi merged 1 commit into
mainfrom
fix/open-interest-global-market

Conversation

@cesarenaldi

@cesarenaldi cesarenaldi commented Jun 9, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Accept the Data API open-interest aggregate row where market is GLOBAL.
  • Keep CTF condition ID validation for normal market rows.
  • Add unit coverage for the aggregate sentinel.

Verification

  • uv run pytest tests/unit/test_data_models.py -q
  • uv run pytest tests/integration/test_data.py -q
  • uv run ruff check src/polymarket/models/data/analytics.py tests/unit/test_data_models.py
  • uv run pyright
  • uv run ruff check src tests

Note: full local uv run pytest tests/unit -q hit an unrelated RPC connect timeout in test_secure_client_defaults_wallet_to_current_deposit_wallet; main CI unit jobs had already passed.

@cesarenaldi cesarenaldi merged commit 1cfda2d into main Jun 9, 2026
7 checks passed
@cesarenaldi cesarenaldi deleted the fix/open-interest-global-market branch June 9, 2026 17:07
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