Skip to content

Add measurable 10M trace contract - #2993

Merged
ibgreen-openai merged 2 commits into
masterfrom
codex/trace-10m-contract
Aug 6, 2026
Merged

Add measurable 10M trace contract#2993
ibgreen-openai merged 2 commits into
masterfrom
codex/trace-10m-contract

Conversation

@ibgreen-openai

Copy link
Copy Markdown
Collaborator

What changed

  • define stable 250K, 1M, 4M, and 10M trace workload contracts
  • report exact source-buffer/device-limit compatibility and persistent GPU allocation metrics
  • feed trace workload, visibility, interaction-mode, and graph-encoding counters into the reusable GPU command graph inspector
  • add node and real-WebGPU regression coverage for the contract and telemetry

Why

The trace viewer can demonstrate very large workloads, but lacked a stable, measurable contract for comparing interaction cost and identifying the actual 10M-span constraint. This makes the current monolithic allocation limit explicit and provides the observability needed to evaluate chunk-native storage in the next tranche.

Impact

The experimental trace example now shows persistent allocation size, largest buffer, device-fit status, and live command-graph counters without changing rendering semantics.

Verification

  • direct full module build passed
  • focused TypeScript check passed
  • GPU trace node tests passed (13/13)
  • Chromium/WebGPU trace test passed (1/1)
  • targeted Biome formatting and git diff --check passed

The repository yarn lint fix, yarn build, and yarn test wrappers could not run locally because the dependency registry returned HTTP 403 while installing the newly referenced dev-tools packages, leaving the ocular-* commands unavailable. CI is expected to exercise the complete repository gates.

@ibgreen-openai
ibgreen-openai marked this pull request as ready for review August 6, 2026 14:20

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 9928e7b5b8

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread examples/experimental/gpu-trace-viewer/app.ts
Comment thread examples/experimental/gpu-trace-viewer/app.ts
Comment thread examples/experimental/gpu-trace-viewer/trace-benchmark.ts Outdated
@ibgreen-openai
ibgreen-openai force-pushed the codex/trace-10m-contract branch from 9928e7b to be393c7 Compare August 6, 2026 14:48
@ibgreen-openai
ibgreen-openai merged commit 7676b95 into master Aug 6, 2026
13 of 15 checks passed
@ibgreen-openai
ibgreen-openai deleted the codex/trace-10m-contract branch August 6, 2026 15:13
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.

2 participants