Repository navigation
Refactor/jevduo env and wording - #16
Merged
Merged
Conversation
added 2 commits
November 6, 2026 10:19
… fallback - scripts/run_experiment.sh reads JEVDUO_EXPERIMENT_ROOT / JEVDUO_CONFIG_FILE. The pre-rename TANDEM_* names are still honored but print a deprecation warning on stderr, so a stale variable never silently falls back to the default directory. The new name wins when both are set. - tests: switch the existing cases to JEVDUO_*, add legacy-only and both-set cases. - README upgrade note and docs/CLUSTER_RUNTIME.md document the new names and the transition.
- README / AGENTS.md / docs/CLUSTER_RUNTIME.md: title and intro wording - demo: page title, header, eyebrow, player and result-table labels, console tag, build_site.py description, and the two HTTP User-Agent strings - tools/render_summary.py: page title - DEVIATIONS.md: add a naming note at the top; historical entries are kept verbatim Display text only, no behavior change. Still unchanged (follow-up PR): harness.name, the episode.json harness string, the version number and the schema $id/$ref/title.
xxxyxun
self-requested a review
November 6, 2026 02:58
xxxyxun
approved these changes
Nov 6, 2026
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
Follow-up to #15. Renames the
TANDEM_*environment variables toJEVDUO_*(with a legacy fallback) and updates the remaining brand wording from Tandem to JevDuo. Two commits:refactor(scripts): rename TANDEM_* env vars to JEVDUO_* with a legacy fallbackdocs: update brand wording from Tandem to JevDuoChanges
Env vars (the only behavior change)
scripts/run_experiment.shreadsJEVDUO_EXPERIMENT_ROOT/JEVDUO_CONFIG_FILE.TANDEM_EXPERIMENT_ROOT/TANDEM_CONFIG_FILEstill work but print a deprecation warning on stderr, so a stale variable never silently falls back to the default directory. The new name wins if both are set.Wording (display text only)
docs/CLUSTER_RUNTIME.md: title and intro.build_site.pydescription, two HTTP User-Agent strings.tools/render_summary.py: page title.DEVIATIONS.md: a naming note at the top; historical entries kept verbatim.Not in this PR
harness.name, theepisode.jsonharness string, the version number, and the schema$id/$ref/title(next PR).tandem_api_pool,tandem_jev_images,tandem_osworld21_*), historical data underdemo/data/, and the real-machine fixture intests/eval/rjob_fakes.py.Verification
ruff check src tests demo: pass.jevduo check: 17 samples pass.pytest: 659 passed, 1 skipped (the existing optional-Chromium test). Two new tests cover legacy-only and both-set env vars.node --checkon the demo JS files: pass.run_experiment.sh --dry-run: new var only → no warning; legacy var only → warning and the legacy path is used; both set → the new path is used and the legacy path is not created.Smoke: not run on the cluster. A real
run_experiment.shrun always submits--benchmark osworld21, so a mock rjob smoke would not exercise the changed code. The changed path was verified with--dry-run. The second commit is display text only.