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

[query] Benchmark configurations #14762

Open
wants to merge 1 commit into
base: ehigham/benchmark-in-batch-variability
Choose a base branch
from

Conversation

ehigham
Copy link
Member

@ehigham ehigham commented Dec 5, 2024

Change Description

Added benchmark parameters to all benchmark tests in the Hail codebase. Each test now specifies:

  • Minimum detectable slowdown (mds)
  • Number of instances (ie number of batch jobs)
  • Number of iterations (how many timed trials of the benchmark per instance)
  • Number of burn-in iterations (number of untimed trials run before timing)

These parameters allow for more precise and controlled benchmark measurements by explicitly defining the statistical requirements and execution parameters for each test.

Security Assessment

This change has no security impact

Impact Description

Low-level test configuration changes only - modifies benchmark test parameters without changing any functional code or test logic.

@ehigham ehigham force-pushed the ehigham/benchmark-suite-improvements branch from 54cd48a to 93e81d9 Compare December 5, 2024 21:16
@ehigham ehigham force-pushed the ehigham/benchmark-configurations branch from fa2e947 to 8b98a0c Compare December 5, 2024 21:16
@ehigham ehigham force-pushed the ehigham/benchmark-suite-improvements branch from 93e81d9 to 2344aed Compare December 5, 2024 21:19
@ehigham ehigham force-pushed the ehigham/benchmark-configurations branch from 8b98a0c to 692247b Compare December 5, 2024 21:20
@ehigham ehigham force-pushed the ehigham/benchmark-suite-improvements branch from 2344aed to e8cb1fb Compare December 5, 2024 21:42
@ehigham ehigham force-pushed the ehigham/benchmark-configurations branch from 692247b to e638c1c Compare December 5, 2024 21:42
@ehigham ehigham force-pushed the ehigham/benchmark-suite-improvements branch from e8cb1fb to c4a8c24 Compare December 5, 2024 21:52
@ehigham ehigham force-pushed the ehigham/benchmark-configurations branch from e638c1c to 8bad700 Compare December 5, 2024 21:53
@ehigham ehigham force-pushed the ehigham/benchmark-suite-improvements branch from c4a8c24 to 687437c Compare December 5, 2024 22:00
@ehigham ehigham force-pushed the ehigham/benchmark-configurations branch from 8bad700 to 69cfc52 Compare December 5, 2024 22:00
@ehigham ehigham changed the base branch from ehigham/benchmark-suite-improvements to ehigham/benchmark-in-batch-variability December 5, 2024 22:01
@ehigham ehigham force-pushed the ehigham/benchmark-in-batch-variability branch from 2c72cb5 to 30d0a86 Compare December 5, 2024 22:06
@ehigham ehigham force-pushed the ehigham/benchmark-configurations branch from 69cfc52 to f5b2196 Compare December 5, 2024 22:06
@ehigham ehigham force-pushed the ehigham/benchmark-in-batch-variability branch from 30d0a86 to e32bc09 Compare December 5, 2024 22:12
@ehigham ehigham force-pushed the ehigham/benchmark-configurations branch from f5b2196 to 0d886d5 Compare December 5, 2024 22:13
@ehigham ehigham force-pushed the ehigham/benchmark-in-batch-variability branch from e32bc09 to 97a40fa Compare December 6, 2024 23:13
@ehigham ehigham force-pushed the ehigham/benchmark-configurations branch from 0d886d5 to 432a727 Compare December 6, 2024 23:13
@ehigham ehigham marked this pull request as ready for review December 6, 2024 23:14
@ehigham ehigham force-pushed the ehigham/benchmark-in-batch-variability branch from 97a40fa to 1118f03 Compare December 12, 2024 21:52
@ehigham ehigham force-pushed the ehigham/benchmark-configurations branch from 432a727 to b1f3066 Compare December 12, 2024 21:52
@ehigham ehigham force-pushed the ehigham/benchmark-in-batch-variability branch from 1118f03 to f79f8c1 Compare December 12, 2024 22:01
@ehigham ehigham force-pushed the ehigham/benchmark-configurations branch from b1f3066 to 2299a17 Compare December 12, 2024 22:01
@ehigham ehigham force-pushed the ehigham/benchmark-in-batch-variability branch from f79f8c1 to af0036a Compare December 12, 2024 22:03
@ehigham ehigham force-pushed the ehigham/benchmark-configurations branch from 2299a17 to a128b0f Compare December 12, 2024 22:03
@ehigham ehigham force-pushed the ehigham/benchmark-in-batch-variability branch from af0036a to e61f806 Compare December 16, 2024 18:50
@ehigham ehigham force-pushed the ehigham/benchmark-configurations branch from a128b0f to db53a7e Compare December 16, 2024 18:50
@ehigham ehigham force-pushed the ehigham/benchmark-in-batch-variability branch from 2214856 to 573cf80 Compare January 31, 2025 21:22
@ehigham ehigham force-pushed the ehigham/benchmark-configurations branch from 2541f2e to d8c14b9 Compare January 31, 2025 21:22
@ehigham ehigham force-pushed the ehigham/benchmark-in-batch-variability branch from 573cf80 to ff1c0f3 Compare February 3, 2025 17:21
@ehigham ehigham force-pushed the ehigham/benchmark-configurations branch from d8c14b9 to 877e857 Compare February 3, 2025 17:21
@ehigham ehigham force-pushed the ehigham/benchmark-in-batch-variability branch from ff1c0f3 to 61d4c21 Compare February 3, 2025 18:02
@ehigham ehigham force-pushed the ehigham/benchmark-configurations branch from 877e857 to 12e3512 Compare February 3, 2025 18:03
@ehigham ehigham force-pushed the ehigham/benchmark-in-batch-variability branch from 61d4c21 to b77e1fc Compare February 3, 2025 20:52
@ehigham ehigham force-pushed the ehigham/benchmark-configurations branch from 12e3512 to a839a42 Compare February 3, 2025 20:52
@ehigham ehigham force-pushed the ehigham/benchmark-in-batch-variability branch from b77e1fc to 199105c Compare February 3, 2025 23:07
@ehigham ehigham force-pushed the ehigham/benchmark-configurations branch from a839a42 to 3212069 Compare February 3, 2025 23:07
@ehigham ehigham force-pushed the ehigham/benchmark-in-batch-variability branch 2 times, most recently from da6a06d to 06cf763 Compare February 6, 2025 18:44
@ehigham ehigham force-pushed the ehigham/benchmark-configurations branch from 3212069 to 15cc9c8 Compare February 6, 2025 18:44
@ehigham ehigham force-pushed the ehigham/benchmark-configurations branch from 15cc9c8 to bb7327d Compare February 20, 2025 16:50
@ehigham ehigham force-pushed the ehigham/benchmark-in-batch-variability branch 2 times, most recently from 6b1cb9e to f13de5e Compare February 20, 2025 17:28
@ehigham ehigham force-pushed the ehigham/benchmark-configurations branch from bb7327d to 37ecc51 Compare February 20, 2025 17:28
@ehigham ehigham force-pushed the ehigham/benchmark-in-batch-variability branch from f13de5e to 9deea97 Compare February 27, 2025 22:17
@ehigham ehigham force-pushed the ehigham/benchmark-configurations branch from 37ecc51 to 53538d2 Compare February 27, 2025 22:17
@ehigham ehigham force-pushed the ehigham/benchmark-in-batch-variability branch from 9deea97 to ff7f6f7 Compare March 3, 2025 17:55
@ehigham ehigham force-pushed the ehigham/benchmark-configurations branch from 53538d2 to ae5600e Compare March 3, 2025 17:55
@ehigham ehigham force-pushed the ehigham/benchmark-in-batch-variability branch from ff7f6f7 to dd7255d Compare March 17, 2025 21:54
@ehigham ehigham force-pushed the ehigham/benchmark-configurations branch 2 times, most recently from d1fc6a3 to f400f4c Compare March 17, 2025 22:48
@ehigham ehigham force-pushed the ehigham/benchmark-in-batch-variability branch from dd7255d to 8c5bf90 Compare March 17, 2025 22:48
@ehigham ehigham force-pushed the ehigham/benchmark-configurations branch from f400f4c to c166142 Compare March 18, 2025 03:43
@ehigham ehigham force-pushed the ehigham/benchmark-in-batch-variability branch from 8c5bf90 to 6043648 Compare March 18, 2025 03:43
@ehigham ehigham force-pushed the ehigham/benchmark-configurations branch from c166142 to c801867 Compare March 19, 2025 19:13
@ehigham ehigham force-pushed the ehigham/benchmark-in-batch-variability branch from 6043648 to 800a3f3 Compare March 19, 2025 19:13
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.

1 participant