Closed utkarsh530 closed 1 year ago
Other than the extra dispatch, this looks good to go.
Can you tag NonlinearSolve.jl
?
Merging #184 (2db273f) into master (32417d4) will not change coverage. The diff coverage is
0.00%
.
@@ Coverage Diff @@
## master #184 +/- ##
======================================
Coverage 0.00% 0.00%
======================================
Files 9 9
Lines 1894 1993 +99
======================================
- Misses 1894 1993 +99
Impacted Files | Coverage Δ | |
---|---|---|
src/DiffEqGPU.jl | 0.00% <0.00%> (ø) |
|
src/integrators/integrator_utils.jl | 0.00% <0.00%> (ø) |
|
src/integrators/types.jl | 0.00% <0.00%> (ø) |
|
src/perform_step/gpu_tsit5_perform_step.jl | 0.00% <0.00%> (ø) |
|
src/perform_step/gpu_vern7_perform_step.jl | 0.00% <0.00%> (ø) |
|
src/perform_step/gpu_vern9_perform_step.jl | 0.00% <0.00%> (ø) |
|
src/solve.jl | 0.00% <ø> (ø) |
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more
The retcode stuff should be good with the new DiffEqBase version that uses SimpleNonlinearSolve. Setup the test and let's get this one merged.
This is still missing tests
Added continuous callbacks tests to Tsit5
.
Looks fine. I would reorganize this in a bit. There's no reason to copy paste the same discrete and continuous callback scripts into a file for each solver: just run the same setup with different solvers (and check that they give close to equivalent results). But let's merge and reorganize in a follow up.
Yes, reorganization is my next task. Let's merge this, and I'll follow up with a reorganization.
Also, I'll need to address the growing If-Else blocks for adaptive and other things that increases compile time.
Real failures.
Strange.. everything is passing locally
LGTM
Hi, I have completed Continuous callbacks for
GPUTsit5
. I tested the bouncing ball problem and works well:I need to write tests, which will not pass until we merge https://github.com/SciML/NonlinearSolve.jl/pull/92 . @ChrisRackauckas, please review.