Skip to content

Conversation

@lohedges
Copy link
Contributor

@lohedges lohedges commented Dec 3, 2025

This PR closes #484 by updating the default GROMACS NPT configuration to match the Recursion sandpit. The updated defaults satisfy the constraint on tau-p, which is now strictly enforced by gmx grompp.

  • I confirm that I have merged the latest version of devel into this branch before issuing this pull request (e.g. by running git pull origin devel): [y]
  • I confirm that I have permission to release this code under the GPL3 license: [y]

@lohedges lohedges added the bug Something isn't working label Dec 3, 2025
@lohedges lohedges temporarily deployed to biosimspace-build December 3, 2025 19:00 — with GitHub Actions Inactive
lohedges added a commit that referenced this pull request Dec 3, 2025
@lohedges
Copy link
Contributor Author

lohedges commented Dec 3, 2025

Well, even the sandpit settings don't work, so there we go.

@lohedges lohedges temporarily deployed to biosimspace-build December 3, 2025 20:25 — with GitHub Actions Inactive
@lohedges lohedges temporarily deployed to biosimspace-build December 3, 2025 20:25 — with GitHub Actions Inactive
@lohedges lohedges temporarily deployed to biosimspace-build December 3, 2025 20:25 — with GitHub Actions Inactive
@lohedges lohedges temporarily deployed to biosimspace-build December 3, 2025 20:25 — with GitHub Actions Inactive
@lohedges lohedges merged commit cc56ae7 into devel Dec 3, 2025
4 of 5 checks passed
@lohedges lohedges deleted the fix_484 branch December 3, 2025 21:01
lohedges added a commit that referenced this pull request Dec 3, 2025
lohedges added a commit that referenced this pull request Dec 3, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[BUG] Latest gmx grommp raises exception if tau-p isn't at least 25x dt

2 participants