From 6faec506c83934028e483556635145b838f71273 Mon Sep 17 00:00:00 2001
From: Nolan Tremelling <34580718+NolanTrem@users.noreply.github.com>
Date: Fri, 20 Dec 2024 13:39:49 -0800
Subject: [PATCH] Remove chunking strategy from test (#1715)

* Remove strategy from test

* Put in chunking_strategy as a comment
---
 py/core/database/filters.py            | 2 --
 py/tests/integration/test_ingestion.py | 2 +-
 2 files changed, 1 insertion(+), 3 deletions(-)

diff --git a/py/core/database/filters.py b/py/core/database/filters.py
index a7f00f2db..be3ebeb60 100644
--- a/py/core/database/filters.py
+++ b/py/core/database/filters.py
@@ -423,8 +423,6 @@ def apply_filters(
     builder = SQLFilterBuilder(params=params, mode=mode)
     filter_clause, new_params = builder.build(expr)
 
-    print("filter_clause = ", filter_clause)
-
     if mode == "where_clause":
         return filter_clause, new_params  # Already includes WHERE
     elif mode == "condition_only":
diff --git a/py/tests/integration/test_ingestion.py b/py/tests/integration/test_ingestion.py
index 403083a57..874b6b8dd 100644
--- a/py/tests/integration/test_ingestion.py
+++ b/py/tests/integration/test_ingestion.py
@@ -235,7 +235,7 @@ def test_custom_ingestion_config(client: R2RClient):
     custom_config = {
         "provider": "r2r",
         "strategy": "auto",
-        "chunking_strategy": "by_title",
+        # "chunking_strategy": "by_title", Fixme: This was not implemented in the ingestion config
         "new_after_n_chars": 256,
         "max_characters": 512,
         "combine_under_n_chars": 64,