Skip to content

Commit 9d49eb0

Browse files
committed
Drop max(1, n_steps) check added previously
The fix in #3571 makes this check unnecessary, since RBParameters objects with no parameters report having one step now.
1 parent 1cbec2c commit 9d49eb0

1 file changed

Lines changed: 2 additions & 4 deletions

File tree

src/reduced_basis/rb_eim_evaluation.C

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -239,8 +239,7 @@ void RBEIMEvaluation::rb_eim_solves(const std::vector<RBParameters> & mus,
239239
{
240240
unsigned int counter = 0;
241241
for (auto mu_index : index_range(mus))
242-
// Do at least one solve even if there are no parameters (hence no steps) defined on this mu
243-
for (auto step_index : make_range(std::max(1u, mus[mu_index].n_steps())))
242+
for (auto step_index : make_range(mus[mu_index].n_steps()))
244243
{
245244
// Ignore compiler warnings about unused loop index
246245
libmesh_ignore(step_index);
@@ -278,8 +277,7 @@ void RBEIMEvaluation::rb_eim_solves(const std::vector<RBParameters> & mus,
278277
{
279278
unsigned int counter = 0;
280279
for (auto mu_index : index_range(mus))
281-
// Do at least one solve even if there are no parameters (hence no steps) defined on this mu
282-
for (auto step_index : make_range(std::max(1u, mus[mu_index].n_steps())))
280+
for (auto step_index : make_range(mus[mu_index].n_steps()))
283281
{
284282
// Ignore compiler warnings about unused loop index
285283
libmesh_ignore(step_index);

0 commit comments

Comments
 (0)