Merge pull request #445 from JuliaRobotics/hotfix/2Q20/paramsdispatch

change to params instead (similar to before)

Author: dehann

src/services/AbstractDFG.jl

@@ -57,7 +57,8 @@ getSolverParams(dfg::AbstractDFG) = dfg.solverParams
 
 Method must be overloaded by the user for Serialization to work.  E.g. IncrementalInference uses `CommonConvWrapper <: FactorOperationalMemory`.
 """
-getFactorOperationalMemoryType(dummy) = error("Please define this function getFactorOperationalMemoryType for your usecase, e.g. = `IIF.CommonConvWrapper <: FactorOperationalMemory`")
+getFactorOperationalMemoryType(dummy) = error("Please extend your workspace with function getFactorOperationalMemoryType(<:AbstractParams) for your usecase, e.g. IncrementalInference uses `CommonConvWrapper <: FactorOperationalMemory`")
+getFactorOperationalMemoryType(dfg::AbstractDFG) = getFactorOperationalMemoryType(getSolverParams(dfg))
 
 
 ##------------------------------------------------------------------------------

test/consolInterfaceDev.jl

@@ -7,11 +7,6 @@ using Dates
 using UUIDs
 # using IncrementalInference
 
-## must overload the function
-import DistributedFactorGraphs: getFactorOperationalMemoryType
-# DF, dropping DFG. so that Main context is used and can be overridden by IIF.getFac...Type
-getFactorOperationalMemoryType(::CloudGraphsDFG) = TestCCW
-
 
 include("testBlocks.jl")