Remove chunking strategy from test (#1715)

* Remove strategy from test * Put in chunking_strategy as a comment
SciPhi-AI · Dec 20, 2024 · 6faec50 · 6faec50
1 parent a4f1eda
commit 6faec50
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 3 deletions.
diff --git 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
@@ -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,