Skip to content

Successful step function no longer works for parallel ODE system #214

@Datseris

Description

@Datseris

Hm, some update in the OrdinaryDiffEq.jl library broke this. So, in the test suite for the successful_step function the parallel ODE fails to convey that the steps take have failed. Perhaps this is related to the fact that we pass in a custom norm internalnorm = inorm when we create a parallel ODE system, here

pds = CoupledODEs(prob, ds.diffeq; internalnorm = inorm)

but honestly I am not sure. What I am sure about is that the test fails:

pds = ParallelDynamicalSystem(deepcopy(ode), [u0,u0])

which is why in the current code this is commented out. We will need to test whether this is truly an OrdinaryDiffEq.jl problem by simply making a MWE with an ODEProblem with a parallel rule for two states and passing in a modified internalnorm as a keyword argument. If it is an OrdinaryDiffEq.jl problem, we open an issue there with our MWE. If not, then we have to fix it here.

Metadata

Metadata

Assignees

No one assigned

    Labels

    good first issueIssue suited for a first time contributortestingThis issue/PR needs tests

    Type

    No type
    No fields configured for issues without a type.

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions