Skip to content

BENCH cone-curve T2 small (worker, 2 crates) - #22

Closed
b7r6 wants to merge 1 commit into
mainfrom
bench/cone-t2-worker
Closed

b7r6 wants to merge 1 commit into
mainfrom
bench/cone-t2-worker

Conversation

@b7r6

@b7r6 b7r6 commented Oct 2, 2026

Copy link
Copy Markdown
Owner

What and why

Fork-only CI timing probe (do not merge). A heavy, big-dependency-cone change (edit to the foundational nativelink-util crate, which invalidates a large downstream compile cone) used to measure the cold whole-PR critical path and per-lane breakdown.

How was this verified?

This PR exists to be measured by CI itself; the job durations are the artifact. Compared against its sibling timing PR under identical cache conditions.

Risk

None. Fork-only measurement branch, never merged. The source change is a two-line comment in nativelink-util/src/lib.rs.

AI assistance

An agent (Claude Opus 4.8) created the probe and ran the measurement under human direction.

…nativelink-worker/src/lib.rs

Measurement-only change for the CI-cost-vs-dependency-cone experiment.

Co-Authored-By: Claude Opus 4.8 <[email protected]>
@b7r6

b7r6 commented Oct 3, 2026

Copy link
Copy Markdown
Owner Author

Closing fork-only measurement/experiment PR; superseded by the single clean PR on ci/bazel-dev-warming (#25).

@b7r6 b7r6 closed this Oct 3, 2026
@b7r6
b7r6 deleted the bench/cone-t2-worker branch October 15, 2026 11:54
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.

1 participant