Skip to content

Commit 43d7e25

Browse files
authored
fix optim converged (#1867)
* fix optim converged * Temporary pin manifolds
1 parent 0bf32a0 commit 43d7e25

File tree

2 files changed

+3
-3
lines changed

2 files changed

+3
-3
lines changed

Project.toml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -87,7 +87,7 @@ JSON3 = "1"
8787
KernelDensityEstimate = "0.5.6"
8888
LinearAlgebra = "1.10"
8989
ManifoldDiff = "0.3, 0.4"
90-
Manifolds = "0.10"
90+
Manifolds = "=0.10.16"
9191
ManifoldsBase = "0.15, 1"
9292
Manopt = "^0.5.14"
9393
MetaGraphs = "0.7, 0.8"

src/parametric/services/ParametricCSMFunctions.jl

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -32,7 +32,7 @@ function solveUp_ParametricStateMachine(csmc::CliqStateMachineContainer)
3232
# @info "$(csmc.cliq.id) Σ $(Σ)"
3333
# Pack all results in variables
3434
# FIXME test f_converged, ls_success, confirm convergence check
35-
if result.f_converged || result.g_converged
35+
if Optim.g_converged(result) || Optim.f_converged(result)
3636
logCSM(csmc, "$(csmc.cliq.id): subfg optim converged updating variables")
3737
for (v, val) in vardict
3838
vnd = getSolverData(getVariable(csmc.cliqSubFg, v), :parametric)
@@ -136,7 +136,7 @@ function solveDown_ParametricStateMachine(csmc::CliqStateMachineContainer)
136136
#TEMP testing difference
137137
# vardict, result = solveGraphParametric(csmc.cliqSubFg)
138138
# Pack all results in variables
139-
if result.g_converged || result.f_converged
139+
if Optim.g_converged(result) || Optim.f_converged(result)
140140
logCSM(
141141
csmc,
142142
"$(csmc.cliq.id): subfg optim converged updating variables";

0 commit comments

Comments
 (0)