Skip to content

Update SUNDIALS support to SUNDIALS v7.0.0#3756

Closed
balos1 wants to merge 2 commits intoAMReX-Codes:developmentfrom
sundials-codes:development
Closed

Update SUNDIALS support to SUNDIALS v7.0.0#3756
balos1 wants to merge 2 commits intoAMReX-Codes:developmentfrom
sundials-codes:development

Conversation

@balos1
Copy link
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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 <[email protected]>
Co-authored-by: Cody Balos <[email protected]>
Co-authored-by: Axel Huebl <[email protected]>
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