Skip to content

Lower pin on openmmforcefields environment - #2177

Open
IAlibay wants to merge 1 commit into
mainfrom
update-lower-pin-openmmforcefields
Open

Lower pin on openmmforcefields environment#2177
IAlibay wants to merge 1 commit into
mainfrom
update-lower-pin-openmmforcefields

Conversation

@IAlibay

@IAlibay IAlibay commented Aug 29, 2026

Copy link
Copy Markdown
Member

LLM / AI generated code disclosure

LLMs or other AI-powered tools (beyond simple IDE use cases) were used in this contribution: no
If yes, please provide details here:

Checklist

  • All new code is appropriately documented (user-facing code must have complete docstrings).
  • Added a news entry, or the changes are not user-facing.
  • Ran pre-commit: you can run pre-commit locally or comment on this PR with pre-commit.ci autofix.
  • Filled in the AI generated code disclosure.

Manual Tests: these are slow so don't need to be run every commit, only before merging and when relevant changes are made (generally at reviewer-discretion).

Developers certificate of origin

@IAlibay

IAlibay commented Aug 29, 2026

Copy link
Copy Markdown
Member Author

Ah that's annoying - I didn't remember openmmforcefields v0.16 needed openmm 8.5+.

We might need to consider reverting OpenFreeEnergy/gufe#831

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.

1 participant