search

AtChem / AtChem2

Atmospheric chemistry box-model for the MCM
MIT License
58 stars 23 forks source link

Upgrade CVODE #257

Open spco opened 6 years ago

spco commented 6 years ago

LLNL have released SUNDIALS/CVODE 3.x : we should probably upgrade to use this unless there's a good reason not to.

rs028 commented 6 years ago

I tried to update this, but it seems they changed the interface significantly. I would hold off the upgrade until the current work is finished.

spco commented 6 years ago

Yeah, I haven't looked into this at all - it may be some small changes required, or something more. We can take a look when we get round to it :)

rs028 commented 6 years ago

Note: if CVODE is updated it overwrites the older version. That means that older models may not run anymore.

spco commented 5 years ago

The branch with this work on is https://github.com/spco/AtChem2/tree/upgrade_cvode , but I haven't looked at it since, so I can't say for sure what state it's in right now. It requires a number of modifications, and different function calls to previously.

rs028 commented 3 years ago

Check the use of LAPACK (and BLAS?) when implementing the new version of CVODE.

  1. are they used? if not, should they be implemented?
  2. if not used or necessary, drop from the list of requirements?
rs028 commented 1 year ago

Bumping to higher priority because CVODE has now reached version 6.x (https://sundials.readthedocs.io/en/latest/cvode/Introduction_link.html#changes-from-previous-versions).