Merge pull request #1690 from JuliaRobotics/23Q1/enh/testsnotbroken

tests not broken w dfg 20
JuliaRobotics · Mar 8, 2023 · 4ac23b2 · 4ac23b2
2 parents 8120845 + 96a21f2
commit 4ac23b2
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/Project.toml b/Project.toml
@@ -2,7 +2,7 @@ name = "IncrementalInference"
 uuid = "904591bb-b899-562f-9e6f-b8df64c7d480"
 keywords = ["MM-iSAMv2", "Bayes tree", "junction tree", "Bayes network", "variable elimination", "graphical models", "SLAM", "inference", "sum-product", "belief-propagation"]
 desc = "Implements the Multimodal-iSAMv2 algorithm."
-version = "0.32.3"
+version = "0.33.0"
 
 [deps]
 ApproxManifoldProducts = "9bbbb610-88a1-53cd-9763-118ce10c1f89"

diff --git a/test/testCompareVariablesFactors.jl b/test/testCompareVariablesFactors.jl
@@ -46,7 +46,7 @@ fg2 = deepcopy(fg)
 @test compareSimilarFactors(fg, fg2; skip=[:particleidx])
 
 @test compareFactorGraphs(fg, fg)
-@test_broken compareFactorGraphs(fg, fg2; skip=[:particleidx; :varidx])
+@test compareFactorGraphs(fg, fg2; skip=[:particleidx; :varidx])
 
 # easier error messages
 getSolverParams(fg).multiproc = false
@@ -113,7 +113,7 @@ sfg = buildSubgraph(fg, [:x0;:x1], 1) # distance=1 to include factors
 #FIXME JT - this doesn't make sense to pass, it is a subgraph so should it not rather be ⊂ [subset]?
 # compareDFG(fg1, fg2, by=⊂, skip=...)
 @test fg.sessionLabel == sfg.sessionLabel[1:length(fg.sessionLabel)]
-@test_broken compareFactorGraphs(fg, sfg, skip=[:labelDict;:addHistory;:logpath;:sessionLabel; :particleidx; :varidx])
+@test compareFactorGraphs(fg, sfg, skip=[:labelDict;:addHistory;:logpath;:sessionLabel; :particleidx; :varidx])
 
 # drawGraph(sfg)