Skip to content

Add optional Adanos sentiment data adapter#239

Open
alexander-schneider wants to merge 1 commit intoedtechre:masterfrom
adanos-software:codex/adanos-sentiment-features
Open

Add optional Adanos sentiment data adapter#239
alexander-schneider wants to merge 1 commit intoedtechre:masterfrom
adanos-software:codex/adanos-sentiment-features

Conversation

@alexander-schneider
Copy link
Copy Markdown

@alexander-schneider alexander-schneider commented Apr 29, 2026

Summary

  • adds an optional AdanosSentiment data-source wrapper in pybroker.ext.data
  • enriches any existing PyBroker DataSource with Adanos sentiment custom columns for ML features and execution rules
  • documents the adapter and covers enrichment, missing-data behavior, source validation, cache-key isolation, timezone-aware date joins, daily-vs-period payload handling, aggregate opt-out, and HTTP client behavior with tests

Why

PyBroker already supports custom data columns and ML walkforward workflows. This adapter keeps OHLCV data ownership with the user's existing data source, while adding optional alternative-data features such as adanos_sentiment, adanos_buzz, and per-source Reddit/X/news/Polymarket columns.

The implementation does not add a required dependency and does not change existing behavior unless users opt into the wrapper.

Tests

  • .venv/bin/python -m ruff check src/pybroker/ext/data.py tests/test_data.py
  • .venv/bin/python -m ruff format --check src/pybroker/ext/data.py tests/test_data.py
  • git diff --check
  • .venv/bin/python -m pytest tests/test_data.py -q — 96 passed
  • .venv/bin/python -m pytest tests/ -q — 4077 passed

@alexander-schneider alexander-schneider marked this pull request as ready for review April 29, 2026 05:31
@alexander-schneider alexander-schneider force-pushed the codex/adanos-sentiment-features branch from 8658d7f to 9c3bf92 Compare April 29, 2026 05:36
@alexander-schneider alexander-schneider marked this pull request as draft April 29, 2026 05:37
@alexander-schneider alexander-schneider marked this pull request as ready for review April 29, 2026 05:40
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