Skip to content

Commit 60f9fee

Browse files
authored
abs -> std::abs
1 parent 52ee9d6 commit 60f9fee

File tree

1 file changed

+8
-7
lines changed

1 file changed

+8
-7
lines changed

src/finitediff.cpp

Lines changed: 8 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -168,15 +168,16 @@ bool compare_gradient(
168168

169169
bool same = true;
170170
for (long i = 0; i < x.rows(); i++) {
171-
double scale = std::max(std::max(abs(x[i]), abs(y[i])), double(1.0));
172-
double abs_diff = abs(x[i] - y[i]);
171+
double scale =
172+
std::max(std::max(std::abs(x[i]), std::abs(y[i])), double(1.0));
173+
double abs_diff = std::abs(x[i] - y[i]);
173174

174175
if (abs_diff > test_eps * scale) {
175176
spdlog::debug(
176177
"{} eps={:.3e} r={} x={:.3e} y={:.3e} |x-y|={:.3e} "
177178
"|x-y|/|x|={:.3e} |x-y|/|y|={:3e}",
178179
msg, test_eps, i, x(i), y(i), abs_diff, abs_diff / abs(x(i)),
179-
abs_diff / abs(y(i)));
180+
abs_diff / std::abs(y(i)));
180181
same = false;
181182
}
182183
}
@@ -196,17 +197,17 @@ bool compare_jacobian(
196197
bool same = true;
197198
for (long i = 0; i < x.rows(); i++) {
198199
for (long j = 0; j < x.cols(); j++) {
199-
double scale =
200-
std::max(std::max(abs(x(i, j)), abs(y(i, j))), double(1.0));
200+
double scale = std::max(
201+
std::max(std::abs(x(i, j)), std::abs(y(i, j))), double(1.0));
201202

202-
double abs_diff = abs(x(i, j) - y(i, j));
203+
double abs_diff = std::abs(x(i, j) - y(i, j));
203204

204205
if (abs_diff > test_eps * scale) {
205206
spdlog::debug(
206207
"{} eps={:.3e} r={} c={} x={:.3e} y={:.3e} "
207208
"|x-y|={:.3e} |x-y|/|x|={:.3e} |x-y|/|y|={:3e}",
208209
msg, test_eps, i, j, x(i, j), y(i, j), abs_diff,
209-
abs_diff / abs(x(i, j)), abs_diff / abs(y(i, j)));
210+
abs_diff / std::abs(x(i, j)), abs_diff / std::abs(y(i, j)));
210211
same = false;
211212
}
212213
}

0 commit comments

Comments
 (0)