Merge pull request #667 from plotly/ff-requires-numpy

FF requires numpy

Author: theengineear

CHANGELOG.md

@@ -12,6 +12,8 @@ which depend on `try-except` blocks containing network requests should be
 revisited.
 - `plotly.py:sign_in` now validates to the plotly server specified in your
   config. If it cannot make a successful request, it raises a `PlotlyError`.
+- `plotly.figure_factory` will raise an `ImportError` if `numpy` is not
+  installed.
 
 ### Deprecated
 - `plotly.tools.FigureFactory`. Use `plotly.figure_factory.*`.

circle.yml

@@ -52,5 +52,9 @@ test:
     - sudo chmod -R 444 ${PLOTLY_CONFIG_DIR} && python -c "import plotly"
 
     # test that giving back write permissions works again
-    # this also has to pass the test suite that follows
     - sudo chmod -R 777 ${PLOTLY_CONFIG_DIR} && python -c "import plotly"
+
+    # test that figure_factory cannot be imported with only core requirements.
+    # since optional requirements is part of the test suite, we don't need to
+    # worry about testing that it *can* be imported in this case.
+    - $(! python -c "import plotly.figure_factory")

plotly/figure_factory/__init__.py

@@ -1,5 +1,8 @@
 from __future__ import absolute_import
 
+# Require that numpy exists for figure_factory
+import numpy
+
 from plotly.figure_factory._2d_density import create_2d_density
 from plotly.figure_factory._annotated_heatmap import create_annotated_heatmap
 from plotly.figure_factory._candlestick import create_candlestick

plotly/tests/test_optional/test_tools/__init__.py

@@ -36,6 +36,7 @@ def readme():
                 'plotly/api/v2',
                 'plotly/plotly',
                 'plotly/plotly/chunked_requests',
+                'plotly/figure_factory',
                 'plotly/graph_objs',
                 'plotly/grid_objs',
                 'plotly/widgets',