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 Sutherland's law to compressible Navier-Stokes #1195

Closed
jlchan opened this issue Aug 8, 2022 · 5 comments · Fixed by #1808
Closed

Add Sutherland's law to compressible Navier-Stokes #1195

jlchan opened this issue Aug 8, 2022 · 5 comments · Fixed by #1808
Labels
enhancement New feature or request

Comments

@jlchan
Copy link
Contributor

jlchan commented Aug 8, 2022

Use Sutherland's law to get the molecular and thermal diffusivity in CompressibleNavierStokesDiffusion2D.

Originally posted by @sloede in #1156 (comment)

@DanielDoehring
Copy link
Contributor

DanielDoehring commented Jan 19, 2024

I planned to look into this for the aerodynamic simulations.

Since in that case $\mu = \mu\big(T(\boldsymbol x)\big)$ the derivatives (gradient & divergence) complicate due to the chain rule.
Is this in general accounted for, or do people use only temperature dependent viscosity in the flux, but drop the dependence on space?

@andrewwinters5000
Copy link
Member

do people use only temperature dependent viscosity in the flux, but drop the dependence on space?

In FLUXO we only use a temperature dependent viscosity see here.

@DanielDoehring
Copy link
Contributor

I.e., the only real change would be re-computing the dynamic viscosity

https://github.com/trixi-framework/Trixi.jl/pull/1808/files#diff-8e1a56d54ddffada1448f2d79a97dcc9ee1d5ea3e91e96311d8bf7f65c968114R358-R359

and leaving the rest untouched?

@andrewwinters5000
Copy link
Member

Yes, I believe this should be all that is necessary.

@DanielDoehring
Copy link
Contributor

DanielDoehring commented Jan 23, 2024

Question is if we want to use the standard version
$$
\mu (T) = \mu_0 \frac{T_0 + C}{T + C} \bigg ( \frac{T}{T_0} \bigg )^{ \frac{3}{2} }
$$

or the extended one present in FLUXO.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants