Closed marcinlos closed 3 years ago
Merging #62 (370e80b) into develop (d387ed0) will not change coverage. The diff coverage is
n/a
.
@@ Coverage Diff @@
## develop #62 +/- ##
========================================
Coverage 35.66% 35.66%
========================================
Files 43 43
Lines 1203 1203
========================================
Hits 429 429
Misses 774 774
Continue to review full report at Codecov.
Legend - Click here to learn more
Δ = absolute <relative> (impact)
,ø = not affected
,? = missing data
Powered by Codecov. Last update d387ed0...370e80b. Read the comment docs.
In the multistep code, we start with a linear multistep method
We want to replace with to use ADS solver, but this introduces error in each time step, which reduces the order of method to 1, making it pointless to go for higher order schemes.
To prevent that, we add some stuff to both sides so that is applied to a small quantity. For example, is of order , so we can construct a third order method starting with
and replacing with :
In the previous version of the code, we would move everything except for to the right-hand side and end up with
since . This is, however, a bad idea, since the bilinear form associated with is unnatural and difficult to evaluate (requires mixed derivatives), especially in 3D. Instead, it is better to solve for and compute from it.
Closes #59