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

Update SUNDIALS support to SUNDIALS v7.0.0 #3756

Closed
wants to merge 2 commits into from

Conversation

balos1
Copy link
Contributor

@balos1 balos1 commented Feb 13, 2024

Summary

This PR updates the SUNDIALS interface to SUNDIALS v7.0.0, which will be released at the end of the month.

Additional background

This PR breaks support for SUNDIALS < v7. I don't think it would difficult to support both versions if that is desired. A release candidate for SUNDIALS v7 is 'released' and tagged, but there will be a few more updates in the final v7 that currently sit in sundials@develop.

Checklist

The proposed changes:

  • fix a bug or incorrect behavior in AMReX
  • add new capabilities to AMReX
  • changes answers in the test suite to more than roundoff level
  • are likely to significantly affect the results of downstream AMReX users
  • include documentation in the code and/or rst files, if appropriate

@WeiqunZhang
Copy link
Member

We do want to make sure the old versions work too. It should not be hard by using the SUNDIALS_VERSION_MAJOR macro. If you don't have time, I am happy to finish this PR. If we can get this merged before the end of this month, we can get this into the next release and then update spack.

@balos1
Copy link
Contributor Author

balos1 commented Mar 21, 2024

Sounds good. If you can finish it up that would be great. If you have any issues just let me know.

@WeiqunZhang
Copy link
Member

Great. Since I cannot push to your branch, I will close this PR. Then I will cherry pick your commits and start from there. Thanks!

ax3l added a commit that referenced this pull request Mar 27, 2024
This is the rebirth of #3756. But it's compatible with SUNDIALS 6.

---------

Co-authored-by: Balos, Cody, J <balos1@llnl.gov>
Co-authored-by: Cody Balos <cjbalos@gmail.com>
Co-authored-by: Axel Huebl <axel.huebl@plasma.ninja>
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