Repository navigation
forward-test: use SEC_USER_AGENT secret, retry throttled GDELT names, bound run time - #3
Merged
Merged
Conversation
Repository variables are printed unmasked in public Actions logs; secrets are masked. Falls back to the variable, then the default. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01HTqzkhjyuhZZun8Az1tW6C
The first live run took 22 minutes and 6 of 20 names (NVDA, AMZN, BAC, BA, GM, COIN) got GDELT 429s after every retry, leaving silent coverage gaps. Failed names now get one retry pass after a cooldown with a longer pause; a 40-minute budget stops new requests and records skipped names; progress prints per name; the job has a 60-minute timeout. The report lists all 20 pre-registered names and a per-name news-fetch-failure count. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01HTqzkhjyuhZZun8Az1tW6C
Deploying aetherius with
|
| Latest commit: |
0fe1c8e
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://fa347027.aetherius-7cp.pages.dev |
| Branch Preview URL: | https://claude-laughing-gates-gmc4fm.aetherius-7cp.pages.dev |
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
What changed?
forward-test.ymlreadsSEC_USER_AGENTfromsecrets.SEC_USER_AGENTfirst, then the repository variable, then the default.run_forward_test.py):summarize.py): every one of the 20 pre-registered names now gets a row, and a new News fetch failures column counts the runs with no GDELT coverage for each name.Why was this needed?
Validation
Relevant tests pass (
pytest -q aetherius/tests): 121 passed (was 116). New tests cover:Retry timing is tested with a fake clock, so no test sleeps.
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.I regenerated the report from the real first-run log: all 20 names are listed, and the 6 throttled names show 1 failure each.
Scope
aetherius/app/services/...)simulations/backtest/...)Forward test only (
simulations/forward_test/,.github/workflows/forward-test.yml).Safety Checklist
.env, API keys, tokens)simulations/artifacts/, runtime logs)Notes for Reviewers
SEC_USER_AGENTsecret is already set. Delete any repository variable with the same name.labels.json.🤖 Generated with Claude Code
https://claude.ai/code/session_01HTqzkhjyuhZZun8Az1tW6C