Skip to content

Conversation

rfhaque
Copy link
Collaborator

@rfhaque rfhaque commented Sep 5, 2025

This PR adds the Sparta application (docs: Adding a Benchmark)

  • Not using develop, source code: Sparta build for AMD GPUs sparta/sparta#573
  • Adding repo/sparta-snl/package.py to specify build
  • Adding repo/sparta-snl/application.py to specify runtime parameters
  • Adding experiments/sparta-snl/experiment.py to specify the experiment to run
  • Sparta dry run is passing
  • Add Sparta+openmp and Sparta+rocm to CI on Dane and Tuo or Tioga
  • Update the package to meet Spack 1.0 requirements @scheibelp

@github-actions github-actions bot added experiment New or modified experiment application labels Sep 5, 2025
@rfhaque rfhaque requested a review from pearce8 September 5, 2025 20:38
@pearce8 pearce8 marked this pull request as ready for review September 6, 2025 15:07
@pearce8 pearce8 added the changes requested Changes requested label Sep 10, 2025
Copy link
Collaborator

@pearce8 pearce8 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  • Sparta dry run should be passing
  • Add Sparta+openmp and Sparta+rocm to CI on Dane and Tuo or Tioga
  • Since this will likely go in after the Spack 1.0 support, please update the package to meet Spack 1.0 requirements.

Copy link
Collaborator

@scheibelp scheibelp left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

(this looks compatible with #953)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
application changes requested Changes requested experiment New or modified experiment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants