Skip to content

Conversation

@TobiKattmann
Copy link
Contributor

Goes along with su2code/su2code.github.io/pull/26.

5 cfg's and 1 mesh file for one cht unsteady case

@oleburghardt
Copy link
Contributor

Thanks. Please hold on a second with this one, some config options in those files are to be replaced in su2code/SU2#1226 :)

Comment on lines 25 to 38
SOLID_TEMPERATURE_INIT= 288.15
%FREESTREAM_TEMPERATURE= 288.15
%
% Solid density (kg/m^3)
SOLID_DENSITY= 0.0210322
%MATERIAL_DENSITY= 0.0210322
%
% Solid specific heat (J/kg*K)
SPECIFIC_HEAT_CP= 1004.703
%
% Solid thermal conductivity (W/m*K)
SOLID_THERMAL_CONDUCTIVITY= 0.1028
%KT_CONSTANT= 0.1028
%
Copy link
Contributor Author

Choose a reason for hiding this comment

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

I already prepared here the cfg option changes for your PR @oleburghardt

Copy link
Contributor Author

Choose a reason for hiding this comment

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

ok i already changed them ahead

Comment on lines +115 to +117
% Max number of iterations of the linear solver for the implicit formulation
LINEAR_SOLVER_ITER= 20
%
Copy link
Contributor Author

Choose a reason for hiding this comment

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

this was 5 iterations before, with this setup you get a vortex shedding, with 5 one doesn't

@TobiKattmann
Copy link
Contributor Author

This is good to go from my point of view. @oleburghardt there is no reason imo to hold back here for su2code/SU2/pull/1226 as these cfg are not yet included in a testcase. I want to move the regression test case to tutorials.py and use these cfgs. If ok i could sneak them in your PR for that. Then it also makes sense to merge this now as otherwise those options would fail...

@TobiKattmann TobiKattmann merged commit 56aedab into develop Mar 18, 2021
@pcarruscag pcarruscag deleted the unsteady_cht_tutorial branch April 30, 2023 21:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants