Skip to content

Sundials upgrade#543

Closed
ergoregion wants to merge 9 commits intoAtChem:masterfrom
ergoregion:sundials-upgrade
Closed

Sundials upgrade#543
ergoregion wants to merge 9 commits intoAtChem:masterfrom
ergoregion:sundials-upgrade

Conversation

@ergoregion
Copy link

This PR updates the code to work with sundials 7.5.0.

  • The install_cvode script has been updated to use sundials 7.5.0
  • There are changes to the atchem2 file to use the new version of cvode.
  • There is a change to OutputFunctions because the means of displaying the data about the run is changed
  • The skeleton of the makefile now requires a new field CVODEOBJDIR the place where thee fortran object files are stored
  • the readme and docker install scripts have been updated to try to tell people about the new makefile setting

This can be made neater with further code changes, I left the main right hand side function FCVFUN unchanged, and invoked it from the new cvode method,
Consolidating these methods would be a wise move.
The class of UserData could be neatened to store named variables, instead of old-style arrays of ints and reals

@ergoregion ergoregion closed this Jan 14, 2026
@ergoregion
Copy link
Author

replaced with #545

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