Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added StaticArrays extension for expv method #180

Merged
merged 4 commits into from
Oct 24, 2024

Conversation

jecs
Copy link
Contributor

@jecs jecs commented Oct 23, 2024

  • Added ext/ExponentialUtilitiesStaticArraysExt.jl, which computes the exponential matrix-vector product efficiently when using StaticArrays.
  • Updated authors list, version #, and uuid in Project.toml.
  • Changed == nothing and === nothing to isnothing.

Checklist

  • Appropriate tests were added
  • Any code changes were done in a way that does not break public API
  • All documentation related to code changes were updated
  • The new code follows the
    contributor guidelines, in particular the SciML Style Guide and
    COLPRAC.
  • Any new documentation only uses public API

Additional context

This implementation is based on the work by Al-Mohy, A. H., & Higham, N. J. titled "Computing the action of the matrix exponential, with an application to exponential integrators", published in 2011 in SIAM journal on scientific computing, 33(2), 488-511.

* Added ext/ExponentialUtilitiesStaticArraysExt.jl, which computes
  the exponential matrix-vector product efficiently when using
  StaticArrays.
* Updated authors list, version #, and uuid in Project.toml.
* Changed == nothing and === nothing to isnothing.
@jecs
Copy link
Contributor Author

jecs commented Oct 23, 2024

ERROR: Refusing to add package ExponentialUtilities [3927d2ad].

Anything I could do to remedy this? I could revert the UUID. I'll give you write access to my fork as well, if that makes things easier.

Project.toml Outdated Show resolved Hide resolved
test/basictests.jl Outdated Show resolved Hide resolved
Project.toml Outdated Show resolved Hide resolved
@ChrisRackauckas ChrisRackauckas merged commit ad797f0 into SciML:master Oct 24, 2024
17 of 30 checks passed
@ChrisRackauckas
Copy link
Member

Thanks!!!

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