Skip to content

Conversation

michaelmckinsey1
Copy link
Collaborator

@michaelmckinsey1 michaelmckinsey1 commented Apr 7, 2025

Description

  • Add MPI pingpong benchmark to benchpark.

  • Figure out how to pass location of pingpong directory (since private repo) on command line, instead of hard coding Waiting for repo to be public so we can change the url to point at public repo

  • Set number of nodes instead of setting total ranks per machine

  • Set required variables for benchpark analyze

    self.set_required_variables(
    n_resources="{npx}*{npy}*{npz}",
    process_problem_size="{nzx}*{nzy}*{nzz}/({npx}*{npy}*{npz})",
    total_problem_size="{nzx}*{nzy}*{nzz}",
    )

  • Depends on Use different value for placeholder instead of relying on max value #957

Adding/modifying a system (docs: Adding a System)

  • Update llnl-*/system.py with sys_sockets_per_node parameter

Adding/modifying a benchmark (docs: Adding a Benchmark)

  • Add mpi-pingpong benchmark

@michaelmckinsey1 michaelmckinsey1 changed the title Mpi-Pingpong Benchmark [WIP] Mpi-Pingpong Benchmark Apr 8, 2025
@github-actions github-actions bot added the system New or modified system config label Apr 30, 2025
@michaelmckinsey1 michaelmckinsey1 self-assigned this May 2, 2025
@michaelmckinsey1
Copy link
Collaborator Author

Running trials in the same allocation parameter (ramble) https://ramble.readthedocs.io/en/latest/workspace_config.html#experiment-repeats

@github-actions github-actions bot added the feature New feature or request label Jun 6, 2025
Stephanie Christine Lam added 2 commits July 31, 2025 09:34
@stephanielam3211 stephanielam3211 force-pushed the benchmark/mpi-pingpong branch from f3335cb to dbeae11 Compare July 31, 2025 18:11
@slabasan slabasan force-pushed the benchmark/mpi-pingpong branch from dbeae11 to dbb09cb Compare August 1, 2025 15:43
@github-actions github-actions bot added the docs Improvements or additions to documentation label Aug 8, 2025
@codecov-commenter
Copy link

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 65.08%. Comparing base (b1e3ac4) to head (e2b7332).
⚠️ Report is 2 commits behind head on develop.

Additional details and impacted files
@@           Coverage Diff            @@
##           develop     #726   +/-   ##
========================================
  Coverage    65.08%   65.08%           
========================================
  Files           44       44           
  Lines         3148     3148           
  Branches       237      237           
========================================
  Hits          2049     2049           
  Misses        1092     1092           
  Partials         7        7           
Files with missing lines Coverage Δ
systems/llnl-elcapitan/system.py 39.20% <ø> (ø)
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
application docs Improvements or additions to documentation experiment New or modified experiment feature New feature or request system New or modified system config WIP A work-in-progress not yet ready to commit
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants