Skip to content
Open
Show file tree
Hide file tree
Changes from 19 commits
Commits
Show all changes
30 commits
Select commit Hold shift + click to select a range
11a88fb
Merge branch 'develop' of https://github.com/su2code/SU2 into feature…
EvertBunschoten Feb 17, 2026
6ba1ffb
fix trailing whitespace
EvertBunschoten Feb 17, 2026
5dca658
Merge branch 'develop' into feature_flame_lengthscale
EvertBunschoten Feb 19, 2026
2df59a8
Added function for thickened flame correction factor
EvertBunschoten Feb 20, 2026
3af0712
Diffusivity is multiplied and source terms are divided by correction …
EvertBunschoten Feb 20, 2026
b575044
Merge branch 'develop' of https://github.com/su2code/SU2 into feature…
EvertBunschoten Feb 20, 2026
e361ca7
Apply source correction only for steady problems
EvertBunschoten Feb 20, 2026
0de7e0d
bug fix
EvertBunschoten Feb 20, 2026
08573ce
Merge branch 'develop' into feature_flame_lengthscale
EvertBunschoten Feb 23, 2026
ffe779b
Merge branch 'develop' into feature_flame_lengthscale
EvertBunschoten Mar 3, 2026
ff6faea
NULL MLP no longer needed
EvertBunschoten Mar 19, 2026
5a9d2a0
Added general flame thickness calculation method
EvertBunschoten Mar 19, 2026
d9e5b59
Added default flame thickness option
EvertBunschoten Mar 19, 2026
dde7b7b
Merge branch 'develop' of https://github.com/su2code/SU2 into feature…
EvertBunschoten Mar 19, 2026
6963939
Removed flame lengthscale option and automatically approximate flame …
EvertBunschoten Mar 20, 2026
e8b31f9
Include flame thickness in convergence history
EvertBunschoten Mar 20, 2026
5dee848
precommit formatting
EvertBunschoten Mar 20, 2026
1010429
code factor
EvertBunschoten Mar 20, 2026
92eec69
Merge branch 'develop' into feature_flame_lengthscale
bigfooted Mar 23, 2026
98de6ac
Applied GitHub code suggestion
EvertBunschoten Mar 25, 2026
4ff73c6
Improved description of coarse grid correction
EvertBunschoten Mar 25, 2026
746a543
Formatting suggestions Pedro
EvertBunschoten Mar 25, 2026
7cdd149
Flame thickness in history output now defined as a user-defined coeff…
EvertBunschoten Mar 25, 2026
7015adc
const correction
EvertBunschoten Mar 25, 2026
671c8c3
Single allreduce of four variables instead of four separate allreduce…
EvertBunschoten Mar 25, 2026
566618e
Geometrytoolbox for gradient projection
EvertBunschoten Mar 25, 2026
8f1f2a1
Added ignition temperature option
EvertBunschoten Mar 25, 2026
d6ddccb
Clip global flame thickness to one meter maximum.
EvertBunschoten Mar 25, 2026
a27d833
Merge branch 'feature_flame_lengthscale' of https://github.com/su2cod…
EvertBunschoten Mar 25, 2026
e9d88dc
precommit
EvertBunschoten Mar 25, 2026
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 1 addition & 0 deletions Common/include/option_structure.hpp
Original file line number Diff line number Diff line change
Expand Up @@ -1446,6 +1446,7 @@ struct FluidFlamelet_ParsedOptions {
su2double* spark_reaction_rates; /*!< \brief Source terms for flamelet spark ignition option. */
unsigned short nspark; /*!< \brief Number of source terms for spark initialization. */
bool preferential_diffusion = false; /*!< \brief Preferential diffusion physics for flamelet solver.*/
bool thickenedflame_correction{true}; /*!< \brief Thickened flame correction. */
Comment thread
EvertBunschoten marked this conversation as resolved.
};

/*!
Expand Down
3 changes: 3 additions & 0 deletions Common/src/CConfig.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -1429,6 +1429,9 @@ void CConfig::SetConfig_Options() {
/*!\brief SPARK_REACTION_RATES \n DESCRIPTION: Net source term values applied to species within spark area during spark ignition. \ingroup Config*/
addDoubleListOption("SPARK_REACTION_RATES", flamelet_ParsedOptions.nspark, flamelet_ParsedOptions.spark_reaction_rates);

/*!\brief THICKENED_FLAME_CORRECTION \n DESCRIPTION: Dampen source terms and enhance diffusion based on the flame length scale. \ingroup Config*/
Comment thread
EvertBunschoten marked this conversation as resolved.
Outdated
addBoolOption("THICKENED_FLAME_CORRECTION", flamelet_ParsedOptions.thickenedflame_correction, true);

/*--- Options related to mass diffusivity and thereby the species solver. ---*/

/*!\brief DIFFUSIVITY_MODEL\n DESCRIPTION: mass diffusivity model \n DEFAULT constant disffusivity \ingroup Config*/
Expand Down
6 changes: 6 additions & 0 deletions SU2_CFD/include/solvers/CSolver.hpp
Original file line number Diff line number Diff line change
Expand Up @@ -580,6 +580,12 @@ class CSolver {
*/
inline virtual void SetPrimitive_Limiter(CGeometry *geometry, const CConfig *config) { }

/*!
* \brief A virtual member.
* \return flame thickness value.
*/
virtual su2double GetFlameThickness() const {return 1.0;}

/*!
* \brief Compute the projection of a variable for MUSCL reconstruction.
* \note The result should be halved when added to i (or subtracted from j).
Expand Down
28 changes: 27 additions & 1 deletion SU2_CFD/include/solvers/CSpeciesFlameletSolver.hpp
Original file line number Diff line number Diff line change
Expand Up @@ -37,6 +37,9 @@
*/
class CSpeciesFlameletSolver final : public CSpeciesSolver {
private:
const su2double default_flame_thickness{1.0};
su2double global_flame_thickness;
bool calc_flame_thickness{false};
FluidFlamelet_ParsedOptions flamelet_config_options;
bool include_mixture_fraction = false; /*!< \brief include mixture fraction as a controlling variable. */
/*!
Expand Down Expand Up @@ -81,10 +84,11 @@ class CSpeciesFlameletSolver final : public CSpeciesSolver {
* \param[in] iPoint - node ID.
* \param[in] scalars - local scalar solution.
* \param[in] table_source_names - variable names of scalar source terms.
* \param[in] F - flame thickness correction factor.
* \return - within manifold bounds (0) or outside manifold bounds (1).
*/
unsigned long SetScalarSources(const CConfig* config, CFluidModel* fluid_model_local, unsigned long iPoint,
const vector<su2double>& scalars);
const vector<su2double>& scalars, const su2double F=1.0);

/*!
* \brief Retrieve passive look-up data from manifold.
Expand All @@ -105,6 +109,22 @@ class CSpeciesFlameletSolver final : public CSpeciesSolver {
*/
unsigned long SetPreferentialDiffusionScalars(CFluidModel* fluid_model_local,
unsigned long iPoint, const vector<su2double>& scalars);

/*!
* \brief Calculate correction factor for flame propagation on coarse grids.
* \param[in] geometry - Geometrical definition of the problem.
* \param[in] iPoint - node ID.
* \return - flame thickness correction factor.
*/
su2double ThickenedFlameCorrection(CGeometry const * geometry, const unsigned long iPoint) const;
Comment thread
EvertBunschoten marked this conversation as resolved.
Outdated

/*!
* \brief Approximate the minimum flame thickness value used for the thickened flame model.
* \param[in] geometry - Geometrical definition of the problem.
* \param[in] solver_container - Container vector with all the solutions.
* \return - approximate flame thickness value.
*/
su2double GetOverallFlameThickness(CGeometry * geometry, CSolver **solver_container) const;
Comment thread
EvertBunschoten marked this conversation as resolved.
Outdated

public:
/*!
Expand Down Expand Up @@ -195,4 +215,10 @@ class CSpeciesFlameletSolver final : public CSpeciesSolver {
*/
void Viscous_Residual(const unsigned long iEdge, const CGeometry* geometry, CSolver** solver_container, CNumerics* numerics,
const CConfig* config) final;

/*!
* \brief Obtain the overall flame thickness value.
* \return flame thickness value.
*/
virtual su2double GetFlameThickness() const {return global_flame_thickness;}
Comment thread
EvertBunschoten marked this conversation as resolved.
Outdated
};
5 changes: 5 additions & 0 deletions SU2_CFD/src/output/CFlowOutput.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -1069,6 +1069,8 @@ void CFlowOutput::AddHistoryOutputFields_ScalarRMS_RES(const CConfig* config) {
const auto& CV_name = flamelet_config_options.controlling_variable_names[iCV];
AddHistoryOutput("RMS_"+CV_name, "rms["+CV_name+"]",ScreenOutputFormat::FIXED, "RMS_RES", "Root-mean squared residual of " + CV_name + " controlling variable equation.", HistoryFieldType::RESIDUAL);
}
if (flamelet_config_options.thickenedflame_correction)
AddHistoryOutput("THICKNESS","flamethickness",ScreenOutputFormat::FIXED, "RMS_RES", "Flame thickness used for thickened flame correction model.", HistoryFieldType::RESIDUAL);
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

It's not a residual

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

I have placed the flame thickness under its own category as a user-defined coefficient.


/*--- auxiliary species transport ---*/
for (auto i_scalar = 0u; i_scalar < flamelet_config_options.n_user_scalars; i_scalar++){
Expand Down Expand Up @@ -1304,6 +1306,9 @@ void CFlowOutput::LoadHistoryDataScalar(const CConfig* config, const CSolver* co
}
}

if (flamelet_config_options.thickenedflame_correction)
SetHistoryOutputValue("THICKNESS", solver[SPECIES_SOL]->GetFlameThickness());

SetHistoryOutputValue("LINSOL_ITER_FLAMELET", solver[SPECIES_SOL]->GetIterLinSolver());
SetHistoryOutputValue("LINSOL_RESIDUAL_FLAMELET", log10(solver[SPECIES_SOL]->GetResLinSolver()));
}
Expand Down
120 changes: 107 additions & 13 deletions SU2_CFD/src/solvers/CSpeciesFlameletSolver.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -39,6 +39,8 @@

/*--- Retrieve options from config. ---*/
flamelet_config_options = config->GetFlameletParsedOptions();
global_flame_thickness = default_flame_thickness;
calc_flame_thickness = flamelet_config_options.thickenedflame_correction;

/*--- Dimension of the problem. ---*/
nVar = flamelet_config_options.n_scalars;
Expand All @@ -65,45 +67,79 @@

/*--- Add the solver name. ---*/
SolverName = "FLAMELET";

if (calc_flame_thickness && rank==MASTER_NODE) {
cout << "Applying thickened flame source and diffusion correction." << endl;
}
}

