-
Notifications
You must be signed in to change notification settings - Fork 3
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
Manufactured solution for Svärd-Kalisch equations with variable bathymetry #84
Conversation
…ions with variable bathymetry
…riable bathymetry to work
This is breaking because I change the behavior of |
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.
Thanks a lot! This looks good to me - nice work 👍
Co-authored-by: Hendrik Ranocha <[email protected]>
….com/JoshuaLampert/DispersiveShallowWater.jl into manufactured-Svaerd-Kalisch-variable
CI failed in the last run. Do you have an idea why - do we need to increase the test tolerances? |
Yes, the tolerances need to be increased. I just did that in 9c94fa3. Let's hope CI passes now. |
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.
Thanks a lot!
I put the tolerance at the wrong place - stupid me. 🙄 |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #84 +/- ##
==========================================
+ Coverage 96.07% 96.12% +0.05%
==========================================
Files 17 17
Lines 1095 1110 +15
==========================================
+ Hits 1052 1067 +15
Misses 43 43 ☔ View full report in Codecov by Sentry. |
To avoid massive clutter of the terminal, I increased the |
I finally found a working version for a manufactured solution for the Svärd-Kalisch equations with a variable bathymetry, see also #65.
Convergence test:
p = 2:
p = 4:
p = 6: