Skip to content
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

Update benchmark results for epochs #117

Merged
merged 1 commit into from
May 17, 2022
Merged

Update benchmark results for epochs #117

merged 1 commit into from
May 17, 2022

Conversation

psalz
Copy link
Member

@psalz psalz commented May 12, 2022

This updates the benchmark results for epochs, as we forgot to include that in #86.

Also I've switched to gpuc2 for benchmarking (using SLURM), where we can already see a reduction in noise for some benchmarks. In general the results are as expected I'd say, with epochs adding some additional cost to all graph generation benchmarks.

Intrusive graph

1
2
3

Generating large command graphs

4
5
6
7

Building CDAGS in scheduler thread (1 node)

10
11
8
9

Building CDAGS in scheduler thread (4 nodes)

12
13
14
15

Also switch from gpuc1 to gpuc2 for benchmarking
@psalz psalz requested review from PeterTh and fknorr May 12, 2022 17:09
@PeterTh
Copy link
Contributor

PeterTh commented May 12, 2022

LGTM.
One note: it would be nice to make the charts have a solid white background, currently they are unreadable on dark mode github.

@psalz psalz merged commit 63d5ffc into master May 17, 2022
@psalz psalz deleted the epoch-benchmarks branch May 17, 2022 08:37
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.

3 participants