From 7b6d0b0269baf70b6f4c4572939ba276218518b7 Mon Sep 17 00:00:00 2001
From: Tom Augspurger <tom.w.augspurger@gmail.com>
Date: Thu, 18 Jan 2018 15:39:17 -0600
Subject: [PATCH] REF: Change how pandas.core.common shim works

I have a PR incoming that would introduce a circular dependency
between pandas.core.arrays.categorical, pandas.api, and pandas.core.common.

This change will allow that PR to avoid hacky workarounds, since the
real problem is pandas.core importing pandas.api.types.

xref https://github.com/pandas-dev/pandas/pull/13990
---
 pandas/__init__.py    |  3 ++
 pandas/core/common.py | 92 +++++++++++++++++++++++--------------------
 2 files changed, 52 insertions(+), 43 deletions(-)

diff --git a/pandas/__init__.py b/pandas/__init__.py
index 78501620d780b..9d76e4ec31922 100644
--- a/pandas/__init__.py
+++ b/pandas/__init__.py
@@ -58,6 +58,9 @@
 from pandas.io.api import *
 from pandas.util._tester import test
 import pandas.testing
+from pandas.core.common import _add_deprecated
+_add_deprecated()
+del _add_deprecated
 
 # extension module deprecations
 from pandas.util._depr_module import _DeprecatedModule
diff --git a/pandas/core/common.py b/pandas/core/common.py
index e606be3cc2a23..581d784daa3c9 100644
--- a/pandas/core/common.py
+++ b/pandas/core/common.py
@@ -19,7 +19,6 @@
 from pandas.core.dtypes.common import _NS_DTYPE
 from pandas.core.dtypes.inference import _iterable_not_string
 from pandas.core.dtypes.missing import isna, isnull, notnull  # noqa
-from pandas.api import types
 from pandas.core.dtypes import common
 from pandas.core.dtypes.cast import construct_1d_object_array_from_listlike
 
@@ -27,48 +26,54 @@
 from pandas.errors import (  # noqa
     PerformanceWarning, UnsupportedFunctionCall, UnsortedIndexError)
 
-# back-compat of public API
-# deprecate these functions
-m = sys.modules['pandas.core.common']
-for t in [t for t in dir(types) if not t.startswith('_')]:
-
-    def outer(t=t):
-
-        def wrapper(*args, **kwargs):
-            warnings.warn("pandas.core.common.{t} is deprecated. "
-                          "import from the public API: "
-                          "pandas.api.types.{t} instead".format(t=t),
-                          DeprecationWarning, stacklevel=3)
-            return getattr(types, t)(*args, **kwargs)
-        return wrapper
-
-    setattr(m, t, outer(t))
-
-# back-compat for non-public functions
-# deprecate these functions
-for t in ['is_datetime_arraylike',
-          'is_datetime_or_timedelta_dtype',
-          'is_datetimelike',
-          'is_datetimelike_v_numeric',
-          'is_datetimelike_v_object',
-          'is_datetimetz',
-          'is_int_or_datetime_dtype',
-          'is_period_arraylike',
-          'is_string_like',
-          'is_string_like_dtype']:
-
-    def outer(t=t):
-
-        def wrapper(*args, **kwargs):
-            warnings.warn("pandas.core.common.{t} is deprecated. "
-                          "These are not longer public API functions, "
-                          "but can be imported from "
-                          "pandas.api.types.{t} instead".format(t=t),
-                          DeprecationWarning, stacklevel=3)
-            return getattr(common, t)(*args, **kwargs)
-        return wrapper
-
-    setattr(m, t, outer(t))
+
+def _add_deprecated():
+    # back-compat of public API
+    # deprecate these functions
+    # This is called at the end of pandas.__init__, after all the imports
+    # have already been resolved, to avoid any circular imports.
+    from pandas.api import types
+
+    m = sys.modules['pandas.core.common']
+    for t in [t for t in dir(types) if not t.startswith('_')]:
+
+        def outer(t=t):
+
+            def wrapper(*args, **kwargs):
+                warnings.warn("pandas.core.common.{t} is deprecated. "
+                              "import from the public API: "
+                              "pandas.api.types.{t} instead".format(t=t),
+                              DeprecationWarning, stacklevel=3)
+                return getattr(types, t)(*args, **kwargs)
+            return wrapper
+
+        setattr(m, t, outer(t))
+
+    # back-compat for non-public functions
+    # deprecate these functions
+    for t in ['is_datetime_arraylike',
+              'is_datetime_or_timedelta_dtype',
+              'is_datetimelike',
+              'is_datetimelike_v_numeric',
+              'is_datetimelike_v_object',
+              'is_datetimetz',
+              'is_int_or_datetime_dtype',
+              'is_period_arraylike',
+              'is_string_like',
+              'is_string_like_dtype']:
+
+        def outer(t=t):
+
+            def wrapper(*args, **kwargs):
+                warnings.warn("pandas.core.common.{t} is deprecated. "
+                              "These are not longer public API functions, "
+                              "but can be imported from "
+                              "pandas.api.types.{t} instead".format(t=t),
+                              DeprecationWarning, stacklevel=3)
+                return getattr(common, t)(*args, **kwargs)
+            return wrapper
+
+        setattr(m, t, outer(t))
 
 
 # deprecate array_equivalent
@@ -646,6 +651,7 @@ def _random_state(state=None):
     -------
     np.random.RandomState
     """
+    from pandas.api import types
 
     if types.is_integer(state):
         return np.random.RandomState(state)