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

Adding lipid aux and core datasets for benchmarking #28

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

JHoeflich1
Copy link

@JHoeflich1 JHoeflich1 commented Dec 19, 2024

Submission Checklist

  • Created a new directory in the submissions directory containing the YDS input file and optionally a force field .offxml file
  • Triggered the benchmark workflow with a PR comment of the form /run-benchmark path/to/submission
  • Waited for the workflow to finish and a comment with Job status: success to be posted
  • Reviewed the results committed by the workflow
  • Published the corresponding Zenodo entry and retrieved the DOI
  • Added the Zenodo DOI to the table in the main README
  • Ready to merge!

Copy link
Collaborator

@ntBre ntBre left a comment

Choose a reason for hiding this comment

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

Looks good to me! I'll start one of them, and if it starts running okay, I'll kick off the second one too. You should see a couple of comments from the bot indicating that the jobs started and eventually finished, and then there will be new commits adding the results!

@ntBre
Copy link
Collaborator

ntBre commented Dec 19, 2024

/run-optimization-benchmarks submissions/2024-12-19-Sage-2.2.1-lipids-aux/input.yaml

Copy link

A workflow has been dispatched to run the benchmarks for this PR.

  • Run ID: 12418830094
  • Triggering actor: github-actions[bot]
  • Target branch: master

Copy link

A workflow dispatched to run optimization benchmarks for this PR has just finished.

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.

2 participants