Skip to content

Commit 26702a7

Browse files
mtaomtao
committed
changing code that depends on the built-in fmt, removing use of fmt/ostream.h because it seems to break
1 parent 19d9b26 commit 26702a7

File tree

5 files changed

+40
-40
lines changed

5 files changed

+40
-40
lines changed

‎src/polysolve/Utils.cpp‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -53,7 +53,7 @@ namespace polysolve
5353

5454
void StopWatch::log_msg()
5555
{
56-
const static std::string log_fmt_text =
56+
const static auto log_fmt_text =
5757
fmt::format("[{}] {{}} {{:.3g}}s", fmt::format(fmt::fg(fmt::terminal_color::magenta), "timing"));
5858

5959
if (!m_name.empty())
@@ -64,7 +64,7 @@ namespace polysolve
6464

6565
void log_and_throw_error(spdlog::logger &logger, const std::string &msg)
6666
{
67-
logger.error(msg);
67+
logger.error("{}", msg);
6868
throw std::runtime_error(msg);
6969
}
7070

‎src/polysolve/Utils.hpp‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -55,7 +55,7 @@ namespace polysolve
5555
template <typename... Args>
5656
[[noreturn]] void log_and_throw_error(spdlog::logger &logger, const std::string &msg, const Args &...args)
5757
{
58-
log_and_throw_error(logger, fmt::format(msg, args...));
58+
log_and_throw_error(logger, fmt::format(fmt::runtime(msg), args...));
5959
}
6060

6161
Eigen::SparseMatrix<double> sparse_identity(int rows, int cols);

‎src/polysolve/nonlinear/Criteria.cpp‎

Lines changed: 23 additions & 28 deletions
Original file line numberDiff line numberDiff line change
@@ -29,7 +29,10 @@ namespace polysolve::nonlinear
2929

3030
void Criteria::print(std::ostream &os) const
3131
{
32-
os << fmt::format(
32+
os << print_message();
33+
}
34+
std::string Criteria::print_message() const {
35+
return fmt::format(
3336
"iters={:d} Δf={:g} ‖∇f‖={:g} ‖Δx‖={:g} Δx⋅∇f(x)={:g}",
3437
iterations, fDelta, gradNorm, xDelta, xDeltaDotGrad);
3538
}
@@ -61,47 +64,39 @@ namespace polysolve::nonlinear
6164
return Status::Continue;
6265
}
6366

64-
std::ostream &operator<<(std::ostream &os, const Status &s)
65-
{
67+
std::string_view status_message(Status s) {
6668
switch (s)
6769
{
6870
case Status::NotStarted:
69-
os << "Solver not started";
70-
break;
71+
return "Solver not started";
7172
case Status::Continue:
72-
os << "Convergence criteria not reached";
73-
break;
73+
return "Convergence criteria not reached";
7474
case Status::IterationLimit:
75-
os << "Iteration limit reached";
76-
break;
75+
return "Iteration limit reached";
7776
case Status::XDeltaTolerance:
78-
os << "Change in parameter vector too small";
79-
break;
77+
return "Change in parameter vector too small";
8078
case Status::FDeltaTolerance:
81-
os << "Change in cost function value too small";
82-
break;
79+
return "Change in cost function value too small";
8380
case Status::GradNormTolerance:
84-
os << "Gradient vector norm too small";
85-
break;
81+
return "Gradient vector norm too small";
8682
case Status::ObjectiveCustomStop:
87-
os << "Objective function specified to stop";
88-
break;
83+
return "Objective function specified to stop";
8984
case Status::NanEncountered:
90-
os << "Objective or gradient function returned NaN";
91-
break;
85+
return "Objective or gradient function returned NaN";
9286
case Status::NotDescentDirection:
93-
os << "Search direction not a descent direction";
94-
break;
87+
return "Search direction not a descent direction";
9588
case Status::LineSearchFailed:
96-
os << "Line search failed";
97-
break;
89+
return "Line search failed";
9890
case Status::UpdateDirectionFailed:
99-
os << "Update direction could not be computed";
100-
break;
91+
return "Update direction could not be computed";
10192
default:
102-
os << "Unknown status";
103-
break;
93+
return "Unknown status";
10494
}
95+
}
96+
97+
std::ostream &operator<<(std::ostream &os, const Status &s)
98+
{
99+
os << status_message(s);
105100
return os;
106101
}
107102

@@ -110,4 +105,4 @@ namespace polysolve::nonlinear
110105
c.print(os);
111106
return os;
112107
}
113-
} // namespace polysolve::nonlinear
108+
} // namespace polysolve::nonlinear

