Skip to content

Commit

Permalink
Removed psycopg2 from Lux requirements
Browse files Browse the repository at this point in the history
Aiming to simplify the initial Lux installation. Will include a notice in the SQL documentation letting users know that they will have to install the library themselves if the want to use the LuxSQLTable functionality.
  • Loading branch information
thyneb19 committed Apr 11, 2021
1 parent bf3cb0f commit 801f3cd
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 11 deletions.
8 changes: 0 additions & 8 deletions lux/_config/config.py
Original file line number Diff line number Diff line change
Expand Up @@ -348,14 +348,6 @@ def set_SQL_connection(self, connection):

def set_executor_type(self, exe):
if exe == "SQL":
import pkgutil

if pkgutil.find_loader("psycopg2") is None:
raise ImportError(
"psycopg2 is not installed. Run `pip install psycopg2' to install psycopg2 to enable the Postgres connection."
)
else:
import psycopg2
from lux.executor.SQLExecutor import SQLExecutor

self.executor = SQLExecutor()
Expand Down
3 changes: 3 additions & 0 deletions requirements-dev.txt
Original file line number Diff line number Diff line change
Expand Up @@ -4,3 +4,6 @@ Sphinx>=3.0.2
sphinx-rtd-theme>=0.4.3
xlrd
black
# Install to use SQLExecutor
psycopg2>=2.8.5
psycopg2-binary>=2.8.5
3 changes: 0 additions & 3 deletions requirements.txt
Original file line number Diff line number Diff line change
Expand Up @@ -4,9 +4,6 @@ numpy>=1.16.5
pandas>=1.2.0
scikit-learn>=0.22
matplotlib>=3.0.0
# Install only to use SQLExecutor
psycopg2>=2.8.5
psycopg2-binary>=2.8.5
lux-widget>=0.1.4
us
iso3166

0 comments on commit 801f3cd

Please sign in to comment.