Open sloede opened 2 months ago
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 66.29%. Comparing base (
b1a84a6
) to head (a54a3e5
).
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
We get errors about overwritten methods, e.g., in the log here or here.
The problem seems to be multiple includes of the same function definition, e.g., when looping over setups (first case) or just reusing a setup with an explicit include
(the second case).
Any idea how this can be easily fixed? For example, would it be possible (sensible) that we create a separate (anonymous?) Julia module upon each invocation of test_trixi_include
?
Ah, I see that @ranocha had the idea of anonymous modules before, but that they do not work with our structured, curvilinear files 😢 https://github.com/trixi-framework/Trixi.jl/blob/b1a84a63298966b67491dacac78e785e0dbfb36c/test/test_trixi.jl#L205-L213 Any other ideas how to proceed?
Let's just include warnings like
"WARNING: Method definition pretty_form_utf(Base.Val{:energy_potential}) in module TrixiTestModule at /home/runner/work/Trixi.jl/Trixi.jl/examples/paper_self_gravitating_gas_dynamics/elixir_eulergravity_jeans_instability.jl:121 overwritten in module TrixiTestModule on the same line (check for duplicate calls to `include`)
in the list of ignored warnings?
Let's just include warnings like
"WARNING: Method definition pretty_form_utf(Base.Val{:energy_potential}) in module TrixiTestModule at /home/runner/work/Trixi.jl/Trixi.jl/examples/paper_self_gravitating_gas_dynamics/elixir_eulergravity_jeans_instability.jl:121 overwritten in module TrixiTestModule on the same line (check for duplicate calls to `include`)
in the list of ignored warnings?
Won't we risk hiding real errors with this? But in general, I do like the idea due to its simplicity 👍
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
.Testing
Performance
Verification
Created with :heart: by the Trixi.jl community.