-
Notifications
You must be signed in to change notification settings - Fork 82
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
[X-PR] Update OrdinaryDiffEq.jl and fix Navier-Stokes solver #917
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## mk/nsgmsh #917 +/- ##
==========================================
Coverage 93.30% 93.30%
==========================================
Files 36 36
Lines 5257 5257
==========================================
Hits 4905 4905
Misses 352 352 ☔ View full report in Codecov by Sentry. |
Dump. Dirichlet rates
|
Thanks for the fix Maxi! |
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.
# At the time of writing this [no Hessenberg index 2 initialization is implemented](https://github.com/SciML/OrdinaryDiffEq.jl/issues/1019). |
Co-authored-by: Maximilian Köhler <maximilian.koehler@ruhr-uni-bochum.de>
Co-authored-by: Maximilian Köhler <maximilian.koehler@ruhr-uni-bochum.de>
Wait. |
I am waiting for another round of fixes in OrdinaryDiffEq.jl, because adaptivity and interpolations are still broken (I just commented both out for now). After these are in one of the next releases we can update Manifest again and merge. |
Okay this one is ready to merge! |
For my own interest, could you comment on what |
Most solvers in DifferentialEquations.jl want to have a function of the form
The limiter is actually a thing from CFD when dealing e.g. with shockwaves. Basically you apply a problem-specific function |
Almosta fix for #508 . The new main idea is to eliminate the Dirichlet dofs from the mass matrix (provided in the solver ctor) and to provide the Jacobian (also with eliminated Dirichlet dofs) manually.TODOs
This type of integration is still not very clean, but at least we obtain quadratic convergence now (manually checked on a locally patched version of OrdinaryDiffEq.jl).