Skip to content

[Helion Benchmark] Use geomean for speedup - #7126

Merged
oulgen merged 1 commit into
mainfrom
oulgen/stack/6
Sep 8, 2025
Merged

[Helion Benchmark] Use geomean for speedup#7126
oulgen merged 1 commit into
mainfrom
oulgen/stack/6

Conversation

@oulgen

@oulgen oulgen commented Sep 8, 2025 •
edited
Loading

Copy link
Copy Markdown
Contributor

@meta-cla meta-cla Bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Sep 8, 2025
oulgen added a commit that referenced this pull request Sep 8, 2025
@oulgen
[Helion Benchmark] Use geomean for speedup
b6268b2 
stack-info: PR: #7126, branch: oulgen/stack/6
@oulgen oulgen mentioned this pull request Sep 8, 2025
Merged
@vercel

vercel Bot commented Sep 8, 2025 •
edited
Loading

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Updated (UTC)
torchci Ready Ready Preview Sep 8, 2025 10:29pm

This was referenced Sep 8, 2025
Merged
Merged
@oulgen
oulgen changed the base branch from oulgen/stack/5 to main September 8, 2025 19:54
oulgen added a commit that referenced this pull request Sep 8, 2025
@oulgen
[Helion Benchmark] Use geomean for speedup
81af8f5 
stack-info: PR: #7126, branch: oulgen/stack/6
@oulgen
oulgen changed the base branch from main to oulgen/stack/5 September 8, 2025 19:54
malfet pushed a commit that referenced this pull request Sep 8, 2025
@oulgen
[Helion Benchmark] Make accuracy pass/fail (#7110)
6eb7084 
Stacked PRs:
 * #7126
 * #7123
 * #7121
 * __->__#7110


--- --- ---

### [Helion Benchmark] Make accuracy pass/fail
malfet
malfet approved these changes Sep 8, 2025
View reviewed changes

@malfet malfet left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

It compiles - Ship It!

@oulgen
oulgen changed the base branch from oulgen/stack/5 to main September 8, 2025 22:22
@oulgen
oulgen changed the base branch from main to oulgen/stack/5 September 8, 2025 22:22
This was referenced Sep 8, 2025
Merged
Merged
oulgen added a commit that referenced this pull request Sep 8, 2025
@oulgen
[Helion Benchmark] Dont display speedup when accuracy has failed (#7121)
e3c9c84 
Stacked PRs:
 * #7129
 * #7128
 * #7126
 * #7123
 * __->__#7121


--- --- ---

### [Helion Benchmark] Dont display speedup when accuracy has failed
@oulgen
oulgen changed the base branch from oulgen/stack/5 to main September 8, 2025 22:24
@oulgen
oulgen changed the base branch from main to oulgen/stack/5 September 8, 2025 22:24
oulgen added a commit that referenced this pull request Sep 8, 2025
@oulgen
[Helion Benchmark] Default to single view (#7123)
1c37877 
Stacked PRs:
 * #7129
 * #7128
 * #7126
 * __->__#7123
 * #7121


--- --- ---

### [Helion Benchmark] Default to single view
Base automatically changed from oulgen/stack/5 to main September 8, 2025 22:24
@oulgen
[Helion Benchmark] Use geomean for speedup
a6de730 
stack-info: PR: #7126, branch: oulgen/stack/6
@oulgen
oulgen merged commit 493670f into main Sep 8, 2025
5 checks passed
@oulgen
oulgen deleted the oulgen/stack/6 branch September 8, 2025 22:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@malfet malfet malfet approved these changes
@yangw-dev yangw-dev Awaiting requested review from yangw-dev
@huydhn huydhn Awaiting requested review from huydhn

Assignees

No one assigned

Labels

CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@oulgen @malfet