Skip to content

[Benchmark] Add initial TritonBench integration and vector_add benchmark example #242

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 16 commits into
base: gh/yf225/5/base
Choose a base branch
from

Conversation

yf225
Copy link
Contributor

@yf225 yf225 commented Jul 7, 2025

Stack from ghstack (oldest at bottom):

Support running Helion benchmark via a single-line command:
python benchmark/run.py --metrics speedup,accuracy --kernel vector_add

Thanks @xuzhao9 for help with authoring pytorch-labs/tritonbench@09b42e8 and also reviewing pytorch-labs/tritonbench@2237731, which are dependencies for this PR.

[ghstack-poisoned]
yf225 added a commit that referenced this pull request Jul 7, 2025
@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Meta Open Source bot. label Jul 7, 2025
@yf225 yf225 requested review from jansel and oulgen and removed request for jansel and oulgen July 7, 2025 22:45
[ghstack-poisoned]
yf225 added a commit that referenced this pull request Jul 7, 2025
[ghstack-poisoned]
yf225 added a commit that referenced this pull request Jul 7, 2025
yf225 added a commit that referenced this pull request Jul 7, 2025
yf225 added a commit that referenced this pull request Jul 7, 2025
yf225 added a commit that referenced this pull request Jul 7, 2025
yf225 added a commit that referenced this pull request Jul 7, 2025
[ghstack-poisoned]
yf225 added a commit that referenced this pull request Jul 7, 2025
[ghstack-poisoned]
yf225 added a commit that referenced this pull request Jul 7, 2025
[ghstack-poisoned]
yf225 added a commit that referenced this pull request Jul 7, 2025
[ghstack-poisoned]
yf225 added a commit that referenced this pull request Jul 7, 2025
[ghstack-poisoned]
yf225 added a commit that referenced this pull request Jul 7, 2025
yf225 added a commit that referenced this pull request Jul 7, 2025
[ghstack-poisoned]
yf225 added a commit that referenced this pull request Jul 8, 2025
[ghstack-poisoned]
yf225 added a commit that referenced this pull request Jul 8, 2025
benchmark/run.py Outdated
Comment on lines 61 to 63
# Add to path
if tritonbench_path not in sys.path:
sys.path.insert(0, tritonbench_path)
Copy link
Contributor

Choose a reason for hiding this comment

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

Modifying the path is a bit hacky. Could we do pip install git+https://github.com/pytorch-labs/tritonbench.git or (cd tritonbench && pip install -e .)? (If the git one works we wont need a submodule.)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I found that tritonbench needs custom installation of dependencies via python install.py which is not run when doing pip install. Currently I removed the submodule but do a git clone within the run.py file so that we can cd into the folder to be able to run install.py (but would be curious for other cleaner approaches too)

[ghstack-poisoned]
yf225 added a commit that referenced this pull request Jul 8, 2025
[ghstack-poisoned]
yf225 added a commit that referenced this pull request Jul 8, 2025
[ghstack-poisoned]
yf225 added a commit that referenced this pull request Jul 8, 2025
[ghstack-poisoned]
yf225 added a commit that referenced this pull request Jul 8, 2025
[ghstack-poisoned]
yf225 added a commit that referenced this pull request Jul 8, 2025
@yf225 yf225 requested a review from jansel July 8, 2025 06:10
[ghstack-poisoned]
yf225 added a commit that referenced this pull request Jul 8, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA Signed This label is managed by the Meta Open Source bot.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants