Skip to content

[TwoFluidExample] Fix frozen free surface in two_fluid_wave - #161

Merged
loumalouomega merged 1 commit into
masterfrom
TwoFluidExample
Oct 21, 2026
Merged

loumalouomega merged 1 commit into
masterfrom
TwoFluidExample

Conversation

@NickNick9

Copy link
Copy Markdown
Contributor

Hi @loumalouomega , I’m new to two-fluid simulations and tried the two_fluid_wave example. The water surface stayed frozen, but adding "eulerian_fm_ale": false made the wave move. It seems the default setting makes VELOCITY − MESH_VELOCITY zero. Could this be a bug, or am I missing something? Could you confirm whether this change is appropriate?

@NickNick9
NickNick9 requested review from loumalouomega and a balanced review from Copilot October 21, 2026 08:33

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

An organization owner can choose the repository organization as the billing source for reviews requested by members with a paid Copilot license. Learn about Copilot code review billing.

@loumalouomega

Copy link
Copy Markdown
Member

Well, it is not new, I just pushed a fix of the scripts that were old and weren't working.

@loumalouomega
loumalouomega merged commit f5b4a92 into master Oct 21, 2026
1 check passed
@loumalouomega
loumalouomega deleted the TwoFluidExample branch October 21, 2026 08:43
@NickNick9

Copy link
Copy Markdown
Contributor Author

Thanks for merging!
Btw, I’m new not the example! 🙂

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.

3 participants