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

Account for bulk viscosity #58

Closed
ali-ramadhan opened this issue Feb 7, 2020 · 1 comment · Fixed by #60
Closed

Account for bulk viscosity #58

ali-ramadhan opened this issue Feb 7, 2020 · 1 comment · Fixed by #60

Comments

@ali-ramadhan
Copy link
Owner

We should probably be including the effects of the bulk viscosity in the momentum equation: the ⅓μ∇(∇⋅u) term.

See: https://en.wikipedia.org/wiki/Navier%E2%80%93Stokes_equations#Compressible_flow

Probably doesn't matter for the simpler verification tests we've carried out. Also not sure if it's included (or accounted for) with LES. Will have to investigate more.

@ali-ramadhan ali-ramadhan changed the title Account for bulk viscosity in constant viscosity closure Account for bulk viscosity Feb 7, 2020
@thabbott
Copy link
Collaborator

thabbott commented Feb 7, 2020 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants