Pandas is optional (#632)

* pandas is optional
* 5.0.1 release

HISTORY.rst

@@ -5,6 +5,10 @@ Change Log
 This document records all notable changes to `django-sql-explorer <https://github.com/explorerhq/django-sql-explorer>`_.
 This project adheres to `Semantic Versioning <https://semver.org/>`_.
 
+`5.0.1`_ (2024-06-26)
+===========================
+* `#631`_: Pandas is only required if EXPLORER_USER_UPLOADS_ENABLED is True
+
 `5.0.0`_ (2024-06-25)
 ===========================
 
@@ -471,7 +475,9 @@ Initial Release
 .. _4.1.0: https://github.com/explorerhq/django-sql-explorer/compare/4.0.2...4.1.0
 .. _4.2.0: https://github.com/explorerhq/django-sql-explorer/compare/4.1.0...4.2.0
 .. _4.3.0: https://github.com/explorerhq/django-sql-explorer/compare/4.2.0...4.3.0
-.. _5.0.0: https://github.com/explorerhq/django-sql-explorer/compare/4.2.0...5.0.0
+.. _5.0.0: https://github.com/explorerhq/django-sql-explorer/compare/4.3.0...5.0.0
+.. _5.0.1: https://github.com/explorerhq/django-sql-explorer/compare/5.0.0...5.0.1
+
 
 .. _#254: https://github.com/explorerhq/django-sql-explorer/pull/254
 .. _#334: https://github.com/explorerhq/django-sql-explorer/pull/334
@@ -573,5 +579,6 @@ Initial Release
 .. _#616: https://github.com/explorerhq/django-sql-explorer/issues/616
 .. _#618: https://github.com/explorerhq/django-sql-explorer/issues/618
 .. _#619: https://github.com/explorerhq/django-sql-explorer/issues/619
+.. _#631: https://github.com/explorerhq/django-sql-explorer/issues/631
 
 .. _furo: https://github.com/pradyunsg/furo

explorer/__init__.py

@@ -1,7 +1,7 @@
 __version_info__ = {
     "major": 5,
     "minor": 0,
-    "patch": 0,
+    "patch": 1,
     "releaselevel": "final",
     "serial": 0
 }

explorer/ee/db_connections/utils.py

@@ -4,7 +4,6 @@
 
 from dateutil import parser
 
-import pandas as pd
 import sqlite3
 import io
 
@@ -126,6 +125,7 @@ def atof_custom(value):
     return float(value)
 
 def csv_to_typed_df(csv_bytes, delimiter=",", has_headers=True):  # noqa
+    import pandas as pd
     try:
 
         csv_file = io.BytesIO(csv_bytes)

explorer/tests/test_db_connection_utils.py

@@ -1,6 +1,9 @@
 from django.test import TestCase
+from unittest import skipIf
 from django.core.files.uploadedfile import SimpleUploadedFile
-import pandas as pd
+from explorer.app_settings import EXPLORER_USER_UPLOADS_ENABLED
+if EXPLORER_USER_UPLOADS_ENABLED:
+    import pandas as pd
 import os
 import sqlite3
 from explorer.models import DatabaseConnection
@@ -25,6 +28,7 @@ def _get_csv(csv_name):
     return csv_bytes
 
 
+@skipIf(not EXPLORER_USER_UPLOADS_ENABLED, "User uploads not enabled")
 class TestCsvToTypedDf(TestCase):
 
     def test_mixed_types(self):
@@ -61,6 +65,7 @@ def test_date_parsing(self):
         self.assertTrue(pd.api.types.is_datetime64_ns_dtype(df["Dates"]))
 
 
+@skipIf(not EXPLORER_USER_UPLOADS_ENABLED, "User uploads not enabled")
 class TestSQLiteConnection(TestCase):
 
     @patch("explorer.utils.get_s3_bucket")
@@ -139,6 +144,7 @@ def test_create_django_style_connection_non_sqlite(self, mock_load_backend):
         mock_backend.DatabaseWrapper.assert_called_once()
 
 
+@skipIf(not EXPLORER_USER_UPLOADS_ENABLED, "User uploads not enabled")
 class TestPandasToSQLite(TestCase):
 
     def test_pandas_to_sqlite(self):