Skip to content

Commit 8099b50

Browse files
committed
more data api and test fixes
1 parent b95544a commit 8099b50

File tree

4 files changed

+22
-16
lines changed

4 files changed

+22
-16
lines changed

src/DataBlobs/DataBlobs.jl

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -21,3 +21,6 @@ export BlobEntry
2121
# export copyStore
2222

2323
export getId, getHash, getTimestamp
24+
25+
# convenience wrappers
26+
export getData, addData!, updateData!, deleteData!

src/DataBlobs/services/BlobEntry.jl

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -320,7 +320,7 @@ function incrDataLabelSuffix(
320320
hasund = false
321321
len = 0
322322
try
323-
de,_ = getBlob(dfg, Symbol(vla), bllb)
323+
de,_ = getData(dfg, Symbol(vla), bllb)
324324
bllb = string(bllb)
325325
# bllb *= bllb[end] != '_' ? "_" : ""
326326
datalabel[] = string(de.label)

test/consol_DataEntryBlobTests.jl

Lines changed: 16 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -10,6 +10,8 @@ using SHA
1010

1111
include("testBlocks.jl")
1212

13+
# import DistributedFactorGraphs: addData!, updateData!, getData, deleteData!
14+
1315
end
1416

1517
# Build a basic graph.
@@ -89,11 +91,11 @@ dataset2 = rand(UInt8, 1000)
8991
ds = FolderStore{Vector{UInt8}}(:filestore, "/tmp/dfgFolderStore")
9092
addBlobStore!(dfg, ds)
9193

92-
ade,adb = addBlob!(dfg, :filestore, :x1, :random, dataset1)
93-
_,_ = addBlob!(dfg, :filestore, :x1, :another_1, dataset1)
94-
_,_ = getBlob(dfg, :x1, "random")
95-
_,_ = getBlob(dfg, :x1, r"rando")
96-
gde,gdb = getBlob(dfg, :x1, :random)
94+
ade,adb = addData!(dfg, :filestore, :x1, :random, dataset1)
95+
_,_ = addData!(dfg, :filestore, :x1, :another_1, dataset1)
96+
_,_ = getData(dfg, :x1, "random")
97+
_,_ = getData(dfg, :x1, r"rando")
98+
gde,gdb = getData(dfg, :x1, :random)
9799

98100
@show gde
99101

@@ -102,19 +104,19 @@ gde,gdb = getBlob(dfg, :x1, :random)
102104
# @test incrDataLabelSuffix(dfg,:x1,:another) == :another_2 # TODO exand support for Regex likely search on labels
103105
# @test incrDataLabelSuffix(dfg,:x1,"random") == "random_1" # TODO expand support for label::String
104106

105-
dde,ddb = deleteBlob!(dfg, :x1, :random)
106-
_,_ = deleteBlob!(dfg, :x1, :another_1)
107+
dde,ddb = deleteData!(dfg, :x1, :random)
108+
_,_ = deleteData!(dfg, :x1, :another_1)
107109

108110
@test ade == gde == dde
109111
@test adb == gdb == ddb
110112

111-
ade2,adb2 = addBlob!(dfg, :x2, deepcopy(ade), dataset1)
113+
ade2,adb2 = addData!(dfg, :x2, deepcopy(ade), dataset1)
112114
# ade3,adb3 = updateBlob!(dfg, :x2, deepcopy(ade), dataset1)
113115

114116
@test ade == ade2# == ade3
115117
@test adb == adb2# == adb3
116118

117-
deleteBlob!(dfg, :x2, :random)
119+
deleteData!(dfg, :x2, :random)
118120

119121
#test default folder store
120122
dfs = FolderStore("/tmp/defaultfolderstore")
@@ -130,20 +132,20 @@ dfs = FolderStore("/tmp/defaultfolderstore")
130132
ds = InMemoryBlobStore()
131133
addBlobStore!(dfg, ds)
132134

133-
ade,adb = addBlob!(dfg, :default_inmemory_store, :x1, :random, dataset1)
134-
gde,gdb = getBlob(dfg, :x1, :random)
135-
dde,ddb = deleteBlob!(dfg, :x1, :random)
135+
ade,adb = addData!(dfg, :default_inmemory_store, :x1, :random, dataset1)
136+
gde,gdb = getData(dfg, :x1, :random)
137+
dde,ddb = deleteData!(dfg, :x1, :random)
136138

137139
@test ade == gde == dde
138140
@test adb == gdb == ddb
139141

140-
ade2,adb2 = addBlob!(dfg, :x2, deepcopy(ade), dataset1)
142+
ade2,adb2 = addData!(dfg, :x2, deepcopy(ade), dataset1)
141143
# ade3,adb3 = updateBlob!(dfg, :x2, deepcopy(ade), dataset1)
142144

143145
@test ade == ade2# == ade3
144146
@test adb == adb2# == adb3
145147

146-
deleteBlob!(dfg, :x2, :random)
148+
deleteData!(dfg, :x2, :random)
147149

148150
##==============================================================================
149151
## Unimplemented store

test/testBlocks.jl

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,8 @@ using Dates
44
using Manifolds
55

66
import Base: convert
7-
import DistributedFactorGraphs: reconstFactorData, getData, addData!, updateData!, deleteData!
7+
import DistributedFactorGraphs: reconstFactorData
8+
# import DistributedFactorGraphs: getData, addData!, updateData!, deleteData!
89

910
# Test InferenceVariable Types
1011
# struct TestVariableType1 <: InferenceVariable

0 commit comments

Comments
 (0)