Skip to content

Adding LEOS interface #340

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

Draft
wants to merge 107 commits into
base: develop
Choose a base branch
from
Draft

Adding LEOS interface #340

wants to merge 107 commits into from

Conversation

jmikeowen
Copy link
Collaborator

@jmikeowen jmikeowen commented Apr 18, 2025

Summary

  • This PR is a feature
  • It does the following:
    • Adding EquationOfState interface for using the Livermore Equation of State (LEOS) package
    • Requires access to the LEOS third-party library, which is only available at LLNL

ToDo :

  • Annotate RELEASE_NOTES.md with notable changes.
  • Create LLNLSpheral PR pointing at this branch. (PR#133)
  • LLNLSpheral PR has passed all tests.

…ancement. Also cleaned up some stuff in the Integrator package.
cleaned up a few details (int->size_t for sizes of arrays handed back
a few places, and DataBase Python interface for getting NodeList
arrays), and these had wide ranging impact in terms of touching a lot
of files.  No substantive changes to most of these files however.
change. In the process I removed some inline headers which resulted in
including fewer headers across other classes.  This cascaded into
touching a lot of files to include those where necessary, which is
most of the files touched in this commit. It's still worthwhile
getting unnecessary includes out of headers so I went with it.
use with implicit integration and packing the solution vector. Our
logic for picking the independent state to be packed though needs to
be improved.
PYB11Generator, and updated our use of this in Spheral accordingly
jmikeowen added 30 commits April 4, 2025 15:56
I'm sure there's a better way to do this...
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