Skip to content

Commit 54a0c69

Browse files
committed
Revert implicit class data switch.
1 parent 31d7a95 commit 54a0c69

5 files changed

Lines changed: 10 additions & 6 deletions

File tree

SU2_CFD/include/solvers/CScalarSolver.hpp

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -57,7 +57,6 @@ class CScalarSolver : public CSolver {
5757

5858
const bool Conservative; /*!< \brief Transported Variable is conservative. Solution has to be multiplied with rho. */
5959

60-
bool implicit; /*!< \brief Solver uses an implicit scheme. */
6160
bool muscl; /*!< \brief Solver uses MUSCL reconstruction. */
6261
bool limiter; /*!< \brief Solver uses limiters. */
6362

SU2_CFD/include/solvers/CScalarSolver.inl

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -79,6 +79,7 @@ CScalarSolver<VariableType>::~CScalarSolver() {
7979

8080
template <class VariableType>
8181
void CScalarSolver<VariableType>::CommonPreprocessing(CGeometry *geometry, const CConfig *config, const bool Output) {
82+
const bool implicit = (config->GetKind_TimeIntScheme() == EULER_IMPLICIT);
8283

8384
/*--- Clear residual and system matrix, not needed for
8485
* reducer strategy as we write over the entire matrix. ---*/
@@ -113,6 +114,8 @@ template <class VariableType>
113114
void CScalarSolver<VariableType>::Upwind_Residual(CGeometry* geometry, CSolver** solver_container,
114115
CNumerics** numerics_container, CConfig* config,
115116
unsigned short iMesh) {
117+
const bool implicit = (config->GetKind_TimeIntScheme() == EULER_IMPLICIT);
118+
116119
/*--- Only reconstruct flow variables if MUSCL is on for flow (requires upwind) and turbulence. ---*/
117120
const bool musclFlow = config->GetMUSCL_Flow() && muscl && (config->GetKind_ConvNumScheme_Flow() == SPACE_UPWIND);
118121
/*--- Only consider flow limiters for cell-based limiters, edge-based would need to be recomputed. ---*/
@@ -455,6 +458,7 @@ template <class VariableType>
455458
void CScalarSolver<VariableType>::SetResidual_DualTime(CGeometry* geometry, CSolver** solver_container, CConfig* config,
456459
unsigned short iRKStep, unsigned short iMesh,
457460
unsigned short RunTime_EqSystem) {
461+
const bool implicit = (config->GetKind_TimeIntScheme() == EULER_IMPLICIT);
458462
const bool first_order = (config->GetTime_Marching() == TIME_MARCHING::DT_STEPPING_1ST);
459463
const bool second_order = (config->GetTime_Marching() == TIME_MARCHING::DT_STEPPING_2ND);
460464
const bool incompressible = (config->GetKind_Regime() == ENUM_REGIME::INCOMPRESSIBLE);

SU2_CFD/src/solvers/CSpeciesSolver.cpp

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -58,8 +58,7 @@ CSpeciesSolver::CSpeciesSolver(CGeometry* geometry, CConfig* config, unsigned sh
5858

5959
nDim = geometry->GetnDim();
6060

61-
/*--- Set whether MUSCL and limiters are used in this solver, and implicit variable ---*/
62-
implicit = (config->GetKind_TimeIntScheme_Species() == EULER_IMPLICIT);
61+
/*--- Set whether MUSCL and limiters are used in this solver. ---*/
6362
muscl = config->GetMUSCL_Species();
6463
limiter = (config->GetKind_SlopeLimit_Species() != NO_LIMITER) &&
6564
(config->GetInnerIter() <= config->GetLimiterIter());
@@ -396,6 +395,7 @@ void CSpeciesSolver::BC_Inlet(CGeometry* geometry, CSolver** solver_container, C
396395
LinSysRes.AddBlock(iPoint, residual);
397396

398397
/*--- Jacobian contribution for implicit integration ---*/
398+
const bool implicit = (config->GetKind_TimeIntScheme() == EULER_IMPLICIT);
399399
if (implicit) Jacobian.AddBlock2Diag(iPoint, residual.jacobian_i);
400400

401401
// Unfinished viscous contribution removed before right after d8a0da9a00. Further testing required.
@@ -540,6 +540,7 @@ void CSpeciesSolver::BC_Outlet(CGeometry* geometry, CSolver** solver_container,
540540
LinSysRes.AddBlock(iPoint, residual);
541541

542542
/*--- Jacobian contribution for implicit integration ---*/
543+
const bool implicit = (config->GetKind_TimeIntScheme() == EULER_IMPLICIT);
543544
if (implicit) Jacobian.AddBlock2Diag(iPoint, residual.jacobian_i);
544545

545546
// Unfinished viscous contribution removed before right after d8a0da9a00. Further testing required.

SU2_CFD/src/solvers/CTurbSolver.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -38,7 +38,6 @@ CTurbSolver::CTurbSolver(CGeometry* geometry, CConfig *config, bool conservative
3838
/*--- Store if an implicit scheme is used, for use during periodic boundary conditions. ---*/
3939
SetImplicitPeriodic(config->GetKind_TimeIntScheme_Turb() == EULER_IMPLICIT);
4040

41-
implicit = (config->GetKind_TimeIntScheme_Turb() == EULER_IMPLICIT);
4241
muscl = config->GetMUSCL_Turb();
4342
limiter = (config->GetKind_SlopeLimit_Turb() != NO_LIMITER) &&
4443
(config->GetInnerIter() <= config->GetLimiterIter());
@@ -337,6 +336,7 @@ void CTurbSolver::LoadRestart(CGeometry** geometry, CSolver*** solver, CConfig*
337336
}
338337

339338
void CTurbSolver::Impose_Fixed_Values(const CGeometry *geometry, const CConfig *config){
339+
const bool implicit = (config->GetKind_TimeIntScheme() == EULER_IMPLICIT);
340340

341341
/*--- Check whether turbulence quantities are fixed to far-field values on a half-plane. ---*/
342342
if(config->GetTurb_Fixed_Values()){

config_template.cfg

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1500,8 +1500,8 @@ CONVEXITY_CHECK_DEPTH = 3
15001500
% 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0, 0.0)
15011501
FFD_DEFINITION= (MAIN_BOX, 0.5, 0.25, -0.25, 1.5, 0.25, -0.25, 1.5, 0.75, -0.25, 0.5, 0.75, -0.25, 0.5, 0.25, 0.25, 1.5, 0.25, 0.25, 1.5, 0.75, 0.25, 0.5, 0.75, 0.25)
15021502
%
1503-
% FFD box degree: 3D case (x_degree, y_degree, z_degree)
1504-
% 2D case (x_degree, y_degree, 0)
1503+
% FFD box degree: 3D case (i_degree, j_degree, k_degree)
1504+
% 2D case (i_degree, j_degree, 0)
15051505
FFD_DEGREE= (10, 10, 1)
15061506
%
15071507
% Surface grid continuity at the intersection with the faces of the FFD boxes.

0 commit comments

Comments
 (0)