fixes a bug on brute force indexes in push_item and append_items

sadit · Feb 20, 2024 · 47f2fa1 · 47f2fa1 · sadit · Feb 20, 2024
1 parent e1bc5e2
commit 47f2fa1
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 9 deletions.
diff --git a/Project.toml b/Project.toml
@@ -1,7 +1,7 @@
 name = "SimilaritySearch"
 uuid = "053f045d-5466-53fd-b400-a066f88fe02a"
 authors = ["Eric S. Tellez <donsadit@gmail.com>"]
-version = "0.11.1"
+version = "0.11.2"
 
 [deps]
 Dates = "ade2ca70-3891-5945-98fb-dc099432e06a"

diff --git a/src/parallel-exhaustive.jl b/src/parallel-exhaustive.jl
@@ -57,19 +57,19 @@ end
 
 function push_item!(ex::ParallelExhaustiveSearch, context::GenericContext, u)
     push_item!(ex.db, u)
-    context.logger !== nothing && LOG(context.logger, push_item!, index, length(seq))
+    context.logger !== nothing && LOG(context.logger, push_item!, ex, length(ex))
     ex
 end
 
 function append_items!(ex::ParallelExhaustiveSearch, context::GenericContext, u::AbstractDatabase)
-    sp = length(seq)
+    sp = length(ex)
     push_item!(ex.db, u)
-    ep = length(seq)
-    context.logger !== nothing && LOG(context.logger, append_items!, index, sp, ep, ep)
+    ep = length(ex)
+    context.logger !== nothing && LOG(context.logger, append_items!, ex, sp, ep, ep)
     ex
 end
 
-function index!(seq::ParallelExhaustiveSearch, ctx::AbstractContext)
+function index!(ex::ParallelExhaustiveSearch, ctx::AbstractContext)
     # do nothing
-    seq
+    ex
 end
diff --git a/src/sequential-exhaustive.jl b/src/sequential-exhaustive.jl
@@ -25,15 +25,15 @@ Base.copy(seq::ExhaustiveSearch; dist=seq.dist, db=seq.db) = ExhaustiveSearch(di
 
 function push_item!(seq::ExhaustiveSearch, context::GenericContext, u)
     push_item!(seq.db, u)
-    context.logger !== nothing && LOG(context.logger, push_item!, index, length(seq))
+    context.logger !== nothing && LOG(context.logger, push_item!, seq, length(seq))
     seq
 end
 
 function append_items!(seq::ExhaustiveSearch, context::GenericContext, u::AbstractDatabase)
     sp = length(seq)
     append_items!(seq.db, u)
     ep = length(seq)
-    context.logger !== nothing && LOG(context.logger, append_items!, index, sp, ep, ep)
+    context.logger !== nothing && LOG(context.logger, append_items!, seq, sp, ep, ep)
     seq
 end