Skip to content

fix: suppress metatomic FPE, disable uncertainty default, align dimer cols#323

Merged
HaoZeke merged 1 commit intoTheochemUI:mainfrom
HaoZeke:fix/metatomic-fpe-and-dimer-fmt
Mar 2, 2026
Merged

fix: suppress metatomic FPE, disable uncertainty default, align dimer cols#323
HaoZeke merged 1 commit intoTheochemUI:mainfrom
HaoZeke:fix/metatomic-fpe-and-dimer-fmt

Conversation

@HaoZeke
Copy link
Collaborator

@HaoZeke HaoZeke commented Mar 2, 2026

  • Suppress FPE trapping during libtorch operations in MetatomicPotential constructor and force(), preventing SIGFPE from benign NaN/Inf in SiLU/autograd internals (follows existing FPEHandler pattern from ASE_ORCA, ASE_NWCHEM, AtomicGPDimer)
  • Change uncertainty_threshold default from 0.1 to -1 (disabled) in C++ (Parameters.cpp) and Python (schema.py), making uncertainty checking opt-in for models that actually provide it
  • Fix [IDimerRot] force column placeholder width (10 to 18 dashes) and angle precision ({:6.2f} to {:6.3f}) for alignment parity with [Dimer]
  • Add missing {:5s} "Align" column to [Dimer] header in MinModeSaddleSearch.cpp

… columns

Wrap MetatomicPotential constructor and force() with FPEHandler to
prevent SIGFPE from benign NaN/Inf in libtorch internals (SiLU/sleef,
autograd). Change uncertainty_threshold default from 0.1 to -1
(disabled) in C++ and Python so models without uncertainty outputs skip
the noisy try/catch. Fix IDimerRot force placeholder width (10 to 18
dashes), angle precision (6.2f to 6.3f), and add missing Align column
to the Dimer header.
@github-actions
Copy link

github-actions bot commented Mar 2, 2026

eOn Documentation Preview

Download: documentation.zip

Unzip and open index.html to view.

@HaoZeke HaoZeke merged commit 9160f9d into TheochemUI:main Mar 2, 2026
12 checks passed
@HaoZeke HaoZeke deleted the fix/metatomic-fpe-and-dimer-fmt branch March 2, 2026 20:00
HaoZeke added a commit that referenced this pull request Mar 4, 2026
fix: suppress metatomic FPE, disable uncertainty default, align dimer cols
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