Skip to content

FSI + kOmegaSST turbulence model #643

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

Draft
wants to merge 7 commits into
base: develop
Choose a base branch
from

Conversation

vidulejs
Copy link
Collaborator

@vidulejs vidulejs commented Apr 4, 2025

As we agreed, this is the pull request where we can discuss the issue with the Poisson wall distance calculation and eventually create a new turbulent FSI tutorial. This case would have to be modified to have a higher Re number.

@MakisH
@KariimAhmed

Checklist:

  • I added a summary of any user-facing changes (compared to the last release) in the changelog-entries/<PRnumber>.md.
  • I will remember to squash-and-merge, providing a useful summary of the changes of this PR.

Sorry, something went wrong.

MakisH added 5 commits April 7, 2025 11:03

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
@MakisH
Copy link
Member

MakisH commented Apr 7, 2025

Thanks for pushing this! @KariimAhmed could you please try to adjust this case to reproduce your issue? It would then be much easier for us to fix.

After we have a fix, we should indeed convert this to a new perpendicular-flap-turbulent tutorial.

@vidulejs I know this was meant to be a complete draft. I just went ahead changed some formatting things to make this already easier to review and transform into a tutorial, so that you can focus on the actual issue. Please pull from your branch.

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
@vidulejs
Copy link
Collaborator Author

vidulejs commented Apr 8, 2025

Thanks for fixing the formatting @MakisH!

Just to re-iterate from our conversation, we want to create a fully turbulent FSI test case to see if an adapter-related issue appears. We should start with increasing the velocity, to achieve a higher Reynolds number. The numerical schemes will probably need updating, because the turbulent energy dissipation rate omega was exploding.

I noticed I accidentally pushed the wrong wallDist method, but during our meeting I had the Poisson method and the case was running succesfully.

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.

None yet

2 participants