void CSpeciesFlameletSolver::Preprocessing(CGeometry* geometry, CSolver** solver_container, CConfig* config,
unsigned short iMesh, unsigned short iRKStep,
unsigned short RunTime_EqSystem, bool Output) {
unsigned long n_not_in_domain_local = 0, n_not_in_domain_global = 0;
vector<su2double> scalars_vector(nVar);

unsigned long spark_iter_start, spark_duration;
bool ignition = false;
auto* flowNodes = su2staticcast_p<CFlowVariable*>(solver_container[FLOW_SOL]->GetNodes());

/*--- Retrieve spark ignition parameters for spark-type ignition. ---*/
unsigned long iter;
if (config->GetMultizone_Problem()) {
iter = config->GetOuterIter();
} else if (config->GetTime_Domain()) {
iter = config->GetTimeIter();
} else {
iter = config->GetInnerIter();
}
if ((flamelet_config_options.ignition_method == FLAMELET_INIT_TYPE::SPARK)) {
auto spark_init = flamelet_config_options.spark_init;
spark_iter_start = ceil(spark_init[4]);
spark_duration = ceil(spark_init[5]);
unsigned long iter = config->GetMultizone_Problem() ? config->GetOuterIter() : config->GetInnerIter();

ignition = ((iter >= spark_iter_start) && (iter <= (spark_iter_start + spark_duration)));
}

SU2_OMP_SAFE_GLOBAL_ACCESS(config->SetGlobalParam(config->GetKind_Solver(), RunTime_EqSystem);)

/* Update global flame thickness value. */
if (calc_flame_thickness) {
su2double test_thickness = GetOverallFlameThickness(geometry, solver_container);
if (test_thickness < global_flame_thickness) {
global_flame_thickness = test_thickness;
} else {
global_flame_thickness = 0.95*global_flame_thickness + 0.05*test_thickness;
}
}

/* Flame thickness correction factors */
su2double F{1.0}, F_source{1.0};

SU2_OMP_FOR_STAT(omp_chunk_size)
for (auto i_point = 0u; i_point < nPoint; i_point++) {
CFluidModel* fluid_model_local = solver_container[FLOW_SOL]->GetFluidModel();
su2double* scalars = nodes->GetSolution(i_point);

/*--- Calculate correction factor for flame propagation on coarse grids. ---*/
if (calc_flame_thickness) {
F = ThickenedFlameCorrection(geometry, i_point);
F_source = 1.0 / F;
}

for (auto iVar = 0u; iVar < nVar; iVar++) scalars_vector[iVar] = scalars[iVar];

/*--- Compute total source terms from the production and consumption. ---*/
unsigned long misses = SetScalarSources(config, fluid_model_local, i_point, scalars_vector);
/*--- Only apply thickened flame correction factor to sources for steady problems. ---*/
unsigned long misses = SetScalarSources(config, fluid_model_local, i_point, scalars_vector, F_source);

if (ignition) {
/*--- Apply source terms within spark radius. ---*/
su2double dist_from_center = 0,
spark_radius = flamelet_config_options.spark_init[3];
dist_from_center = GeometryToolbox::SquaredDistance(nDim, geometry->nodes->GetCoord(i_point), flamelet_config_options.spark_init.data());
if (dist_from_center < pow(spark_radius,2)) {
for (auto iVar = 0u; iVar < nVar; iVar++)
nodes->SetScalarSource(i_point, iVar, nodes->GetScalarSources(i_point)[iVar] + flamelet_config_options.spark_reaction_rates[iVar]);
for (auto iVar = 0u; iVar < nVar; iVar++) {
su2double source_total = nodes->GetScalarSources(i_point)[iVar] + F_source * flamelet_config_options.spark_reaction_rates[iVar];
nodes->SetScalarSource(i_point, iVar, source_total);
}
}
}

Expand All @@ -115,9 +151,9 @@
/*--- Set mass diffusivity based on thermodynamic state. ---*/
auto T = flowNodes->GetTemperature(i_point);
fluid_model_local->SetTDState_T(T, scalars);
/*--- set the diffusivity in the fluid model to the diffusivity obtained from the lookup table ---*/
/*--- set the diffusivity in the fluid model to the diffusivity obtained from the lookup table, multiplied by flame thickness correction factor ---*/
for (auto i_scalar = 0u; i_scalar < nVar; ++i_scalar) {
nodes->SetDiffusivity(i_point, fluid_model_local->GetMassDiffusivity(i_scalar), i_scalar);
nodes->SetDiffusivity(i_point, F * (fluid_model_local->GetMassDiffusivity(i_scalar)), i_scalar);
}

/*--- Obtain preferential diffusion scalar values. ---*/
Expand Down Expand Up @@ -213,8 +249,6 @@

for (unsigned long i_mesh = 0; i_mesh <= config->GetnMGLevels(); i_mesh++) {
fluid_model_local = solver_container[i_mesh][FLOW_SOL]->GetFluidModel();
if (flame_front_ignition) prog_burnt = GetBurntProgressVariable(fluid_model_local, scalar_init);

for (auto iVar = 0u; iVar < nVar; iVar++) scalar_init[iVar] = config->GetSpecies_Init()[iVar];

/*--- Set enthalpy based on initial temperature and scalars. ---*/
Expand All @@ -228,6 +262,7 @@

if (flame_front_ignition) {

prog_burnt = GetBurntProgressVariable(fluid_model_local, scalar_init);
/*--- Determine if point is above or below the plane, assuming the normal
is pointing towards the burned region. ---*/
point_loc = 0.0;
Expand Down Expand Up @@ -383,7 +418,7 @@

void CSpeciesFlameletSolver::Source_Residual(CGeometry* geometry, CSolver** solver_container,
CNumerics** numerics_container, CConfig* config, unsigned short iMesh) {

Comment thread
EvertBunschoten marked this conversation as resolved.
Outdated
SU2_OMP_FOR_STAT(omp_chunk_size)
for (auto i_point = 0u; i_point < nPointDomain; i_point++) {
/*--- Add source terms from the lookup table directly to the residual. ---*/
Expand Down Expand Up @@ -516,7 +551,7 @@
}

unsigned long CSpeciesFlameletSolver::SetScalarSources(const CConfig* config, CFluidModel* fluid_model_local,
unsigned long iPoint, const vector<su2double>& scalars) {
unsigned long iPoint, const vector<su2double>& scalars, const su2double F) {
/*--- Compute total source terms from the production and consumption. ---*/

vector<su2double> table_sources(flamelet_config_options.n_control_vars + 2 * flamelet_config_options.n_user_scalars);
Expand All @@ -538,8 +573,9 @@
su2double source_cons = table_sources[flamelet_config_options.n_control_vars + 2 * i_aux + 1];
source_scalar[flamelet_config_options.n_control_vars + i_aux] = source_prod + source_cons * y_aux;
}
/*--- Source term is divided by flame thickness correction factor to improve stability on coarse grids. ---*/
for (auto i_scalar = 0u; i_scalar < nVar; i_scalar++)
nodes->SetScalarSource(iPoint, i_scalar, source_scalar[i_scalar]);
nodes->SetScalarSource(iPoint, i_scalar, F*source_scalar[i_scalar]);
return misses;
}

Expand Down Expand Up @@ -804,3 +840,61 @@
su2double pv_burnt = scalars[I_PROGVAR] - delta;
return pv_burnt;
}


su2double CSpeciesFlameletSolver::ThickenedFlameCorrection(CGeometry const * geometry, const unsigned long iPoint) const {
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

You need to open PRs more often so you don't forge what style we follow in the code 😉

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

More PRs coming in! You refer to the const and pointer placement?

su2double F{1.0};
if (global_flame_thickness != default_flame_thickness) {
Comment thread
github-advanced-security[bot] marked this conversation as resolved.
Fixed
su2double max_flame_vol = pow(global_flame_thickness, nDim);
F = max(1.0, geometry->nodes->GetVolume(iPoint) / max_flame_vol);
}
return F;
}

su2double CSpeciesFlameletSolver::GetOverallFlameThickness(CGeometry * geometry, CSolver **solver_container) const {
CFlowVariable const * flowNodes = su2staticcast_p<CFlowVariable*>(solver_container[FLOW_SOL]->GetNodes());
Comment thread
EvertBunschoten marked this conversation as resolved.
Outdated
su2double pvmax_local{-1e3}, pvmin_local{1e3}, pvmax_global{0.0},pvmin_global{0.0}, gradpv_local{0.0}, gradpv_global{0.0}, Tmax_local{-1e6},Tmax_global{0.0};

SU2_OMP_FOR_STAT(omp_chunk_size)
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This does not work with OpenMP because the local variables are also thread-local, you can take a look at how we compute time steps for the pattern you need to use.

for (auto iPoint = 0u; iPoint < nPoint; iPoint++) {
Comment thread
EvertBunschoten marked this conversation as resolved.
Outdated
su2double pv_local = nodes->GetSolution(iPoint, I_PROGVAR);
su2double T_local = solver_container[FLOW_SOL]->GetNodes()->GetTemperature(iPoint);

/* Parallel projection of progress variable gradient against velocity */
su2double proj_grad_pv_u[MAXNDIM];
for (auto iDim=0u; iDim < nDim; iDim++) {
su2double gradpv = nodes->GetGradient(iPoint, I_PROGVAR, iDim);
su2double val_u = flowNodes->GetVelocity(iPoint, iDim);
proj_grad_pv_u[iDim] = gradpv * val_u * val_u / (flowNodes->GetVelocity2(iPoint) + 1e-5);
}

/* Parallel projection of temperature gradient against projected progress variable gradient */
su2double proj_grad_T_u = 0, gradT2{0}, mag_gradT{0};
for (auto iDim=0u; iDim < nDim; iDim++) {
su2double gradT = flowNodes->GetGradient_Primitive(iPoint, prim_idx.Temperature(), iDim);
proj_grad_T_u += gradT * proj_grad_pv_u[iDim];
gradT2 += gradT*gradT;
}
Comment thread
EvertBunschoten marked this conversation as resolved.
Outdated
mag_gradT = sqrt(gradT2);
proj_grad_T_u /= (gradT2 + 1e-5);
proj_grad_T_u *= mag_gradT;

/* Update minimum and maximum values. */
gradpv_local = max(gradpv_local, proj_grad_T_u);
pvmax_local = max(pvmax_local, pv_local);
pvmin_local = min(pvmin_local, pv_local);
Tmax_local = max(Tmax_local, T_local);
}
END_SU2_OMP_FOR
SU2_MPI::Barrier(SU2_MPI::GetComm());
Comment thread
EvertBunschoten marked this conversation as resolved.
Outdated
SU2_MPI::Allreduce(&gradpv_local, &gradpv_global, 1, MPI_DOUBLE, MPI_MAX, SU2_MPI::GetComm());
SU2_MPI::Allreduce(&pvmin_local, &pvmin_global, 1, MPI_DOUBLE, MPI_MIN, SU2_MPI::GetComm());
SU2_MPI::Allreduce(&pvmax_local, &pvmax_global, 1, MPI_DOUBLE, MPI_MAX, SU2_MPI::GetComm());
SU2_MPI::Allreduce(&Tmax_local, &Tmax_global, 1, MPI_DOUBLE, MPI_MAX, SU2_MPI::GetComm());
Comment thread
EvertBunschoten marked this conversation as resolved.
Outdated

/* Update flame thickness value. */
su2double flame_thickness{default_flame_thickness};
if (Tmax_global > 800.0) flame_thickness = (pvmax_global - pvmin_global) / (gradpv_global+1e-5);
Comment thread
EvertBunschoten marked this conversation as resolved.
Outdated
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Is this temperature a tunable parameter?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Now it is


return flame_thickness;
}
5 changes: 5 additions & 0 deletions config_template.cfg
Original file line number Diff line number Diff line change
Expand Up @@ -974,6 +974,11 @@ SPARK_INIT= (0.004, 0.0, 0.0, 1e-4, 100, 10)
% The number of terms should equate the total number of species in the flamelet problem.
SPARK_REACTION_RATES= (1000, 0, 0)

% Thickened flame correction
% Approximate the thickness of the flame and suppress source terms and enhance diffusivity in cells with a lengthscale higher
% than the flame thickness.
THICKENED_FLAME_CORRECTION= YES

%
% --------------------- INVERSE DESIGN SIMULATION -----------------------------%
%
Expand Down
Loading