avoid double initialize_state call in solve

Author: lkdvos

src/interface/interface.jl

@@ -40,8 +40,33 @@ returns a state.
 By default this method continues to call [`solve!`](@ref).
 """
 function solve(problem::Problem, algorithm::Algorithm; kwargs...)
+    # obtain logger once to minimize overhead from accessing ScopedValue
+    # additionally handle logging initialization to enable stateful LoggingAction
+    logger = algorithm_logger()
+    # initialize_logger(logger, problem, algorithm, state)
+
+    # initialize the state and emit message
     state = initialize_state(problem, algorithm; kwargs...)
-    return solve!(problem, algorithm, state; kwargs...)
+    emit_message(logger, problem, algorithm, state, :Start)
+
+    # main body of the algorithm
+    while !is_finished!(problem, algorithm, state)
+        # logging event between convergence check and algorithm step
+        emit_message(logger, problem, algorithm, state, :PreStep)
+
+        # algorithm step
+        increment!(state)
+        step!(problem, algorithm, state)
+
+        # logging event between algorithm step and convergence check
+        emit_message(logger, problem, algorithm, state, :PostStep)
+    end
+
+    # emit message about finished state
+    output = finalize_state!(problem, algorithm, state)
+    emit_message(logger, problem, algorithm, state, :Stop)
+
+    return output
 end
 
 @doc """