-
Notifications
You must be signed in to change notification settings - Fork 37
RK sweeper #163
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
RK sweeper #163
Conversation
pancetta
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could you move the test_order into the tests? Then it'll be picked up during CI testing.
with the new sweepers which avoids integration with both preconditioner and quadrature matrix. The nodes might be wrong, which would only show up in problems with time-dependent right hand sides.
|
Please have a look at the stability plots at the bottom of https://github.com/Parallel-in-Time/pySDC/blob/8201e6e4e46d1866b506b6081065456b8d313aa2/pySDC/playgrounds/Runge_Kutta/Runge-Kutta-Methods.ipynb Are they what you would expect? I have no idea about RK methods and stability. In particular: Is implicit midpoint A-stable or did I make a mistake? |
LGTM. The plot for the explicit methods is somewhat hard to read, though. Could you highlight the region of stability? |
|
And yes, implicit midpoint is A-stable, being a Gauss-Legendre method. |
collocation are given and I shaded the regions of stability in the plot
|
Explicit Euler is unusually stable (entire left half plane). That's weird.. Any ideas why? |
|
Where do you see this? Smells like you did implicit Euler by accident. |
Again in the last plot in https://github.com/Parallel-in-Time/pySDC/blob/a1bfc50fe1e15799d260e0b51362cb949dd78b75/pySDC/playgrounds/Runge_Kutta/Runge-Kutta-Methods.ipynb The stability region looks different than that of implicit Euler, though.. |
New sweeper that takes a lower triangular Butcher tableau as input and does Runge-Kutta. I tried to eliminate unnecessary computations by forgoing the preconditioner, rather than setting preconditioner equal to the quadrature rule.
In the jupyter notebook there is a plot that confirms the order of a few RK schemes, but I have not made any further analysis and also implemented only few RK schemes.