Open sloede opened 1 month ago
Attention: Patch coverage is 83.09859%
with 12 lines
in your changes are missing coverage. Please review.
Project coverage is 96.09%. Comparing base (
3b52a30
) to head (38b566d
).
Files | Patch % | Lines |
---|---|---|
src/callbacks_stage/modal_filter_dg2d.jl | 65.38% | 9 Missing :warning: |
src/callbacks_stage/modal_filter.jl | 89.29% | 3 Missing :warning: |
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
Currently, running a convergence test does not work, in the sense that it gives exactly the same errors independent of the mesh resolution. It's probably just a denkfehler that requires a simple fix, most likely to the way we use the AnalysisCallback
here.
Nevertheless, here are the current results (careful, the last run takes quite a long time (~8 mins)):
julia> convergence_test("examples/tree_2d_dgsem/elixir_euler_ec_modal_filter.jl", 2, initial_condition=initial_condition_convergence_test, n_cells_max=1_000_000, polydeg=7, tspan=(0.0, 0.3))
###################################################################################################
l2
rho rho_v1 rho_v2 rho_e
error EOC error EOC error EOC error EOC
6.99e-02 - 1.31e-01 - 1.31e-01 - 3.67e-01 -
6.99e-02 -0.00 1.31e-01 -0.00 1.31e-01 -0.00 3.67e-01 -0.00
6.99e-02 -0.00 1.31e-01 -0.00 1.31e-01 -0.00 3.67e-01 -0.00
6.99e-02 -0.00 1.31e-01 -0.00 1.31e-01 -0.00 3.67e-01 -0.00
mean -0.00 mean -0.00 mean -0.00 mean -0.00
----------------------------------------------------------------------------------------------------
linf
rho rho_v1 rho_v2 rho_e
error EOC error EOC error EOC error EOC
9.89e-02 - 1.87e-01 - 1.87e-01 - 5.24e-01 -
9.89e-02 -0.00 1.87e-01 0.00 1.87e-01 0.00 5.24e-01 -0.00
9.89e-02 0.00 1.87e-01 0.00 1.87e-01 0.00 5.24e-01 -0.00
9.89e-02 0.00 1.87e-01 -0.00 1.87e-01 -0.00 5.24e-01 -0.00
mean -0.00 mean 0.00 mean 0.00 mean -0.00
----------------------------------------------------------------------------------------------------
Review checklist
This checklist is meant to assist creators of PRs (to let them know what reviewers will typically look for) and reviewers (to guide them in a structured review process). Items do not need to be checked explicitly for a PR to be eligible for merging.
Purpose and scope
Code quality
Documentation
NEWS.md
with its PR number.Testing
Performance
Verification
Created with :heart: by the Trixi.jl community.