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

Add Sage 2.1.0 benchmark #9

Merged
merged 4 commits into from
Nov 13, 2024
Merged

Add Sage 2.1.0 benchmark #9

merged 4 commits into from
Nov 13, 2024

Conversation

ntBre
Copy link
Collaborator

@ntBre ntBre commented Nov 7, 2024

No description provided.

@ntBre
Copy link
Collaborator Author

ntBre commented Nov 7, 2024

/run-benchmark submissions/2024-11-07-Sage-2.1.0/yds.yaml

Copy link

github-actions bot commented Nov 7, 2024

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

  • Run ID: 11729983094
  • Triggering actor: github-actions[bot]
  • Target branch: sage-2.1.0

@@ -0,0 +1,3 @@
forcefield: openff-2.1.0.offxml
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
forcefield: openff-2.1.0.offxml
forcefield: openff_unconstrained-2.1.0.offxml

See recent discussion on Slack - assuming you want to use the unconstrained version, that is

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Thanks, I think you're right. I'm planning just to let this one run to make sure I didn't break anything with the last PR (I should probably set up some actual test CI soon) and then I'll open another PR with the unconstrained version.

Copy link

github-actions bot commented Nov 8, 2024

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

@ntBre
Copy link
Collaborator Author

ntBre commented Nov 13, 2024

Just rebased onto master to get the new plotting script.

@ntBre
Copy link
Collaborator Author

ntBre commented Nov 13, 2024

The record has been published to Zenodo with DOI: https://doi.org/10.5281/zenodo.14053221

Please add this to the main README table before merging.

@ntBre ntBre merged commit 5129429 into master Nov 13, 2024
@ntBre ntBre deleted the sage-2.1.0 branch November 13, 2024 19:31
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