Skip to content

Commit 0e198fc

Browse files
committed
initial cleanup commit
1 parent d5d1e27 commit 0e198fc

9 files changed

Lines changed: 27 additions & 44 deletions

File tree

Common/src/CConfig.cpp

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -5796,14 +5796,14 @@ void CConfig::SetOutput(SU2_COMPONENT val_software, unsigned short val_izone) {
57965796
break;
57975797
case MAIN_SOLVER::NEMO_EULER:
57985798
if (Kind_Regime == ENUM_REGIME::COMPRESSIBLE) cout << "Compressible two-temperature thermochemical non-equilibrium Euler equations." << endl;
5799-
if(Kind_FluidModel == SU2_NONEQ){
5799+
if (Kind_FluidModel == SU2_NONEQ){
58005800
if ((GasModel != "N2") && (GasModel != "AIR-5") && (GasModel != "ARGON"))
58015801
SU2_MPI::Error("The GAS_MODEL given as input is not valid. Choose one of the options: N2, AIR-5, ARGON.", CURRENT_FUNCTION);
58025802
}
58035803
break;
58045804
case MAIN_SOLVER::NEMO_NAVIER_STOKES:
58055805
if (Kind_Regime == ENUM_REGIME::COMPRESSIBLE) cout << "Compressible two-temperature thermochemical non-equilibrium Navier-Stokes equations." << endl;
5806-
if(Kind_FluidModel == SU2_NONEQ){
5806+
if (Kind_FluidModel == SU2_NONEQ){
58075807
if ((GasModel != "N2") && (GasModel != "AIR-5") && (GasModel != "ARGON"))
58085808
SU2_MPI::Error("The GAS_MODEL given as input is not valid. Choose one of the options: N2, AIR-5, ARGON.", CURRENT_FUNCTION);
58095809
}

SU2_CFD/include/output/CNEMOCompOutput.hpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -45,7 +45,7 @@ class CNEMOCompOutput final: public CFlowOutput {
4545
* \brief Constructor of the class
4646
* \param[in] config - Definition of the particular problem.
4747
*/
48-
CNEMOCompOutput(CConfig *config, unsigned short nDim);
48+
CNEMOCompOutput(const CConfig *config, unsigned short nDim);
4949

5050
/*!
5151
* \brief Load the history output field values

SU2_CFD/include/solvers/CSolver.hpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1253,7 +1253,7 @@ class CSolver {
12531253
* \param[in] val_marker - Surface marker where the boundary condition is applied.
12541254
*/
12551255
inline virtual void BC_Smoluchowski_Maxwell(CGeometry *geometry,
1256-
CSolver **solution_container,
1256+
CSolver **solver_container,
12571257
CNumerics *conv_numerics,
12581258
CNumerics *visc_numerics,
12591259
CConfig *config,

SU2_CFD/src/drivers/CDriver.cpp

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3123,7 +3123,6 @@ bool CFluidDriver::Monitor(unsigned long ExtIter) {
31233123

31243124
switch (config_container[ZONE_0]->GetKind_Solver()) {
31253125
case MAIN_SOLVER::EULER: case MAIN_SOLVER::NAVIER_STOKES: case MAIN_SOLVER::RANS:
3126-
StopCalc = integration_container[ZONE_0][INST_0][FLOW_SOL]->GetConvergence(); break;
31273126
case MAIN_SOLVER::NEMO_EULER: case MAIN_SOLVER::NEMO_NAVIER_STOKES:
31283127
StopCalc = integration_container[ZONE_0][INST_0][FLOW_SOL]->GetConvergence(); break;
31293128
case MAIN_SOLVER::HEAT_EQUATION:

SU2_CFD/src/fluid/CSU2TCLib.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -781,7 +781,7 @@ vector<su2double>& CSU2TCLib::ComputeSpeciesEve(su2double val_T, bool vibe_only)
781781
}
782782
Eel = Ru/MolarMass[iSpecies] * (num/denom);
783783
}
784-
if(vibe_only == true) {eves[iSpecies] = Ev;}
784+
if (vibe_only == true) {eves[iSpecies] = Ev;}
785785
else {eves[iSpecies] = Ev + Eel;}
786786
}
787787

SU2_CFD/src/output/CNEMOCompOutput.cpp

Lines changed: 9 additions & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -4,22 +4,12 @@
44
* \author W. Maier, R. Sanchez
55
* \version 7.2.1 "Blackbird"
66
*
7-
* The current SU2 release has been coordinated by the
8-
* SU2 International Developers Society <www.su2devsociety.org>
9-
* with selected contributions from the open-source community.
7+
* SU2 Project Website: https://su2code.github.io
108
*
11-
* The main research teams contributing to the current release are:
12-
* - Prof. Juan J. Alonso's group at Stanford University.
13-
* - Prof. Piero Colonna's group at Delft University of Technology.
14-
* - Prof. Nicolas R. Gauger's group at Kaiserslautern University of Technology.
15-
* - Prof. Alberto Guardone's group at Polytechnic University of Milan.
16-
* - Prof. Rafael Palacios' group at Imperial College London.
17-
* - Prof. Vincent Terrapon's group at the University of Liege.
18-
* - Prof. Edwin van der Weide's group at the University of Twente.
19-
* - Lab. of New Concepts in Aeronautics at Tech. Institute of Aeronautics.
9+
* The SU2 Project is maintained by the SU2 Foundation
10+
* (http://su2foundation.org)
2011
*
21-
* Copyright 2012-2018, Francisco D. Palacios, Thomas D. Economon,
22-
* Tim Albring, and the SU2 contributors.
12+
* Copyright 2012-2021, SU2 Contributors (cf. AUTHORS.md)
2313
*
2414
* SU2 is free software; you can redistribute it and/or
2515
* modify it under the terms of the GNU Lesser General Public
@@ -40,9 +30,9 @@
4030
#include "../../../Common/include/geometry/CGeometry.hpp"
4131
#include "../../include/solvers/CSolver.hpp"
4232

43-
CNEMOCompOutput::CNEMOCompOutput(CConfig *config, unsigned short nDim) : CFlowOutput(config, nDim, false) {
33+
CNEMOCompOutput::CNEMOCompOutput(const CConfig *config, unsigned short nDim) : CFlowOutput(config, nDim, false) {
4434

45-
turb_model = config->GetKind_Turb_Model();
35+
turb_model = config->GetKind_Turb_Model();
4636
nSpecies = config->GetnSpecies();
4737

4838
/*--- Set the default history fields if nothing is set in the config file ---*/
@@ -118,6 +108,7 @@ CNEMOCompOutput::CNEMOCompOutput(CConfig *config, unsigned short nDim) : CFlowOu
118108
void CNEMOCompOutput::SetHistoryOutputFields(CConfig *config){
119109

120110
/// BEGIN_GROUP: RMS_RES, DESCRIPTION: The root-mean-square residuals of the SOLUTION variables.
111+
/// DESCRIPTION: Root-mean square residual of the species densities.
121112
for(iSpecies = 0; iSpecies < nSpecies; iSpecies++)
122113
AddHistoryOutput("RMS_DENSITY_" + std::to_string(iSpecies), "rms[Rho_" + std::to_string(iSpecies) + "]", ScreenOutputFormat::FIXED, "RMS_RES", "Root-mean square residual of the species density " + std::to_string(iSpecies) + ".", HistoryFieldType::RESIDUAL);
123114
/// DESCRIPTION: Root-mean square residual of the momentum x-component.
@@ -279,7 +270,7 @@ void CNEMOCompOutput::SetVolumeOutputFields(CConfig *config){
279270
AddVolumeOutput("MACH", "Mach", "PRIMITIVE", "Mach number");
280271
AddVolumeOutput("PRESSURE_COEFF", "Pressure_Coefficient", "PRIMITIVE", "Pressure coefficient");
281272

282-
if (config->GetKind_Solver() == MAIN_SOLVER::NEMO_NAVIER_STOKES){
273+
if (config->GetViscous()) {
283274
AddVolumeOutput("LAMINAR_VISCOSITY", "Laminar_Viscosity", "PRIMITIVE", "Laminar viscosity");
284275

285276
AddVolumeOutput("SKIN_FRICTION-X", "Skin_Friction_Coefficient_x", "PRIMITIVE", "x-component of the skin friction vector");
@@ -367,7 +358,7 @@ void CNEMOCompOutput::LoadVolumeData(CConfig *config, CGeometry *geometry, CSolv
367358
su2double factor = 1.0/(0.5*solver[FLOW_SOL]->GetDensity_Inf()*VelMag);
368359
SetVolumeOutputValue("PRESSURE_COEFF", iPoint, (Node_Flow->GetPressure(iPoint) - solver[FLOW_SOL]->GetPressure_Inf())*factor);
369360

370-
if (config->GetKind_Solver() == MAIN_SOLVER::NEMO_NAVIER_STOKES){
361+
if (config->GetViscous()){
371362
SetVolumeOutputValue("LAMINAR_VISCOSITY", iPoint, Node_Flow->GetLaminarViscosity(iPoint));
372363
}
373364

SU2_CFD/src/solvers/CNEMOEulerSolver.cpp

Lines changed: 1 addition & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -1553,7 +1553,7 @@ void CNEMOEulerSolver::BC_Far_Field(CGeometry *geometry, CSolver **solver_contai
15531553
unsigned short iDim;
15541554
unsigned long iVertex, iPoint, Point_Normal;
15551555

1556-
su2double *V_infty, *V_domain, *U_domain,*U_infty;
1556+
su2double *V_infty, *V_domain;
15571557

15581558
/*--- Set booleans from configuration parameters ---*/
15591559
bool implicit = (config->GetKind_TimeIntScheme() == EULER_IMPLICIT);
@@ -1578,13 +1578,10 @@ void CNEMOEulerSolver::BC_Far_Field(CGeometry *geometry, CSolver **solver_contai
15781578
conv_numerics->SetNormal(Normal);
15791579

15801580
/*--- Retrieve solution at the boundary node & free-stream ---*/
1581-
U_domain = nodes->GetSolution(iPoint);
15821581
V_domain = nodes->GetPrimitive(iPoint);
1583-
U_infty = node_infty->GetSolution(0);
15841582
V_infty = node_infty->GetPrimitive(0);
15851583

15861584
/*--- Pass conserved & primitive variables to CNumerics ---*/
1587-
conv_numerics->SetConservative(U_domain, U_infty);
15881585
conv_numerics->SetPrimitive(V_domain, V_infty);
15891586

15901587
/*--- Pass supplementary information to CNumerics ---*/
@@ -1614,8 +1611,6 @@ void CNEMOEulerSolver::BC_Far_Field(CGeometry *geometry, CSolver **solver_contai
16141611
visc_numerics->SetNormal(Normal);
16151612

16161613
/*--- Primitive variables, and gradient ---*/
1617-
visc_numerics->SetConservative(nodes->GetSolution(iPoint),
1618-
node_infty->GetSolution(0) );
16191614
visc_numerics->SetPrimitive(nodes->GetPrimitive(iPoint),
16201615
node_infty->GetPrimitive(0) );
16211616
visc_numerics->SetPrimVarGradient(nodes->GetGradient_Primitive(iPoint),
@@ -2403,7 +2398,6 @@ void CNEMOEulerSolver::BC_Supersonic_Outlet(CGeometry *geometry, CSolver **solve
24032398
unsigned short iDim;
24042399
unsigned long iVertex, iPoint;
24052400
su2double *V_outlet, *V_domain;
2406-
su2double *U_outlet, *U_domain;
24072401

24082402
bool implicit = (config->GetKind_TimeIntScheme() == EULER_IMPLICIT);
24092403
bool dynamic_grid = config->GetGrid_Movement();
@@ -2424,11 +2418,9 @@ void CNEMOEulerSolver::BC_Supersonic_Outlet(CGeometry *geometry, CSolver **solve
24242418

24252419
/*--- Current solution at this boundary node ---*/
24262420
V_domain = nodes->GetPrimitive(iPoint);
2427-
U_domain = nodes->GetSolution(iPoint);
24282421

24292422
/*--- Allocate the value at the outlet ---*/
24302423
V_outlet = V_domain;
2431-
U_outlet = U_domain;
24322424

24332425
/*--- Normal vector for this vertex (negate for outward convention) ---*/
24342426
geometry->vertex[val_marker][iVertex]->GetNormal(Normal);
@@ -2437,7 +2429,6 @@ void CNEMOEulerSolver::BC_Supersonic_Outlet(CGeometry *geometry, CSolver **solve
24372429
/*--- Set various quantities in the solver class ---*/
24382430
conv_numerics->SetNormal(Normal);
24392431
conv_numerics->SetPrimitive(V_domain, V_outlet);
2440-
conv_numerics->SetConservative(U_domain, U_outlet);
24412432

24422433
/*--- Pass supplementary information to CNumerics ---*/
24432434
conv_numerics->SetdPdU (nodes->GetdPdU(iPoint), nodes->GetdPdU(iPoint));

SU2_CFD/src/solvers/CNEMONSSolver.cpp

Lines changed: 8 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -96,8 +96,12 @@ void CNEMONSSolver::Preprocessing(CGeometry *geometry, CSolver **solver_containe
9696
SetPrimitive_Limiter(geometry, config);
9797
}
9898

99+
/*--- Compute vorticity and strain mag. ---*/
100+
99101
ComputeVorticityAndStrainMag(*config, iMesh);
100102

103+
/*--- Compute the TauWall from the wall functions ---*/
104+
101105
if (wall_functions) {
102106
SetTau_Wall_WF(geometry, solver_container, config);
103107
}
@@ -319,10 +323,9 @@ void CNEMONSSolver::BC_HeatFluxNonCatalytic_Wall(CGeometry *geometry,
319323
su2double zero[MAXNDIM] = {0.0};
320324
nodes->SetVelocity_Old(iPoint, zero);
321325

322-
for (auto iDim = 0u; iDim < nDim; iDim++){
326+
for (auto iDim = 0u; iDim < nDim; iDim++)
323327
LinSysRes(iPoint, nSpecies+iDim) = 0.0;
324-
nodes->SetVal_ResTruncError_Zero(iPoint,nSpecies+iDim);
325-
}
328+
nodes->SetVel_ResTruncError_Zero(iPoint);
326329

327330
/*--- Apply viscous residual to the linear system ---*/
328331
LinSysRes.SubtractBlock(iPoint, Res_Visc);
@@ -635,10 +638,9 @@ void CNEMONSSolver::BC_IsothermalNonCatalytic_Wall(CGeometry *geometry,
635638
/*--- Initialize viscous residual to zero ---*/
636639
for (auto iVar = 0u; iVar < nVar; iVar ++) {Res_Visc[iVar] = 0.0;}
637640

638-
for (auto iDim = 0u; iDim < nDim; iDim++){
641+
for (auto iDim = 0u; iDim < nDim; iDim++)
639642
LinSysRes(iPoint, nSpecies+iDim) = 0.0;
640-
nodes->SetVal_ResTruncError_Zero(iPoint,nSpecies+iDim);
641-
}
643+
nodes->SetVel_ResTruncError_Zero(iPoint);
642644

643645
/*--- Calculate the gradient of temperature ---*/
644646
su2double Ti = nodes->GetTemperature(iPoint);

SU2_CFD/src/variables/CNEMONSVariable.cpp

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -78,7 +78,7 @@ CNEMONSVariable::CNEMONSVariable(su2double val_pressure,
7878

7979
bool CNEMONSVariable::SetPrimVar(unsigned long iPoint, CFluidModel *FluidModel) {
8080

81-
unsigned long iVar, iSpecies;
81+
8282

8383
fluidmodel = static_cast<CNEMOGas*>(FluidModel);
8484

@@ -87,7 +87,7 @@ bool CNEMONSVariable::SetPrimVar(unsigned long iPoint, CFluidModel *FluidModel)
8787

8888
/*--- Reset solution to previous one, if nonphys ---*/
8989
if (nonPhys) {
90-
for (iVar = 0; iVar < nVar; iVar++)
90+
for (auto iVar = 0u; iVar < nVar; iVar++)
9191
Solution(iPoint,iVar) = Solution_Old(iPoint,iVar);
9292

9393
/*--- Recompute Primitive from previous solution ---*/
@@ -100,15 +100,15 @@ bool CNEMONSVariable::SetPrimVar(unsigned long iPoint, CFluidModel *FluidModel)
100100
SetVelocity2(iPoint);
101101

102102
const auto& Ds = fluidmodel->GetDiffusionCoeff();
103-
for (iSpecies = 0; iSpecies < nSpecies; iSpecies++)
103+
for (auto iSpecies = 0u; iSpecies < nSpecies; iSpecies++)
104104
DiffusionCoeff(iPoint, iSpecies) = Ds[iSpecies];
105105

106106
su2double T = Primitive(iPoint,nSpecies);
107107
su2double Tve = Primitive(iPoint,nSpecies+1);
108108

109109
su2double* val_eves = GetEve(iPoint);
110110
const auto& hs = fluidmodel->ComputeSpeciesEnthalpy(T, Tve, val_eves);
111-
for (iSpecies = 0; iSpecies < nSpecies; iSpecies++)
111+
for (auto iSpecies = 0u; iSpecies < nSpecies; iSpecies++)
112112
Enthalpys(iPoint, iSpecies) = hs[iSpecies];
113113

114114
LaminarViscosity(iPoint) = fluidmodel->GetViscosity();

0 commit comments

Comments
 (0)