diff --git a/docs/_templates/layout.html b/docs/_templates/layout.html deleted file mode 100644 index ecc205f981..0000000000 --- a/docs/_templates/layout.html +++ /dev/null @@ -1,95 +0,0 @@ -{% extends "!layout.html" %} - -{% set bootswatch_css_custom = ['_static/docs.css'] %} - -{%- block content %} - {{ navBar() }} - {% if pagename == 'index' %} - -
-
-
-

Superset

-

- is an open source data visualization platform that provides easy - exploration of your data and allows you to create and share - beautiful charts and dashboards -

-
-
- {% endif %} -
- {% block body %}{% endblock %} -
-{%- endblock %} diff --git a/docs/requirements.txt b/docs/requirements.txt index 99f31cac65..9035feadcd 100644 --- a/docs/requirements.txt +++ b/docs/requirements.txt @@ -1,2 +1,2 @@ -sphinx==1.7.1 -sphinx-rtd-theme==0.2.4 +sphinx==1.8.1 +sphinx-rtd-theme==0.3.1 diff --git a/docs/visualization.rst b/docs/visualization.rst index f2e4636214..94a787e3d4 100644 --- a/docs/visualization.rst +++ b/docs/visualization.rst @@ -496,11 +496,11 @@ List of Countries +------+------------------------------------+ |IT-PE |Pescara | +------+------------------------------------+ -|IT-TE |Teramo | +|IT-TE |Teramo | +------+------------------------------------+ |IT-BA |Bari | +------+------------------------------------+ -|IT-BT |Barletta-Andria-Trani | +|IT-BT |Barletta-Andria-Trani | +------+------------------------------------+ |IT-BR |Brindisi | +------+------------------------------------+ @@ -536,177 +536,177 @@ List of Countries +------+------------------------------------+ |IT-BO |Bologna | +------+------------------------------------+ -|IT-FE |Ferrara | +|IT-FE |Ferrara | +------+------------------------------------+ |IT-FC | Forli' - Cesena | +------+------------------------------------+ -|IT-MO |Modena | +|IT-MO |Modena | +------+------------------------------------+ |IT-PR |Parma | +------+------------------------------------+ -|IT-PC |Piacenza | +|IT-PC |Piacenza | +------+------------------------------------+ -|IT-RA |Ravenna | +|IT-RA |Ravenna | +------+------------------------------------+ -|IT-RE |Reggio Nell'Emilia | +|IT-RE |Reggio Nell'Emilia | +------+------------------------------------+ -|IT-RN |Rimini | +|IT-RN |Rimini | +------+------------------------------------+ -|IT-GO |Gorizia | +|IT-GO |Gorizia | +------+------------------------------------+ -|IT-PN |Pordenone | +|IT-PN |Pordenone | +------+------------------------------------+ -|IT-TS |Trieste | +|IT-TS |Trieste | +------+------------------------------------+ -|IT-UD |Udine | +|IT-UD |Udine | +------+------------------------------------+ -|IT-FR |Frosinone | +|IT-FR |Frosinone | +------+------------------------------------+ -|IT-LT |Latina | +|IT-LT |Latina | +------+------------------------------------+ -|IT-RI |Rieti | +|IT-RI |Rieti | +------+------------------------------------+ -|IT-RM |Roma | +|IT-RM |Roma | +------+------------------------------------+ -|IT-VT |Viterbo | +|IT-VT |Viterbo | +------+------------------------------------+ -|IT-GE |Genova | +|IT-GE |Genova | +------+------------------------------------+ -|IT-IM |Imperia | +|IT-IM |Imperia | +------+------------------------------------+ -|IT-SP |La Spezia | +|IT-SP |La Spezia | +------+------------------------------------+ -|IT-SV |Savona | +|IT-SV |Savona | +------+------------------------------------+ -|IT-BG |Bergamo | +|IT-BG |Bergamo | +------+------------------------------------+ -|IT-BS |Brescia | +|IT-BS |Brescia | +------+------------------------------------+ -|IT-CO |Como | +|IT-CO |Como | +------+------------------------------------+ -|IT-CR |Cremona | +|IT-CR |Cremona | +------+------------------------------------+ -|IT-LC |Lecco | +|IT-LC |Lecco | +------+------------------------------------+ -|IT-LO |Lodi | +|IT-LO |Lodi | +------+------------------------------------+ -|IT-MN |Mantua | +|IT-MN |Mantua | +------+------------------------------------+ -|IT-MI |Milano | +|IT-MI |Milano | +------+------------------------------------+ -|IT-MB |Monza and Brianza | +|IT-MB |Monza and Brianza | +------+------------------------------------+ -|IT-PV |Pavia | +|IT-PV |Pavia | +------+------------------------------------+ -|IT-SO |Sondrio | +|IT-SO |Sondrio | +------+------------------------------------+ -|IT-VA |Varese | +|IT-VA |Varese | +------+------------------------------------+ -|IT-AN |Ancona | +|IT-AN |Ancona | +------+------------------------------------+ -|IT-AP |Ascoli Piceno | +|IT-AP |Ascoli Piceno | +------+------------------------------------+ -|IT-FM |Fermo | +|IT-FM |Fermo | +------+------------------------------------+ -|IT-MC |Macerata | +|IT-MC |Macerata | +------+------------------------------------+ -|IT-PU |Pesaro E Urbino | +|IT-PU |Pesaro E Urbino | +------+------------------------------------+ -|IT-CB |Campobasso | +|IT-CB |Campobasso | +------+------------------------------------+ -|IT-IS |Isernia | +|IT-IS |Isernia | +------+------------------------------------+ -|IT-AL |Alessandria | +|IT-AL |Alessandria | +------+------------------------------------+ -|IT-AT |Asti | +|IT-AT |Asti | +------+------------------------------------+ -|IT-BI |Biella | +|IT-BI |Biella | +------+------------------------------------+ -|IT-CN |Cuneo | +|IT-CN |Cuneo | +------+------------------------------------+ -|IT-NO |Novara | +|IT-NO |Novara | +------+------------------------------------+ -|IT-TO |Torino | +|IT-TO |Torino | +------+------------------------------------+ -|IT-VB |Verbano-Cusio-Ossola | +|IT-VB |Verbano-Cusio-Ossola | +------+------------------------------------+ -|IT-VC |Vercelli | +|IT-VC |Vercelli | +------+------------------------------------+ -|IT-CA |Cagliari | +|IT-CA |Cagliari | +------+------------------------------------+ -|IT-CI |Carbonia-Iglesias | +|IT-CI |Carbonia-Iglesias | +------+------------------------------------+ -|IT-VS |Medio Campidano | +|IT-VS |Medio Campidano | +------+------------------------------------+ -|IT-NU |Nuoro | +|IT-NU |Nuoro | +------+------------------------------------+ -|IT-OG |Ogliastra | +|IT-OG |Ogliastra | +------+------------------------------------+ -|IT-OT |Olbia-Tempio | +|IT-OT |Olbia-Tempio | +------+------------------------------------+ -|IT-OR |Oristano | +|IT-OR |Oristano | +------+------------------------------------+ -|IT-SS |Sassari | +|IT-SS |Sassari | +------+------------------------------------+ -|IT-AG |Agrigento | +|IT-AG |Agrigento | +------+------------------------------------+ -|IT-CL |Caltanissetta | +|IT-CL |Caltanissetta | +------+------------------------------------+ |IT-CT |Catania | +------+------------------------------------+ |IT-EN |Enna | +------+------------------------------------+ -|IT-ME |Messina | +|IT-ME |Messina | +------+------------------------------------+ |IT-PA |Palermo | +------+------------------------------------+ |IT-RG |Ragusa | +------+------------------------------------+ -|IT-SR |Syracuse | +|IT-SR |Syracuse | +------+------------------------------------+ |IT-TP |Trapani | +------+------------------------------------+ -|IT-AR |Arezzo | +|IT-AR |Arezzo | +------+------------------------------------+ -|IT-FI |Florence | +|IT-FI |Florence | +------+------------------------------------+ -|IT-GR |Grosseto | +|IT-GR |Grosseto | +------+------------------------------------+ -|IT-LI |Livorno | +|IT-LI |Livorno | +------+------------------------------------+ |IT-LU |Lucca | +------+------------------------------------+ -|IT-MS |Massa Carrara | +|IT-MS |Massa Carrara | +------+------------------------------------+ |IT-PI |Pisa | +------+------------------------------------+ -|IT-PT |Pistoia | +|IT-PT |Pistoia | +------+------------------------------------+ |IT-PO |Prato | +------+------------------------------------+ |IT-SI |Siena | +------+------------------------------------+ -|IT-BZ |Bolzano | +|IT-BZ |Bolzano | +------+------------------------------------+ -|IT-TN |Trento | +|IT-TN |Trento | +------+------------------------------------+ -|IT-PG |Perugia | +|IT-PG |Perugia | +------+------------------------------------+ -|IT-TR |Terni | +|IT-TR |Terni | +------+------------------------------------+ -|IT-AO |Aosta | +|IT-AO |Aosta | +------+------------------------------------+ -|IT-BL |Belluno | +|IT-BL |Belluno | +------+------------------------------------+ -|IT-PD |Padua | +|IT-PD |Padua | +------+------------------------------------+ -|IT-RO |Rovigo | +|IT-RO |Rovigo | +------+------------------------------------+ -|IT-TV |Treviso | +|IT-TV |Treviso | +------+------------------------------------+ -|IT-VE |Venezia | +|IT-VE |Venezia | +------+------------------------------------+ -|IT-VR |Verona | +|IT-VR |Verona | +------+------------------------------------+ |IT-VI |Vicenza | +------+------------------------------------+ diff --git a/requirements-dev.txt b/requirements-dev.txt index 53dcbe7ac7..0ac1c57075 100644 --- a/requirements-dev.txt +++ b/requirements-dev.txt @@ -1,16 +1,16 @@ console_log==0.2.10 -python-dotenv==0.10.1 flake8-commas==2.0.0 flake8-import-order==0.18 flake8-quotes==1.0.0 -flake8==3.5.0 -flask-cors==3.0.3 +flake8==3.6.0 +flask-cors==3.0.6 ipdb==0.11 mysqlclient==1.3.13 pip-tools==3.1.0 psycopg2-binary==2.7.5 -pycodestyle==2.3.1 +pycodestyle==2.4.0 pylint==1.9.2 +python-dotenv==0.10.1 redis==2.10.6 -statsd==3.2.2 -tox==3.1.2 +statsd==3.3.0 +tox==3.5.3 diff --git a/superset/db_engine_specs.py b/superset/db_engine_specs.py index bba5e81c54..1a9bf817a4 100644 --- a/superset/db_engine_specs.py +++ b/superset/db_engine_specs.py @@ -1000,7 +1000,7 @@ class HiveEngineSpec(PrestoEngineSpec): @classmethod def patch(cls): - from pyhive import hive + from pyhive import hive # pylint: disable=no-name-in-module from superset.db_engines import hive as patched_hive from TCLIService import ( constants as patched_constants, @@ -1163,7 +1163,7 @@ class HiveEngineSpec(PrestoEngineSpec): @classmethod def handle_cursor(cls, cursor, query, session): """Updates progress information""" - from pyhive import hive + from pyhive import hive # pylint: disable=no-name-in-module unfinished_states = ( hive.ttypes.TOperationState.INITIALIZED_STATE, hive.ttypes.TOperationState.RUNNING_STATE, diff --git a/superset/db_engines/hive.py b/superset/db_engines/hive.py index 8bc6cb9286..f0f8995d9b 100644 --- a/superset/db_engines/hive.py +++ b/superset/db_engines/hive.py @@ -15,7 +15,7 @@ # specific language governing permissions and limitations # under the License. # pylint: disable=C,R,W -from pyhive import hive +from pyhive import hive # pylint: disable=no-name-in-module from TCLIService import ttypes from thrift import Thrift diff --git a/superset/stats_logger.py b/superset/stats_logger.py index bcd8459f31..ce4b40033a 100644 --- a/superset/stats_logger.py +++ b/superset/stats_logger.py @@ -54,8 +54,7 @@ class DummyStatsLogger(BaseStatsLogger): def decr(self, key): logging.debug(( - Fore.CYAN + '[stats_logger] (decr) ' + key + - Style.RESET_ALL)) + Fore.CYAN + '[stats_logger] (decr) ' + key + Style.RESET_ALL)) def timing(self, key, value): logging.debug(( diff --git a/superset/viz.py b/superset/viz.py index 7fd717bb6f..7cf6465ff4 100644 --- a/superset/viz.py +++ b/superset/viz.py @@ -2129,10 +2129,10 @@ class BaseDeckGLViz(BaseViz): return None try: p = Point(s) + return (p.latitude, p.longitude) # pylint: disable=no-member except Exception: raise SpatialException( _('Invalid spatial point encountered: %s' % s)) - return (p.latitude, p.longitude) @staticmethod def reverse_geohash_decode(geohash_code): diff --git a/tests/core_tests.py b/tests/core_tests.py index e765d93a9a..322e974401 100644 --- a/tests/core_tests.py +++ b/tests/core_tests.py @@ -549,7 +549,7 @@ class CoreTests(SupersetTestCase): def test_fetch_datasource_metadata(self): self.login(username='admin') url = ( - '/superset/fetch_datasource_metadata?' + + '/superset/fetch_datasource_metadata?' 'datasourceKey=1__table' ) resp = self.get_json_resp(url) diff --git a/tox.ini b/tox.ini index 6fed8b5180..f23e1009f7 100644 --- a/tox.ini +++ b/tox.ini @@ -20,6 +20,8 @@ ignore = FI51 FI53 FI54 + W504 + W605 import-order-style = google max-line-length = 90 require-code = true