-
Notifications
You must be signed in to change notification settings - Fork 25.6k
Add benchmark for performance comparison #43221
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
Conversation
[ghstack-poisoned]
💊 CI failures summary and remediationsAs of commit 0ca3f12 (more details on the Dr. CI page):
1 failure confirmed as flaky and can be ignored:
ci.pytorch.org: 1 failedThis comment was automatically generated by Dr. CI (expand for details).Follow this link to opt-out of these comments for your Pull Requests.Please report bugs/suggestions on the GitHub issue tracker or post in the (internal) Dr. CI Users group. This comment has been revised 16 times. |
for (size_t batch_size = 1; batch_size < 5; ++batch_size) { | ||
for (size_t width = 32; width < 256; width *= 2) { | ||
b->Args({input_channels, output_channels, kernel, batch_size, width, true}); | ||
b->Args({input_channels, output_channels, kernel, batch_size, width, false}); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think you want two benchmark, no? Otherwise how are you distinguish the reported results between optimized vs unoptimized? Looking at the values reported by benchmark? I think that is fine as well.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
So currently if we look at two consecutive lines, we can see the results for the two cases: optimized (Optimzed:1) and unoptimized(Optimzed:0). I think that way we can easily compare the performance for a given set of hyperparameter. Let me know if that's ok.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sure, thats ok.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good. Can you add the benchmark results here?
This comment has been minimized.
This comment has been minimized.
Results are looking good. I am wondering if you can summarize the average improvement. |
Average Improvement: 85% Example Run:
|
I mean more side-to-side comparison would have been nice. So that you can just show what is the % improvement. |
Differential Revision: [D23197567](https://our.internmc.facebook.com/intern/diff/D23197567) [ghstack-poisoned]
Differential Revision: [D23197567](https://our.internmc.facebook.com/intern/diff/D23197567) [ghstack-poisoned]
Differential Revision: [D23197567](https://our.internmc.facebook.com/intern/diff/D23197567) [ghstack-poisoned]
Differential Revision: [D23197567](https://our.internmc.facebook.com/intern/diff/D23197567) [ghstack-poisoned]
@kimishpatel merged this pull request in da03625. |
Stack from ghstack:
Differential Revision: D23197567