FIX: get strategy version through vector_rep

discourse · Dec 13, 2024 · 4cfb174 · 4cfb174
1 parent fae1fbc
commit 4cfb174
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 1 deletion.
diff --git a/app/jobs/scheduled/embeddings_backfill.rb b/app/jobs/scheduled/embeddings_backfill.rb
@@ -88,7 +88,7 @@ def execute(args)
         .where(<<~SQL)
           #{table_name}.model_version < #{vector_rep.version}
           OR
-          #{table_name}.strategy_version < #{strategy.version}
+          #{table_name}.strategy_version < #{vector_rep.strategy_version}
         SQL
         .limit(limit - rebaked)
         .pluck(:id)

diff --git a/spec/jobs/scheduled/embeddings_backfill_spec.rb b/spec/jobs/scheduled/embeddings_backfill_spec.rb
@@ -25,6 +25,7 @@
     SiteSetting.ai_embeddings_enabled = true
     SiteSetting.ai_embeddings_discourse_service_api_endpoint = "http://test.com"
     SiteSetting.ai_embeddings_backfill_batch_size = 1
+    SiteSetting.ai_embeddings_per_post_enabled = true
     Jobs.run_immediately!
   end