Skip to content

Conversation

@pshriwise
Copy link
Contributor

Description

Output of libMesh tally results to exodus is failing on newer versions of libMesh. This is an error on our side as the n_dimension_ attribute of the mesh is not being set for UnstructuredMesh objects. This value is propagated to the libMesh mesh object These changes

Tested with libMesh v1.8.2.

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)

@pshriwise pshriwise marked this pull request as ready for review September 19, 2025 18:00
Copy link
Contributor

@paulromano paulromano left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the fix!

@paulromano paulromano enabled auto-merge (squash) September 19, 2025 18:17
@paulromano paulromano merged commit ca63da9 into openmc-dev:develop Sep 19, 2025
14 checks passed
Grego01-biot pushed a commit to Grego01-biot/openmc that referenced this pull request Oct 27, 2025
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.

2 participants