‎src/polysolve/nonlinear/Criteria.hpp‎

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,7 @@
22

33
#include <cstddef>
44
#include <iostream>
5+
#include <string_view>
56

67
namespace polysolve::nonlinear
78
{
@@ -43,11 +44,15 @@ namespace polysolve::nonlinear
4344
void reset();
4445

4546
void print(std::ostream &os) const;
47+
std::string print_message() const;
4648
};
4749

4850
Status checkConvergence(const Criteria &stop, const Criteria &current);
4951

52+
std::string_view status_message(Status s);
53+
std::string criteria_message(const Criteria& s);
54+
5055
std::ostream &operator<<(std::ostream &os, const Status &s);
5156

5257
std::ostream &operator<<(std::ostream &os, const Criteria &c);
53-
} // namespace polysolve::nonlinear
58+
} // namespace polysolve::nonlinear

‎src/polysolve/nonlinear/Solver.cpp‎

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -283,7 +283,7 @@ namespace polysolve::nonlinear
283283

284284
m_logger.debug(
285285
"Starting {} with {} solve f₀={:g} (stopping criteria: {})",
286-
descent_strategy_name(), m_line_search->name(), objFunc(x), m_stop);
286+
descent_strategy_name(), m_line_search->name(), objFunc(x), m_stop.print_message());
287287

288288
update_solver_info(objFunc(x));
289289
objFunc.post_step(PostStepData(m_current.iterations, solver_info, x, grad));
@@ -354,12 +354,12 @@ namespace polysolve::nonlinear
354354
m_status = Status::UpdateDirectionFailed;
355355
log_and_throw_error(
356356
m_logger, "[{}][{}] {} on last strategy; stopping",
357-
current_name, m_line_search->name(), m_status);
357+
current_name, m_line_search->name(), status_message(m_status));
358358
}
359359

360360
m_logger.debug(
361361
"[{}][{}] {}; reverting to {}", current_name, m_line_search->name(),
362-
Status::UpdateDirectionFailed, descent_strategy_name());
362+
status_message(Status::UpdateDirectionFailed), descent_strategy_name());
363363
m_status = Status::Continue;
364364
continue;
365365
}
@@ -378,15 +378,15 @@ namespace polysolve::nonlinear
378378
m_status = Status::NotDescentDirection;
379379
log_and_throw_error(
380380
m_logger, "[{}][{}] {} on last strategy (‖Δx‖={:g}; ‖g‖={:g}; Δx⋅g={:g}≥0); stopping",
381-
current_name, m_line_search->name(), m_status, delta_x.norm(), compute_grad_norm(x, grad),
381+
current_name, m_line_search->name(), status_message(m_status), delta_x.norm(), compute_grad_norm(x, grad),
382382
m_current.xDeltaDotGrad);
383383
}
384384
else
385385
{
386386
m_status = Status::Continue;
387387
m_logger.debug(
388388
"[{}][{}] {} (‖Δx‖={:g}; ‖g‖={:g}; Δx⋅g={:g}≥0); reverting to {}",
389-
current_name, m_line_search->name(), Status::NotDescentDirection,
389+
current_name, m_line_search->name(), status_message(Status::NotDescentDirection),
390390
delta_x.norm(), compute_grad_norm(x, grad), m_current.xDeltaDotGrad,
391391
descent_strategy_name());
392392
}
@@ -477,7 +477,7 @@ namespace polysolve::nonlinear
477477

478478
m_logger.debug(
479479
"[{}][{}] {} (stopping criteria: {})",
480-
descent_strategy_name(), m_line_search->name(), m_current, m_stop);
480+
descent_strategy_name(), m_line_search->name(), m_current.print_message(), m_stop.print_message());
481481

482482
if (++m_current.iterations >= m_stop.iterations)
483483
m_status = Status::IterationLimit;
@@ -499,8 +499,8 @@ namespace polysolve::nonlinear
499499
m_logger.log(
500500
succeeded ? spdlog::level::info : spdlog::level::err,
501501
"[{}][{}] Finished: {} took {:g}s ({}) (stopping criteria: {})",
502-
descent_strategy_name(), m_line_search->name(), m_status, tot_time,
503-
m_current, m_stop);
502+
descent_strategy_name(), m_line_search->name(), status_message(m_status), tot_time,
503+
m_current.print_message(), m_stop.print_message());
504504

505505
log_times();
506506
update_solver_info(objFunc(x));

0 commit comments

Comments
 (0)