You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
While studying and making some test cases about the Warm Bubble Test for compressible Euler, the mesh on the y direction seems to be shifted, rather than being from 0 to 10_000, it is from -5000 to 15000.
Looks like a (documentation) bug to me. The TreeMesh supports only quads/cubes, no rectangles. But the constructor should complain in a case like this, I think. What's your opinion, @sloede?
MarcoArtiano
changed the title
Tree Mesh centering the mesh and shifting the boundaries
Tree Mesh centering the mesh and shifting the boundaries (rectangles to quads)
Mar 25, 2024
I added this test case for StructuredMesh and then copied it for TreeMesh to test the different flux implementations. I was not aware of the quad restriction in the first place, realized how TreeMesh adapts to the provided parameters and therefore added this comment in the elixir.
Looks like a (documentation) bug to me. The TreeMesh supports only quads/cubes, no rectangles. But the constructor should complain in a case like this, I think. What's your opinion, @sloede?
I thus understand why the code in the example behaves as expected. I think that changing the referenced lines to first compute the lengths in each direction and to then trow an error if they do not match (within an epsilon) before taking the maximum would be a nice addition for usability!
While studying and making some test cases about the Warm Bubble Test for compressible Euler, the mesh on the y direction seems to be shifted, rather than being from 0 to 10_000, it is from -5000 to 15000.
https://github.com/trixi-framework/Trixi.jl/blob/main/examples/tree_2d_dgsem/elixir_euler_warm_bubble.jl
As far as I know the mesh has to be a Quad, consequently it changes boundaries according to it.
The text was updated successfully, but these errors were encountered: