Skip to content

Commit

Permalink
small fixes
Browse files Browse the repository at this point in the history
  • Loading branch information
dehann committed Jul 20, 2021
1 parent abab8aa commit c0f0f57
Showing 1 changed file with 2 additions and 3 deletions.
5 changes: 2 additions & 3 deletions src/SerializationMKD.jl
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@ import DistributedFactorGraphs: getVariableType

export PackedManifoldKernelDensity

# NOTE, user factors will require the same definitions
# NOTE, user variables and manifolds will require the same definitions, TODO perhaps add into `@defVariable`
# unusual definitions, but all they need to do is pack and unpack as one-to-one
# this step actually occurs separate from the actual variables or factors (with their own manifolds)
# relies on later use of getManifold to give back the same <:AbstractManifold
Expand Down Expand Up @@ -61,7 +61,6 @@ Base.convert(::Type{<:PackedSamplableBelief}, mkd::ManifoldKernelDensity) = conv
# https://discourse.julialang.org/t/converting-string-to-datatype-with-meta-parse/33024/2
# https://discourse.julialang.org/t/is-there-a-way-to-import-modules-with-a-string/15723/6
function Base.convert(::Type{<:ManifoldKernelDensity}, str::AbstractString)
@info "UNPACK MKD" str
data = JSON2.read(str, PackedManifoldKernelDensity)

# piggy back on serialization of InferenceVariable rather than try serialize anything Manifolds.jl
Expand All @@ -74,4 +73,4 @@ end



#
#

0 comments on commit c0f0f57

Please sign in to comment.