Skip to content

Commit f58f4ff

Browse files
committed
fill_prof_email: cleaner hardware/platform wording (no 'OpenMM OpenMM')
Template said 'Hardware: ... (OpenMM {platform_hint} backend)' which produced 'OpenMM CPU (OpenMM 8.5.1, no Metal in build) backend' when the user passed --platform with 'OpenMM' in it. Splitting into two clean lines: 'Hardware: ...' and 'OpenMM platform: ...'. Test updated to match. 10/10 still pass.
1 parent d447827 commit f58f4ff

2 files changed

Lines changed: 3 additions & 2 deletions

File tree

scripts/fill_prof_email.py

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -47,7 +47,8 @@
4747
4848
== What ran ==
4949
50-
- Hardware: {hardware_hint} (OpenMM {platform_hint} backend)
50+
- Hardware: {hardware_hint}
51+
- OpenMM platform: {platform_hint}
5152
- Force field: OpenFF Sage 2.1.0 (small molecules) + AM1-BCC partial
5253
charges + TIP3P water — no learned surrogate at any layer
5354
- MD: openmmtools.alchemy + GHMC integrator (Metropolised Langevin),

tests/fep/test_fill_prof_email_smoke.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -246,7 +246,7 @@ def test_hydration_hardware_override_lands_in_body():
246246
assert "Henry's MacBook Pro M-series, 16 GB RAM" in out, (
247247
"hydration body must use --hardware override; got:\n"
248248
f"{out[:800]}")
249-
assert "OpenMM CPU (Reference/CPU only; no Metal) backend" in out, (
249+
assert "OpenMM platform: CPU (Reference/CPU only; no Metal)" in out, (
250250
"hydration body must use --platform override; got:\n"
251251
f"{out[:800]}")
252252
# The old M5 Max hardcoding must be gone for the hydration path.

0 commit comments

Comments
 (0)