Skip to content

forward-test: pace GDELT requests 20 s apart - #7

Merged
zariffromlatif merged 1 commit into
mainfrom
claude/laughing-gates-gmc4fm
Oct 18, 2026
Merged

zariffromlatif merged 1 commit into
mainfrom
claude/laughing-gates-gmc4fm

Conversation

@zariffromlatif

Copy link
Copy Markdown
Owner

Summary

  • What changed? In simulations/forward_test/run_forward_test.py, GDELT requests in the first pass now wait 20 s between companies (was 6 s), and the retry pass waits 30 s (was 15 s). The 40-minute budget and the 60-minute job timeout are unchanged. The forward-test README explains why.

  • Why was this needed? Both live runs so far lost 6 of 20 names even after the retry pass: GDELT throttles GitHub's shared runners.

    • 2026-10-16: NVDA, AMZN, BAC, BA, GM, COIN.
    • 2026-10-17: GOOGL, JPM, BAC, BA (429), MSFT (timeout), DIS (skipped when the budget ran out).

    Retrying harder doesn't help; spacing requests out should. At about 20 s each, 20 names take roughly 7 minutes.

Validation

  • Relevant tests pass (pytest -q aetherius/tests): 122 passed. The retry and budget tests use a fake clock and still hold at the new pacing.
  • SVB-2023 backtest still passes if scoring / mapping / ingestion changed: not applicable. The frozen scorer is untouched.
  • Lint passes (ruff check aetherius simulations scripts).
  • Docs updated: simulations/forward_test/README.md.

Worst case: if every request is refused, the budget stops new requests at 40 minutes. The last in-flight request can take up to about 3 minutes with its internal retries, plus about 1 minute for EDGAR, so the run still ends under the 60-minute job timeout.

Scope

  • Ingestion, entity mapping, scoring, or signals (aetherius/app/services/...)
  • Delivery / templates / review workflow
  • Backtests (simulations/backtest/...)
  • Docs / governance / landing page only

Forward test only.

Safety Checklist

  • No secrets committed (.env, API keys, tokens)
  • No generated artifacts committed (simulations/artifacts/, runtime logs)
  • No client-specific / private data included
  • Marketing / README claims still match implementation reality

Notes for Reviewers

  • The report's News fetch failures column will show whether this helps over the next few runs.

Generated by Claude Code

Both live runs at a 6 s pace lost 6 of 20 names to GDELT 429s (or a
timeout) even after the retry pass. Space first-pass requests 20 s apart
and retry-pass requests 30 s apart; about 7 minutes for 20 names, inside
the existing 40-minute budget and 60-minute job timeout.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01HTqzkhjyuhZZun8Az1tW6C
Copilot AI balanced review requested due to automatic review settings October 18, 2026 04:03
@cloudflare-workers-and-pages

Copy link
Copy Markdown

Deploying aetherius with  Cloudflare Pages  Cloudflare Pages

Latest commit: 9343b16
Status: ✅  Deploy successful!
Preview URL: https://a38e86be.aetherius-7cp.pages.dev
Branch Preview URL: https://claude-laughing-gates-gmc4fm.aetherius-7cp.pages.dev

View logs

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@zariffromlatif
zariffromlatif merged commit 70991c0 into main Oct 18, 2026
2 checks passed
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.

3 participants