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

More interpolation operations #8

Merged
merged 12 commits into from
Jun 28, 2021
Merged

More interpolation operations #8

merged 12 commits into from
Jun 28, 2021

Conversation

Yurlungur
Copy link
Collaborator

PR Summary

For singularity-opac, I need the capability to perform interpolation of the form:

Real v = db.interpToReal(x4, x3, x2, j, x1)

where x1, x2, x3, and x4 are real numbers that are interpolated on and j is an integer that's indexed on. This is a common pattern for multi-species transport such as neutrino transport. I don't expect it's needed outside of neutrino transport, but you never know.

PR Checklist

  • Adds a test for any bugs fixed. Adds tests for new features.

@Yurlungur Yurlungur self-assigned this Jun 28, 2021
@Yurlungur
Copy link
Collaborator Author

I also add raw 4D interpolation, needed for transport, and tests. Test for the indexing operation not yet complete.

@Yurlungur
Copy link
Collaborator Author

I also fix an edge case I hadn't encountered previously, where I use Kokkos on host only.

@Yurlungur Yurlungur changed the title Jmm/more interps [WIP]: More interpolation operations Jun 28, 2021
@Yurlungur Yurlungur changed the title [WIP]: More interpolation operations More interpolation operations Jun 28, 2021
@Yurlungur
Copy link
Collaborator Author

This is now ready for review.

databox.hpp Outdated Show resolved Hide resolved
databox.hpp Outdated Show resolved Hide resolved
@Yurlungur
Copy link
Collaborator Author

Thanks for the comments, @dholladay00 I think I should have addressed everything. If i did, can you approve?

@Yurlungur Yurlungur merged commit 8fb666f into main Jun 28, 2021
@Yurlungur Yurlungur deleted the jmm/more-interps branch June 28, 2021 21:31
dholladay00 pushed a commit that referenced this pull request May 26, 2023
Co-authored-by: Chad Meyer <chadmeyer@lanl.gov>
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.

None yet

3 participants