Skip to content

ARKODE_MRI interface - #3013

Closed
maggul wants to merge 0 commit into
boutproject:nextfrom
maggul:next
Closed

maggul wants to merge 0 commit into
boutproject:nextfrom
maggul:next

Conversation

@maggul

@maggul maggul commented Oct 25, 2024

Copy link
Copy Markdown
Contributor

ARKODE MRI interface
An integrated test with ARKODE MRI interface

Comment thread src/solver/impls/arkode/arkode_mri.hxx Outdated
Comment thread src/solver/impls/arkode/arkode_mri.hxx Outdated
Comment thread src/solver/impls/arkode/arkode_mri.hxx Outdated
Comment thread src/solver/impls/arkode/arkode_mri.hxx Outdated
Comment thread src/solver/impls/arkode/arkode_mri.hxx Outdated
Comment thread src/solver/solver.cxx Outdated
Comment thread tests/integrated/test-kpr_mri/README.md Outdated
Comment thread tests/integrated/test-kpr_mri/test_kpr_mri.cxx Outdated
Comment thread tests/integrated/test-kpr_mri/test_kpr_mri.cxx Outdated
Comment thread tests/integrated/test-kpr_mri/test_kpr_mri.cxx Outdated

@drreynolds drreynolds left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I believe that these should take care of the updates related to the new SUNDIALS release (that supports MRI temporal adaptivity).

Comment thread src/solver/impls/arkode/arkode_mri.cxx Outdated
Comment thread src/solver/impls/arkode/arkode_mri.cxx Outdated
Comment thread src/solver/impls/arkode/arkode_mri.cxx Outdated
Comment thread src/solver/impls/arkode/arkode_mri.cxx Outdated
Comment thread src/solver/impls/arkode/arkode_mri.cxx Outdated
@maggul

maggul commented Jan 9, 2025

Copy link
Copy Markdown
Contributor Author

I think it should be ready now for your reviews @bendudson @drreynolds

@maggul
maggul marked this pull request as ready for review January 16, 2025 18:51
@maggul maggul closed this Sep 13, 2026
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.

3 participants