many things

Author: TomAugspurger

doc/source/install.rst

@@ -279,6 +279,7 @@ pandas-gbq                0.8.0              Google Big Query access
 psycopg2                                     PostgreSQL engine for sqlalchemy
 pyarrow                   0.9.0              Parquet and feather reading / writing
 pymysql                                      MySQL engine for sqlalchemy
+pytables                  3.4.2              HDF5 reading / writing
 qtpy                                         Clipboard I/O
 s3fs                      0.0.8              Amazon S3 access
 xarray                    0.8.2              pandas-like API for N-dimensional data

pandas/compat/_optional.py

@@ -15,7 +15,7 @@
     "openpyxl": "2.4.0",
     "pandas_gbq": "0.8.0",
     "pyarrow": "0.9.0",
-    "pytables": "3.4.2",
+    "tables": "3.4.2",
     "s3fs": "0.0.8",
     "scipy": "0.19.0",
     "sqlalchemy": "1.1.4",

pandas/io/pytables.py

@@ -19,6 +19,7 @@
 
 from pandas._libs import lib, writers as libwriters
 from pandas._libs.tslibs import timezones
+from pandas.compat._optional import import_optional_dependency
 from pandas.errors import PerformanceWarning
 
 from pandas.core.dtypes.common import (
@@ -448,11 +449,7 @@ def __init__(self, path, mode=None, complevel=None, complib=None,
         if 'format' in kwargs:
             raise ValueError('format is not a defined argument for HDFStore')
 
-        try:
-            import tables  # noqa
-        except ImportError as ex:  # pragma: no cover
-            raise ImportError('HDFStore requires PyTables, "{ex!s}" problem '
-                              'importing'.format(ex=ex))
+        tables = import_optional_dependency("tables")
 
         if complib is not None and complib not in tables.filters.all_complibs:
             raise ValueError(

pandas/tests/test_optional_dependency.py

@@ -4,7 +4,9 @@
 import pytest
 
 from pandas.compat._optional import VERSIONS, import_optional_dependency
+import pandas.util._test_decorators as td
 
+import pandas as pd
 import pandas.util.testing as tm
 
 
@@ -50,3 +52,11 @@ def test_no_version_raises():
 
     with pytest.raises(ImportError, match="Can't determine .* fakemodule"):
         import_optional_dependency(name)
+
+
+@td.skip_if_installed("tables")
+def test_pytables_raises():
+    df = pd.DataFrame({"A": [1, 2]})
+    with pytest.raises(ImportError, match="tables"):
+        with tm.ensure_clean("foo.h5") as path:
+            df.to_hdf(path, "df")

pandas/util/_print_versions.py

@@ -6,7 +6,8 @@
 import subprocess
 import sys
 
-from pandas.compat._optional import _get_version, import_optional_dependency
+from pandas.compat._optional import (
+    VERSIONS, _get_version, import_optional_dependency)
 
 
 def get_sys_info():
@@ -59,45 +60,35 @@ def get_sys_info():
 
 def show_versions(as_json=False):
     sys_info = get_sys_info()
-
-    deps = [
-        "pandas",
-        "pytest",
-        "pip",
-        "setuptools",
-        "Cython",
-        "numpy",
-        "scipy",
-        "pyarrow",
-        "xarray",
-        "IPython",
+    deps = list(VERSIONS)
+    deps.extend([
+        'pandas',
+        # required
+        'numpy',
+        'pytz',
+        'dateutil',
+        # install / build,
+        'pip',
+        'setuptools',
+        'Cython',
+        # test
+        'pytest',
+        'hypothesis',
+        # docs
         "sphinx",
-        "patsy",
-        "dateutil",
-        "pytz",
+        # Other, need a min version
         "blosc",
-        "bottleneck",
-        "tables",
-        "numexpr",
         "feather",
-        "matplotlib",
-        "openpyxl",
-        "xlrd",
-        "xlwt",
         "xlsxwriter",
         "lxml.etree",
-        "bs4",
         "html5lib",
-        "sqlalchemy",
         "pymysql",
         "psycopg2",
         "jinja2",
-        "s3fs",
-        "fastparquet",
-        "pandas_gbq",
+        # Other, not imported.
+        "IPython",
         "pandas_datareader",
-        "gcsfs",
-    ]
+    ])
 
     deps_blob = list()
     for modname in deps:

pandas/util/_test_decorators.py

@@ -26,12 +26,12 @@ def test_foo():
 import locale
 from typing import Optional
 
-from _pytest.mark.structures import MarkDecorator
 import pytest
 
 from pandas.compat import is_platform_32bit, is_platform_windows
 from pandas.compat.numpy import _np_version_under1p15
 
+from _pytest.mark.structures import MarkDecorator
 from pandas.core.computation.expressions import (
     _NUMEXPR_INSTALLED, _USE_NUMEXPR)
 
@@ -140,6 +140,17 @@ def skip_if_no(
     )
 
 
+def skip_if_installed(
+    package: str,
+    min_version: Optional[str] = None
+) -> MarkDecorator:
+    """Skip if a package *is* present."""
+    msg = "Skipping because {} is installed.".format(package)
+    return pytest.mark.skipif(
+        safe_import(package, min_version=min_version), reason=msg
+    )
+
+
 skip_if_no_mpl = pytest.mark.skipif(_skip_if_no_mpl(),
                                     reason="Missing matplotlib dependency")
 skip_if_np_lt_115 = pytest.mark.skipif(_np_version_under1p15,