DuckDB version fix

CHANGELOG.md

@@ -14,6 +14,7 @@
 * [Fix] Fix result not displayed when `SUMMARIZE` argument is used in duckdb with a sqlalchemy connection (#836)
 * [Fix] Show deprecation warnings for legacy plot API (#513)
 * [Fix] Fix error when trying to access previously non-existing file (#840)
+* [Fix] Testing with latest DuckDB version (#498)
 
 ## 0.10.1 (2023-08-30)
 

setup.py

@@ -33,18 +33,18 @@
 DEV = [
     "flake8",
     "pytest",
-    "pandas",
+    "pandas==2.0.3",
     "polars==0.17.2",  # 04/18/23 this breaks our CI
     "pyarrow",
     "invoke",
     "pkgmt",
     "twine",
     # tests
-    "duckdb<0.8.0",
+    "duckdb",
     "duckdb-engine",
     "pyodbc",
     # sql.plot module tests
-    "matplotlib",
+    "matplotlib==3.7.2",
     "black",
     # for %%sql --interact
     "ipywidgets",

src/sql/ggplot/facet_wrap.py

@@ -17,6 +17,7 @@ def get_facet_values(self, table, column, with_):
             SELECT
             distinct ({{column}})
             FROM {{table}}
+            ORDER BY {{column}}
             """
         )
         query = template.render(table=table, column=column)
@@ -25,6 +26,7 @@ def get_facet_values(self, table, column, with_):
         # Added to make histogram more inclusive to NULLs
         # Filter out NULL values
         # If value[0] is NULL we skip it
+
         values = [value for value in values if value[0] is not None]
         n_plots = len(values)
         n_cols = len(values) if len(values) < 3 else 3

src/tests/test_ggplot.py

@@ -431,7 +431,7 @@ def test_facet_wrap_stacked_histogram_cmap(diamonds_data):
 
 @_cleanup_cm()
 @image_comparison(
-    baseline_images=["test_facet_wrap_default_with_nulls"],
+    baseline_images=["facet_wrap_default_with_nulls"],
     extensions=["png"],
     remove_text=False,
 )