Skip to content

merge main into PTP_TermCache#64

Merged
ajpal merged 42 commits into
PTP_TermCachefrom
PTP_TermCache_merge_main
May 19, 2026
Merged

merge main into PTP_TermCache#64
ajpal merged 42 commits into
PTP_TermCachefrom
PTP_TermCache_merge_main

Conversation

@ajpal

@ajpal ajpal commented May 19, 2026

Copy link
Copy Markdown
Owner

No description provided.

FTRobbin and others added 30 commits April 9, 2026 14:10
Extract toolchain update + benchmarks clone into infra/setup.sh, and have
nightly.sh skip that setup when run by the combined-nightly orchestrator
(which sets POACH_NIGHTLY_COMBINED=1 and supplies POACH_BENCHMARKS_DIR).
Standalone behavior is unchanged.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
When the combined-nightly orchestrator runs this branch in a worktree, it
passes POACH_BENCHMARKS_DIR pointing at a shared clone above the worktree
(so we don't re-clone per branch). The benchmark_root.relative_to(REPO_ROOT)
call then raises ValueError. Add display_path() that falls back to the
absolute path when the input isn't a subpath of REPO_ROOT.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Emit benchmark_path relative to the suite's train/ subdir (matching the
serialize branch's convention) so the combined-nightly comparison table
can join rows by benchmark_path without the orchestrator having to strip
the prefix.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Match the serialize branch exactly so the combined-nightly merger can join
rows by benchmark_path with no normalization.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Serialize emits `<name>.egg`, not `<name>` — the previous edit over-corrected.
Drop the .with_suffix("") so vanilla and serialize both emit `<name>.egg`
and the merger joins rows correctly.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@ajpal ajpal merged commit 656dad2 into PTP_TermCache May 19, 2026
1 check passed
@ajpal ajpal deleted the PTP_TermCache_merge_main branch May 19, 2026 00:23
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