Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix lagrangian interpolation #2775

Merged
merged 5 commits into from
Nov 15, 2023

Conversation

pshriwise
Copy link
Contributor

Description

This fixes the y value used in Lagrangian interpolation to use the values in the local interval rather than only the ones at the beginning of the array. Tests have been added for correctness for several orders of interpolation with spot checks over all the interpolation orders we use in the EnergyFunctionFilter.

Fixes #2765

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)

Copy link
Member

@shimwell shimwell left a comment

Choose a reason for hiding this comment

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

Thanks Patrick much appreciated

@shimwell
Copy link
Member

This works nicely and I see it has passed all the tests. Tagging with the will merge in less than 24 hours, let me know if there are objections

@shimwell shimwell added the Merging Soon PR will be merged in < 24 hrs if no further comments are made. label Nov 14, 2023
@shimwell shimwell merged commit 24e1c95 into openmc-dev:develop Nov 15, 2023
18 checks passed
church89 pushed a commit to openmsr/openmc that referenced this pull request Jul 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Merging Soon PR will be merged in < 24 hrs if no further comments are made.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Cubic interpolation of dose coefficients producing order of magnitude lower doses
2 participants