search

gridap / GridapODEs.jl

Time stepping for Gridap
MIT License
33 stars 7 forks source link

Test Set #56

Closed ConnorMallon closed 2 years ago

ConnorMallon commented 2 years ago

Is there a reason why not all of the tests are in the runtests.jl file?

i.e. there are 14 test in test/TransientFEsTests but only 11 tests in test/TransientFEsTests/runtests.jl

ConnorMallon commented 2 years ago

Hey @oriolcg, the autodiff tests are currently not included in the test set. Should these be working or is there more development required to get them working? For me, the StokesEquationAutoDiffTests.jl errors.

oriolcg commented 2 years ago

Hi @ConnorMallon, I updated the auto-diff part of TransientFEOperators in the branch "Fixing_autodiff". The Stokes test still doesn't work, but I think that the problem might be related to issue gridap/Gridap.jl#584. The other test that is not added is the ForwardEuler test, which I haven't had time to check where is the problem.

ConnorMallon commented 2 years ago

Ok I see, thanks Oriol :)