Add Google Finance mirror + task verifiers (site by @QiushiSun, verifiers by reviewer) - #76
Open
reacher-z wants to merge 3 commits into
Open
Add Google Finance mirror + task verifiers (site by @QiushiSun, verifiers by reviewer)#76reacher-z wants to merge 3 commits into
reacher-z wants to merge 3 commits into
Conversation
Mirrors https://www.google.com/finance/ as site 17 on port 40016. Registers the site in the three required places (websyn_start.sh SITES, control_server.py SITES, Dockerfile EXPOSE 40000-40016) and pins MarkupSafe==3.0.3, which app.py imports directly for its server-rendered SVG charts. Covers the home page with seven region tabs, quote pages for stocks, ETFs, indexes, sector indexes, crypto, FX and futures across eight chart ranges and five tabs, eight market boards, scored search with autocomplete, news, a currency converter, a compare view, and auth-gated watchlists and portfolios. 10,275 seeded rows across 18 models; market data is synthetic and frozen at 2026-07-24, company identities and logos are real. 20 benchmark tasks in sites/google_finance/tasks.jsonl, contributor keys only. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
fetch_assets.sh otherwise resolves a revision that predates the paired assets PR, and websyn_start.sh copies instance_seed to instance for every site under set -e — so a missing directory aborts the whole entrypoint and no site comes up, not just this one. Verified from a clean clone of this branch: fetch_assets.sh, build.sh, and a container on 41000-41016 give /health ok with 17/17 alive and every port 200, with the seed DB byte-identical after POST /reset/google_finance. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
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
2 participants
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.
Reviewer PR for the Google Finance mirror (site by @QiushiSun, original PR #73). The paired assets are in Hugging Face PR #47.
This adds the reviewer-owned grading contract:
sites/google_finance/verify/Bank basketportfolio and JPM lotverifier_path+judge_rubricon every task, with noanswerkeyValidation performed against commit
a1879e9and HFrefs/pr/47:reset-allreturned all sites ready and Google Finance live DB was byte-identical to the seedBank basket, cash5000, one JPM lot of 25 shares at300; page showed position market value$9,203.00and gain+22.71%eval_judge.py --verifier Trueintegration passedflake8,py_compile, andgit diff --checkpass