search

SciML / StochasticDelayDiffEq.jl

Stochastic delay differential equations (SDDE) solvers for the SciML scientific machine learning ecosystem
MIT License
25 stars 10 forks source link

Update CI tests #3

Closed devmotion closed 4 years ago

devmotion commented 4 years ago

Run CI tests on Julia LTS, stable, and master.

ChrisRackauckas commented 4 years ago

This library probably needs quite a bit of updates. I've been waiting on getting it updated before adding it to the main docs, and adding regression tests to it.

devmotion commented 4 years ago

We should disable the tests on travis-ci.org, it seems currently they run both on travis-ci.com and travis-ci.org.

devmotion commented 4 years ago

Hmm seems tests on Travis run with DiffEqBase 6.20 whereas locally tests passed with DiffEqBase 6.29. Maybe we should just lower bounding DiffEqBase?

ChrisRackauckas commented 4 years ago

Yes, that should help. Note that I think callbacks will be broken for sure, and they aren't tested.

devmotion commented 4 years ago

Yes, I assume there's a lot of things we don't catch right now and should add to the test suite.

devmotion commented 4 years ago

Ah I just noticed, the problem is StochasticDiffEq. Tests fail with version 6.17 but it is the latest version that is compatible with Julia < 1.2. Should we just drop support for Julia < 1.2 in StochasticDelayDiffEq?

ChrisRackauckas commented 4 years ago

Yes, just up it to at least v1.3.

devmotion commented 4 years ago

OK, I guess this is good to go, tests pass now on Julia 1.4 and master. I'd suggest creating separate PRs for adding more tests and fixing bugs. Can you disable the tests on travis-ci.org? I don't have any permissions for StochasticDelayDiffEq it seems.