Skip to content

Commit c096dea

Browse files
authored
Merge pull request #198 from JuliaRobotics/feature/4Q19/ppesuggestedhelper
Add getSuggestedPPE
2 parents a1f16da + eee3585 commit c096dea

File tree

3 files changed

+5
-0
lines changed

3 files changed

+5
-0
lines changed

src/DistributedFactorGraphs.jl

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -21,6 +21,7 @@ export AbstractParams, NoSolverParams
2121
export DFGNode, DFGVariable, DFGFactor, AbstractDFGVariable, AbstractDFGFactor
2222
export InferenceType, PackedInferenceType, FunctorInferenceType, InferenceVariable, ConvolutionObject
2323
export FunctorSingleton, FunctorPairwise, FunctorPairwiseMinimize
24+
export getMaxPPE, getMeanPPE, getSuggestedPPE
2425
export label, timestamp, tags, estimates, estimate, data, softtype, solverData, getData, solverDataDict, setSolverData, internalId, smallData, bigData
2526
export DFGVariableSummary, DFGFactorSummary, AbstractDFGSummary
2627

src/entities/DFGVariable.jl

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -97,6 +97,7 @@ MeanMaxPPE(solverKey::Symbol, suggested::Vector{Float64}, max::Vector{Float64},m
9797

9898
getMaxPPE(est::AbstractPointParametricEst) = est.max
9999
getMeanPPE(est::AbstractPointParametricEst) = est.mean
100+
getSuggestedPPE(est::AbstractPointParametricEst) = est.suggested
100101
getLastUpdatedTimestamp(est::AbstractPointParametricEst) = est.lastUpdatedTimestamp
101102

102103

test/interfaceTests.jl

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -216,6 +216,9 @@ end
216216
#For now spot check
217217
@test solverDataDict(newvar) == solverDataDict(var)
218218
@test estimates(newvar) == estimates(var)
219+
@test getMaxPPE(estimates(newvar)[:default]) == estimates(newvar)[:default].max
220+
@test getMeanPPE(estimates(newvar)[:default]) == estimates(newvar)[:default].mean
221+
@test getSuggestedPPE(estimates(newvar)[:default]) == estimates(newvar)[:default].suggested
219222

220223
# Delete :default and replace to see if new ones can be added
221224
delete!(estimates(newvar), :default)

0 commit comments

Comments
 (0)