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

Adapt KrylovSubspace Vtype to u #2538

Merged
merged 2 commits into from
Nov 19, 2024
Merged

Conversation

mcarmesin
Copy link
Contributor

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
  • [x ] The new code follows the
    contributor guidelines, in particular the SciML Style Guide and
    COLPRAC.
  • Any new documentation only uses public API

Additional context

Specify the type of the KrylovSubspace basis vector matrix according to the specified u. This allows for u being a GPUArray.

ExponentiationUtilities has to be adjusted as well to provide this functionality, see my PR https://github.com/SciML/ExponentialUtilities.jl/pull/183 there.

@mcarmesin mcarmesin changed the title Adopt KrylovSubspace Vtype to u Adapt KrylovSubspace Vtype to u Nov 18, 2024
@ChrisRackauckas
Copy link
Member

This should get a GPU test, but I'll take it for now. If you follow up with a GPU test I'd be super happy though.

@ChrisRackauckas ChrisRackauckas merged commit 9ab927b into SciML:master Nov 19, 2024
131 of 141 checks passed
@ChrisRackauckas
Copy link
Member

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