-
Notifications
You must be signed in to change notification settings - Fork 29
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
Add LSRK54 time stepping method #335
Conversation
Should we include the |
+ change Fi -> RHS, fix docstrings, cleanup
I made a few changes @doraemonho. Have a look and let me know what you think. |
It is workable but I am not sure if the solution is stable in the combination with the filter + low storage method. |
@navidcy I changed the coef. values to rational form and also tested once. Could you check if the formatting is correct or not? |
When tests pass I'll merge. |
@doraemonho welcome to FourierFlows contributors ;) |
It is my pleasure : D |
Implements a 4th-order 5-stages 2-storage Runge-Kutta timestepper,
LSRK54
.Closes #334