Skip to content
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

Add lower-resolution mesh for BFS #177

Merged
merged 1 commit into from
Mar 21, 2024
Merged

Conversation

jcallaham
Copy link
Collaborator

Added a new mesh that has about 1/3 of the elements of the full-resolution mesh but is pretty close to matching validated results. Here is a time series of the fluctuation KE (scaled by domain size) with the same random seed in the new mesh and "fine":

image

The new mesh has outflow length L=15 compared to L=25 for the "fine" mesh. We can't really compress it much more than that without losing significant resolution, since the upper flow doesn't reattach until x=10 or so. The new mesh has about 81k elements, compared to 223k elements for the fine mesh.

Coarse mesh:

Lower reattachment point: 5.66 (L0)
Upper separation point: 4.46 (L0)
Upper reattachment point: 10.09 (L0)

Fine mesh:

Lower reattachment point: 5.84 (L0)
Upper separation point: 4.63 (L0)
Upper reattachment point: 10.19 (L0)

Boujo & Gallaire (rescaled to our length scale):

Lower reattachment point: 5.9 (L0)
Upper separation point: 4.7 (L0)
Upper reattachment point: 10.3 (L0)

The examples/step/solve-steady.py script also includes stagnation point calculation.

Copy link
Collaborator

@ludgerpaehler ludgerpaehler left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me

@ludgerpaehler ludgerpaehler merged commit 0ca12c6 into main Mar 21, 2024
6 checks passed
@ludgerpaehler ludgerpaehler deleted the jc/coarse-step-mesh branch March 21, 2024 08:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants