Skip to content

test: product-formation Ea limit split at 1e-2 M PMe3 - #31

Merged
0rkhann merged 1 commit into
mainfrom
product-ea-limit
Oct 2, 2026
Merged

0rkhann merged 1 commit into
mainfrom
product-ea-limit

Conversation

@0rkhann

@0rkhann 0rkhann commented Oct 2, 2026

Copy link
Copy Markdown
Owner

The nightly's pre-YAML comparison failed at random on product-formation Ea at 3.2e-2 M PMe3: two GitHub runs of the same commit (#30) gave 0.081 and 0.008 kcal/mol, against a 0.03 limit.

tests/check_equivalence.py now splits this limit at 1e-2 M, like its DRC check already does:

  • up to 1e-2 M: 0.03, unchanged (largest seen 0.0145);
  • above 1e-2 M: 0.12, 1.5 times the largest seen (0.081).

Checked on the results of both CI runs and a local run: all print Run matches the baseline. The comparison with the paper (tests/check_reproduction.py) is unchanged.

🤖 Generated with Claude Code

Two GitHub runs of the same commit differed by 0.081 and 0.008 kcal/mol at
3.2e-2 M, so the 0.03 limit failed at random there. Up to 1e-2 M the limit
stays 0.03; above it, 0.12, like the DRC check's split.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@0rkhann
0rkhann merged commit 772c66d into main Oct 2, 2026
2 checks passed
@0rkhann
0rkhann deleted the product-ea-limit branch October 2, 2026 19:05
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