Skip to content

fix: show "--" instead of "0.00s" for time metrics when all iterations fail#68

Merged
kisernl merged 1 commit intomasterfrom
noah/gh-cron-rerun-fix
Apr 3, 2026
Merged

fix: show "--" instead of "0.00s" for time metrics when all iterations fail#68
kisernl merged 1 commit intomasterfrom
noah/gh-cron-rerun-fix

Conversation

@kisernl
Copy link
Copy Markdown
Contributor

@kisernl kisernl commented Apr 3, 2026

No description provided.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 3, 2026

Storage Benchmark Results

10MB Files

# Provider Score Download Throughput Upload Status
1 Tigris 86.4 0.23s 365.1 Mbps 0.70s 10/10
2 AWS S3 82.1 0.38s 251.5 Mbps 0.48s 10/10
3 Cloudflare R2 82.0 0.49s 173.2 Mbps 0.79s 10/10

View full run · SVGs available as build artifacts

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 3, 2026

Sandbox Benchmark Results

Sequential

# Provider Score Median TTI P95 P99 Status
1 blaxel 94.4 0.46s 0.70s 0.70s 10/10
2 e2b 91.3 0.58s 1.29s 1.29s 10/10
3 daytona 88.2 0.13s 0.31s 0.31s 9/10
4 hopx 87.5 1.15s 1.41s 1.41s 10/10
5 vercel 81.0 1.79s 2.07s 2.07s 10/10
6 runloop 80.7 1.90s 1.98s 1.98s 10/10
7 modal 80.5 1.88s 2.06s 2.06s 10/10
8 codesandbox 74.5 2.32s 2.89s 2.89s 10/10
9 cloudflare 73.0 2.07s 3.65s 3.65s 10/10
10 namespace 59.3 2.34s 6.68s 6.68s 10/10
11 just-bash 0.0 0.00s 0.00s 0.00s 0/10

Staggered

# Provider Score Median TTI P95 P99 Status
1 daytona 98.1 0.11s 0.30s 0.30s 10/10
2 blaxel 93.3 0.62s 0.74s 0.74s 10/10
3 e2b 92.6 0.51s 1.08s 1.08s 10/10
4 hopx 87.1 1.11s 1.56s 1.56s 10/10
5 vercel 80.4 1.86s 2.11s 2.11s 10/10
6 namespace 76.1 2.28s 2.55s 2.55s 10/10
7 cloudflare 75.4 2.19s 2.87s 2.87s 10/10
8 codesandbox 74.4 2.44s 2.73s 2.73s 10/10
9 runloop 71.3 1.88s 4.36s 4.36s 10/10
10 modal 48.8 1.87s 25.08s 25.08s 10/10
11 just-bash 0.0 0.00s 0.00s 0.00s 0/10

Burst

# Provider Score Median TTI P95 P99 Status
1 daytona 98.5 0.13s 0.19s 0.19s 10/10
2 e2b 93.3 0.44s 1.00s 1.00s 10/10
3 blaxel 90.4 0.88s 1.08s 1.08s 10/10
4 hopx 82.7 1.72s 1.75s 1.75s 10/10
5 vercel 81.6 1.71s 2.03s 2.03s 10/10
6 modal 78.9 1.89s 2.45s 2.45s 10/10
7 runloop 73.0 1.92s 3.88s 3.88s 10/10
8 namespace 72.1 2.65s 3.00s 3.00s 10/10
9 codesandbox 69.2 2.80s 3.49s 3.49s 10/10
10 cloudflare 62.7 3.47s 4.12s 4.12s 10/10
11 just-bash 0.0 0.00s 0.00s 0.00s 0/10

View full run · SVGs available as build artifacts

Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates benchmark result renderers to avoid displaying misleading 0.00s timing values when a provider has 0 successful iterations, showing -- instead.

Changes:

  • README HTML table: render -- for Median/P95/P99 TTI when ok === 0.
  • Sandbox SVG: render -- for Median/P95/P99 and force “slow” coloring when ok === 0.
  • Storage SVG: render -- for Download/Throughput/Upload and force “slow” coloring when ok === 0.
  • Sandbox CLI table: render -- for Median/P95/P99 columns when successful === 0.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated no comments.

File Description
src/update-readme.ts Replaces TTI timing cells with -- when all iterations fail in README table output.
src/storage/generate-svg.ts Shows -- for storage metrics and adjusts speed class when all iterations fail in generated SVG.
src/sandbox/table.ts Prints -- for TTI timing columns when all iterations fail in the console table.
src/sandbox/generate-svg.ts Shows -- for TTI metrics and adjusts speed class when all iterations fail in generated SVG.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@kisernl kisernl merged commit 1c0968c into master Apr 3, 2026
20 checks passed
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