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

Log warmup overhead time #339

Merged
merged 1 commit into from
Feb 23, 2024
Merged

Log warmup overhead time #339

merged 1 commit into from
Feb 23, 2024

Conversation

adarshyoga
Copy link
Contributor

@adarshyoga adarshyoga commented Feb 21, 2024

  • Have you provided a meaningful PR description?

Adds an entry to the output log called warmup overhead time which is the difference of the median execution time and warm up time. This ovehead time points to the time spent in jit-compilation (as in the case of numba-dpex) or framework load time (as in the case of dpnp, numpy). Sample output below.

Fixes #336 .


================ implementation numba_dpex_k ========================
implementation: numba_dpex_k
framework: numba_dpex
framework version: 0.22.0.dev2+3.g59d523892
input size: 20971520
setup time: 372.803065ms (372803065 ns)
warmup time: 821.023662ms (821023662 ns)
teardown time: 4.727934ms (4727934 ns)
max execution times: 2.488874ms (2488874 ns)
min execution times: 2.062158ms (2062158 ns)
median execution times: 2.091041ms (2091041 ns)
warmup overhead time (warmup time - median execution time): 818.932621ms (818932621 ns)
repeats: 10
preset: S
validated: Success

  • Have you added a test, reproducer or referred to an issue with a reproducer?
  • Have you tested your changes locally for CPU and GPU devices?
  • Have you made sure that new changes do not introduce compiler warnings?
  • If this PR is a work in progress, are you filing the PR as a draft?

@adarshyoga adarshyoga self-assigned this Feb 21, 2024
@adarshyoga adarshyoga requested a review from ZzEeKkAa February 21, 2024 08:31
@adarshyoga adarshyoga force-pushed the feature/log_compilation_ime branch from 467f528 to e77c8ae Compare February 21, 2024 17:12
@adarshyoga adarshyoga marked this pull request as draft February 21, 2024 17:30
@adarshyoga adarshyoga force-pushed the feature/log_compilation_ime branch 2 times, most recently from 4f73783 to e54dfd7 Compare February 21, 2024 17:52
@adarshyoga adarshyoga marked this pull request as ready for review February 21, 2024 17:53
@adarshyoga adarshyoga force-pushed the feature/log_compilation_ime branch from e54dfd7 to 374b4d1 Compare February 21, 2024 20:20
@diptorupd diptorupd merged commit 66e2931 into main Feb 23, 2024
45 checks passed
@diptorupd diptorupd deleted the feature/log_compilation_ime branch February 23, 2024 19:08
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.

Adding JIT compilation time of benchmarks to dpbench report
2 participants