Skip to content

Commit 9604192

Browse files
committed
bandaid on updateVariableSolverData!
1 parent 15e57d5 commit 9604192

File tree

2 files changed

+22
-11
lines changed

2 files changed

+22
-11
lines changed

src/Deprecated.jl

Lines changed: 0 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -57,18 +57,8 @@ const AbstractBigDataEntry = AbstractDataEntry
5757
##==============================================================================
5858
## Remove in 0.11
5959
##==============================================================================
60-
6160
@deprecate addVariableSolverData!(dfg::AbstractDFG, variablekey::Symbol, vnd::VariableNodeData, solverKey::Symbol) addVariableSolverData!(dfg, variablekey, vnd)
6261

63-
@deprecate updateVariableSolverData!(dfg::AbstractDFG,
64-
variablekey::Symbol,
65-
vnd::VariableNodeData,
66-
solverKey::Symbol,
67-
useCopy::Bool=true,
68-
fields::Vector{Symbol}=Symbol[],
69-
verbose::Bool=true ) updateVariableSolverData!(dfg, variablekey, vnd, useCopy, fields, verbose)
70-
71-
7262
@deprecate updatePPE!(dfg::AbstractDFG, variablekey::Symbol, ppe::AbstractPointParametricEst, ppekey::Symbol) updatePPE!(dfg, variablekey, ppe)
7363

7464
@deprecate addPPE!(dfg::AbstractDFG, variablekey::Symbol, ppe::AbstractPointParametricEst, ppekey::Symbol) addPPE!(dfg, variablekey, ppe)

src/services/DFGVariable.jl

Lines changed: 22 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -409,6 +409,27 @@ function updateVariableSolverData!(dfg::AbstractDFG,
409409
return var.solverDataDict[vnd.solverKey]
410410
end
411411

412+
413+
function updateVariableSolverData!(dfg::AbstractDFG,
414+
variablekey::Symbol,
415+
vnd::VariableNodeData,
416+
solverKey::Symbol,
417+
useCopy::Bool=true,
418+
fields::Vector{Symbol}=Symbol[],
419+
verbose::Bool=true)
420+
421+
# TODO not very clean
422+
if vnd.solverKey != solverKey
423+
@warn "TODO It looks like solverKey as parameter is deprecated, set it in vnd, or keep this function?"
424+
usevnd = useCopy ? deepcopy(vnd) : vnd
425+
usevnd.solverKey = solverKey
426+
return updateVariableSolverData!(dfg, variablekey, usevnd, useCopy, fields, verbose)
427+
else
428+
return updateVariableSolverData!(dfg, variablekey, vnd, useCopy, fields, verbose)
429+
end
430+
end
431+
432+
412433
updateVariableSolverData!(dfg::AbstractDFG,
413434
sourceVariable::DFGVariable,
414435
solverKey::Symbol=:default,
@@ -441,7 +462,7 @@ function deepcopySolvekeys!(dfg::AbstractDFG,
441462
for x in labels
442463
sd = deepcopy(getSolverData(getVariable(dfg,x), src))
443464
sd.solverKey = dest
444-
updateVariableSolverData!(dfg, x, true, Symbol[], verbose )
465+
updateVariableSolverData!(dfg, x, sd, true, Symbol[], verbose )
445466
end
446467
end
447468
const deepcopySupersolve! = deepcopySolvekeys!

0 commit comments

Comments
 (0)