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

Does StochasticDiffEq need the full OrdinaryDiffEq dependency? #596

Open
oameye opened this issue Dec 31, 2024 · 3 comments
Open

Does StochasticDiffEq need the full OrdinaryDiffEq dependency? #596

oameye opened this issue Dec 31, 2024 · 3 comments
Labels

Comments

@oameye
Copy link

oameye commented Dec 31, 2024

Now that OrdinaryDiffEq has been split up, does StochasticDiffEq need the full OrdinaryDiffEq library?

@ytdHuang
Copy link

ytdHuang commented Jan 20, 2025

any update for this issue ?

Cause StochasticDiffEq depends on the entire OrdinaryDiffEq instead of OrdinaryDiffEqCore, and installs a lot more libraries and packages.

@ChrisRackauckas
Copy link
Member

No movement on it yet, and I don't htink I can get to it this month. But maybe the first change is that, you probably could make it just depend on OrdinaryDiffEqCore, OrdinaryDiffEqNonlinearSolve and that might be a minimal amount. We should split StochasticDiffEq into sublibs like OrdinaryDiffEq in order to fully minimize it, but that would be a bit more work.

@ytdHuang
Copy link

ytdHuang commented Jan 20, 2025

@ChrisRackauckas

Thank you for the information.

I'm already using SciMLBase as the minimal dependency of my package (QuantumToolbox.jl).

For the StochasticDiffEq, I just want to set a default algorithm for our function to use stochastic differential equation, something like:

import SciMLBase: solve, SDEProblem
import StochasticDiffEq: StochasticDiffEqAlgorithm, SRA1

function sde_function(...; alg::StochasticDiffEqAlgorithm = SRA1(), ...)
    ...
    prob = SDEProblem(...)
    sol = solve(prob, alg, ...)
    ...
end

I noticed that, for the StochasticDiffEqAlgorithm, I can use SciMLBase.AbstractSDEAlgorithm instead.

Would it be possible to not import StochasticDiffEq: SRA1, but also solve with SDE algorithms?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants