Skip to content

Do not ignore dagmc errors#3764

Open
GuySten wants to merge 4 commits intoopenmc-dev:developfrom
GuySten:dagmc-silent-errors
Open

Do not ignore dagmc errors#3764
GuySten wants to merge 4 commits intoopenmc-dev:developfrom
GuySten:dagmc-silent-errors

Conversation

@GuySten
Copy link
Contributor

@GuySten GuySten commented Feb 3, 2026

Description

Currently dagmc errors are silently ignored.
This PR makes openmc terminate when dagmc errors appear.

Fixes #3224.

Checklist

  • I have performed a self-review of my own code
  • I have run clang-format (version 15) on any C++ source files (if applicable)
  • I have followed the style guidelines for Python source files (if applicable)
  • I have made corresponding changes to the documentation (if applicable)
  • I have added tests that prove my fix is effective or that my feature works (if applicable)

@GuySten GuySten marked this pull request as ready for review February 3, 2026 07:21
@GuySten GuySten requested a review from pshriwise as a code owner February 3, 2026 07:21
@GuySten GuySten added the Bugs label Feb 4, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

When DAGMC fails during initialisation OpenMC should halt

2 participants