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

Workflow template #571

Closed
wants to merge 14 commits into from
Closed

Workflow template #571

wants to merge 14 commits into from

Conversation

rfhaque
Copy link
Collaborator

@rfhaque rfhaque commented Jan 24, 2025

Description

  • Replace with: A short description of the change, including motivation and context.
  • Replace with: A list of any dependencies.
  • Replace with: Link(s) to relevant issue(s)
  • Complete the checklist for a relevant section(s) below
  • Delete sections below that are not relevant to this PR

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

  • Add/modify systems/system_name/Create a new directory for the system, add/modify system.py` file
  • Add/modify a dry run unit test in .github/workflows/run.yml
  • Add/modify `systems/all_system_definitions/system-hardware/system_definition.yaml which will appear in the docs catalogue

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

  • (optional) If package upstreamed to Spack is insufficient, add/modify repo/benchmark_name/package.py
  • (optional) If application upstreamed to Ramble is insufficient, add/modify repo/benchmark_name/application.py
  • Tags in Ramble's application.py or in repo/benchmark_name/application.py will appear in the docs catalogue
  • Add/modify an experiments/benchmark_name/experiment.py to define a single node and multi-node experiments
  • Add/modify a dry run unit test in .github/workflows/run.yml

Adding/modifying core functionality, CI, or documentation:

  • Update docs
  • Update .github/workflows and .gitlab/ci unit tests (if needed)

@github-actions github-actions bot added the ci CI, unit tests, GitHub actions label Jan 24, 2025
@rfhaque
Copy link
Collaborator Author

rfhaque commented Jan 26, 2025

Moving changes to #560

@rfhaque rfhaque closed this Jan 26, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci CI, unit tests, GitHub actions
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant