OpenFreeEnergy / OpenFreeEnergy/openfe

Should standard state correction be 0 or None when there's no correction?

Open
#1,875 0 comments 0 reactions 3 assignees View on GitHub

@jthorton is already working on this.

Since Mar 17, 2026.

Dominant language
Python
Stars
331
Forks
56
Avg merge
3d 9h
Merged PRs (30d)
13

Description

Prior to v1.9, we didn't include standard_state_correction in the results dictionary when there were no restraints being applied (e.g. when we were doing a solvent or vacuum leg). However, in v1.9 I naively changed this to always have the key in the results dictionary and having 0 kcal/mol as the correction when no correction is needed.

Disregarding the unintended API break, there is a question on whether or not we should revert this change or at least set the standard correction to None when it's not needed.

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.