Skip to content

CI upgrades to support sundials@7.4.0#164

Merged
pelesh merged 1 commit intosundials-7.4.0from
nicholson/sundials-7.4.0
Jun 27, 2025
Merged

CI upgrades to support sundials@7.4.0#164
pelesh merged 1 commit intosundials-7.4.0from
nicholson/sundials-7.4.0

Conversation

@nkoukpaizan
Copy link
Copy Markdown
Collaborator

Description

CI upgrades to support sundials@7.4.0

Proposed changes

Updated Spack fork with sundials@7.4.0 and setting CI to use that version.

Checklist

  • All tests pass.
  • Code compiles cleanly with flags -Wall -Wpedantic -Wconversion -Wextra.
  • [N/A] The new code follows GridKit™ style guidelines.
  • [N/A] There are unit tests for the new code.
  • [N/A] The new code is documented.
  • The feature branch is rebased with respect to the target branch.

@nkoukpaizan nkoukpaizan self-assigned this Jun 27, 2025
@Steven-Roberts
Copy link
Copy Markdown
Collaborator

Someone on the SUNDIALS team should be opening a PR to update the spack package for v7.4.0 today.

Does this need updating too?

RUN git clone --single-branch --branch develop https://github.com/LLNL/sundials.git /tmp/sundials && \

@nkoukpaizan
Copy link
Copy Markdown
Collaborator Author

Someone on the SUNDIALS team should be opening a PR to update the spack package for v7.4.0 today.

Does this need updating too?

RUN git clone --single-branch --branch develop https://github.com/LLNL/sundials.git /tmp/sundials && \

Yes, the container should be updated too. There is a specific commit being set. cc @superwhiskers @alexander-novo

Note that for CI, we are currently pointing Spack to my fork, which is behind upstream Spack. So, yes we want the new SUNDIALS version to be in upstream spack and I will let the SUNDIALS team do that, but we wouldn't get that update in CI.

Copy link
Copy Markdown
Collaborator

@pelesh pelesh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

SUNDIALS Spack has been updated and correct version of the library has been set in our CI.

@pelesh pelesh merged commit cae6a55 into sundials-7.4.0 Jun 27, 2025
4 checks passed
pelesh pushed a commit that referenced this pull request Jun 27, 2025
* Update build requirements to SUNDIALS version 7.4.0

* Updated Spack fork with sundials@7.4.0 and set CI to use that version. (#164)

---------

Co-authored-by: Nicholson Koukpaizan <72402802+nkoukpaizan@users.noreply.github.com>
@nkoukpaizan nkoukpaizan deleted the nicholson/sundials-7.4.0 branch June 30, 2025 15:08
WiktoriaZielinskaORNL pushed a commit that referenced this pull request Jul 23, 2025
* Update build requirements to SUNDIALS version 7.4.0

* Updated Spack fork with sundials@7.4.0 and set CI to use that version. (#164)

---------

Co-authored-by: Nicholson Koukpaizan <72402802+nkoukpaizan@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants