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

Remove mesh velocity from VOF scaling #1328

Merged
merged 2 commits into from
Nov 21, 2024
Merged

Remove mesh velocity from VOF scaling #1328

merged 2 commits into from
Nov 21, 2024

Conversation

wjhorne
Copy link
Contributor

@wjhorne wjhorne commented Nov 15, 2024

This PR uses a computed face velocity rather than the volume flow rate to calculate the velocity scaling of all VOF sharpening and diffusion at liquid/gas interfaces. This removes erroneous dependence of such interfaces on the mesh_velocity, which appears in the volume flow rate.

@wjhorne
Copy link
Contributor Author

wjhorne commented Nov 15, 2024

@mbkuhn I have to rebuild my spack environment to test this due to some recent changes to dependencies in spack. The changes here should remove mesh_velocity from the thickness of the interface in Nalu-Wind. I am in the process of fixing my spack and will test it out on a spinning patch

@mbkuhn
Copy link
Contributor

mbkuhn commented Nov 19, 2024

I'll try to test this today!

@mbkuhn
Copy link
Contributor

mbkuhn commented Nov 20, 2024

do you expect this to be equivalent in the static case? Because I'm not seeing that

@mbkuhn
Copy link
Contributor

mbkuhn commented Nov 20, 2024

The two are similar after 500 steps of the flat interface problem, but I was expecting them to be closer.
PR vs master (vof)
Screenshot 2024-11-20 at 3 57 37 PM
Screenshot 2024-11-20 at 3 58 25 PM

@mbkuhn
Copy link
Contributor

mbkuhn commented Nov 20, 2024

PR vs master (velocity_magnitude)
Screenshot 2024-11-20 at 4 02 33 PM

Screenshot 2024-11-20 at 4 02 44 PM

@wjhorne
Copy link
Contributor Author

wjhorne commented Nov 21, 2024

They will not be completely the same for static cases since the volume flow rate includes pressure gradient terms in Nalu-Wind. This is due to Nalu-Wind being collocated and requiring pressure terms in the volume flow rate. Using straight velocity, as is done here, removes all pressure and mesh velocity influences

@mbkuhn
Copy link
Contributor

mbkuhn commented Nov 21, 2024

Sounds good, let's go ahead with it, then. In this static case, the interface is thicker in the center because there are smaller spurious currents there, but that is just the nature of the case. In the moving case, it made a clear improvement to the consistent thickness of the interface.

@mbkuhn mbkuhn merged commit 0b4372f into Exawind:master Nov 21, 2024
4 checks passed
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