CLN: various lgtm.com cleanups (#30258)

simonjayhawkins · jreback · commit 8c6a994b2ecc · 2019-12-15T16:46:16.000-05:00
diff --git a/pandas/core/apply.py b/pandas/core/apply.py
@@ -320,7 +320,6 @@ def apply_series_generator(self) -> Tuple[ResType, "Index"]:
         series_gen = self.series_generator
         res_index = self.result_index
 
-        i = None
         keys = []
         results = {}
         if self.ignore_failures:
diff --git a/pandas/core/arrays/datetimes.py b/pandas/core/arrays/datetimes.py
@@ -2109,7 +2109,6 @@ def _validate_dt64_dtype(dtype):
         dtype = pandas_dtype(dtype)
         if is_dtype_equal(dtype, np.dtype("M8")):
             # no precision, disallowed GH#24806
-            dtype = _NS_DTYPE
             msg = (
                 "Passing in 'datetime64' dtype with no precision is not allowed. "
                 "Please pass in 'datetime64[ns]' instead."
diff --git a/pandas/core/arrays/integer.py b/pandas/core/arrays/integer.py
@@ -78,7 +78,8 @@ def itemsize(self):
 
     @classmethod
     def construct_array_type(cls):
-        """Return the array type associated with this dtype
+        """
+        Return the array type associated with this dtype.
 
         Returns
         -------
diff --git a/pandas/core/arrays/numpy_.py b/pandas/core/arrays/numpy_.py
@@ -79,7 +79,15 @@ def construct_from_string(cls, string):
                 f"Cannot construct a 'PandasDtype' from '{string}'"
             ) from err
 
+    @classmethod
     def construct_array_type(cls):
+        """
+        Return the array type associated with this dtype.
+
+        Returns
+        -------
+        type
+        """
         return PandasArray
 
     @property
diff --git a/pandas/core/arrays/sparse/dtype.py b/pandas/core/arrays/sparse/dtype.py
@@ -168,6 +168,13 @@ def __repr__(self) -> str:
 
     @classmethod
     def construct_array_type(cls):
+        """
+        Return the array type associated with this dtype.
+
+        Returns
+        -------
+        type
+        """
         from .array import SparseArray
 
         return SparseArray
diff --git a/pandas/core/arrays/timedeltas.py b/pandas/core/arrays/timedeltas.py
@@ -1122,7 +1122,6 @@ def _validate_td64_dtype(dtype):
     dtype = pandas_dtype(dtype)
     if is_dtype_equal(dtype, np.dtype("timedelta64")):
         # no precision disallowed GH#24806
-        dtype = _TD_DTYPE
         msg = (
             "Passing in 'timedelta' dtype with no precision is not allowed. "
             "Please pass in 'timedelta64[ns]' instead."
diff --git a/pandas/core/dtypes/base.py b/pandas/core/dtypes/base.py
@@ -183,7 +183,7 @@ def names(self) -> Optional[List[str]]:
     @classmethod
     def construct_array_type(cls):
         """
-        Return the array type associated with this dtype
+        Return the array type associated with this dtype.
 
         Returns
         -------
diff --git a/pandas/core/dtypes/dtypes.py b/pandas/core/dtypes/dtypes.py
@@ -486,7 +486,7 @@ def _hash_categories(categories, ordered: Ordered = True) -> int:
     @classmethod
     def construct_array_type(cls):
         """
-        Return the array type associated with this dtype
+        Return the array type associated with this dtype.
 
         Returns
         -------
@@ -668,7 +668,7 @@ def __init__(self, unit="ns", tz=None):
                 unit = result.unit
                 tz = result.tz
                 msg = (
-                    "Passing a dtype alias like 'datetime64[ns, {tz}]' "
+                    f"Passing a dtype alias like 'datetime64[ns, {tz}]' "
                     "to DatetimeTZDtype is no longer supported. Use "
                     "'DatetimeTZDtype.construct_from_string()' instead."
                 )
@@ -704,7 +704,7 @@ def tz(self):
     @classmethod
     def construct_array_type(cls):
         """
-        Return the array type associated with this dtype
+        Return the array type associated with this dtype.
 
         Returns
         -------
@@ -936,6 +936,13 @@ def is_dtype(cls, dtype) -> bool:
 
     @classmethod
     def construct_array_type(cls):
+        """
+        Return the array type associated with this dtype.
+
+        Returns
+        -------
+        type
+        """
         from pandas.core.arrays import PeriodArray
 
         return PeriodArray
@@ -1030,7 +1037,7 @@ def subtype(self):
     @classmethod
     def construct_array_type(cls):
         """
-        Return the array type associated with this dtype
+        Return the array type associated with this dtype.
 
         Returns
         -------
diff --git a/pandas/core/frame.py b/pandas/core/frame.py
@@ -2587,7 +2587,6 @@ def _ixs(self, i: int, axis: int = 0):
         """
         # irow
         if axis == 0:
-            label = self.index[i]
             new_values = self._data.fast_xs(i)
 
             # if we are a copy, mark as such
diff --git a/pandas/core/generic.py b/pandas/core/generic.py
@@ -5746,8 +5746,6 @@ def __deepcopy__(self: FrameOrSeries, memo=None) -> FrameOrSeries:
         memo, default None
             Standard signature. Unused
         """
-        if memo is None:
-            memo = {}
         return self.copy(deep=True)
 
     def _convert(
diff --git a/pandas/core/groupby/grouper.py b/pandas/core/groupby/grouper.py
@@ -131,7 +131,7 @@ def _get_grouper(self, obj, validate: bool = True):
         """
 
         self._set_grouper(obj)
-        self.grouper, exclusions, self.obj = get_grouper(
+        self.grouper, _, self.obj = get_grouper(
             self.obj,
             [self.key],
             axis=self.axis,
diff --git a/pandas/core/indexes/base.py b/pandas/core/indexes/base.py
@@ -911,8 +911,6 @@ def __deepcopy__(self, memo=None):
         memo, default None
             Standard signature. Unused
         """
-        if memo is None:
-            memo = {}
         return self.copy(deep=True)
 
     # --------------------------------------------------------------------
diff --git a/pandas/core/internals/blocks.py b/pandas/core/internals/blocks.py
@@ -3024,7 +3024,6 @@ def _merge_blocks(blocks, dtype=None, _can_consolidate=True):
         if dtype is None:
             if len({b.dtype for b in blocks}) != 1:
                 raise AssertionError("_merge_blocks are invalid!")
-            dtype = blocks[0].dtype
 
         # FIXME: optimization potential in case all mgrs contain slices and
         # combination of those slices is a slice, too.
diff --git a/pandas/core/reshape/reshape.py b/pandas/core/reshape/reshape.py
@@ -230,11 +230,9 @@ def get_new_values(self):
         if needs_i8_conversion(values):
             sorted_values = sorted_values.view("i8")
             new_values = new_values.view("i8")
-            name = "int64"
         elif is_bool_dtype(values):
             sorted_values = sorted_values.astype("object")
             new_values = new_values.astype("object")
-            name = "object"
         else:
             sorted_values = sorted_values.astype(name, copy=False)
 
diff --git a/pandas/plotting/_matplotlib/hist.py b/pandas/plotting/_matplotlib/hist.py
@@ -27,7 +27,7 @@ def _args_adjust(self):
             values = np.ravel(values)
             values = values[~isna(values)]
 
-            hist, self.bins = np.histogram(
+            _, self.bins = np.histogram(
                 values,
                 bins=self.bins,
                 range=self.kwds.get("range", None),
diff --git a/pandas/tests/arrays/test_array.py b/pandas/tests/arrays/test_array.py
@@ -287,6 +287,13 @@ class DecimalDtype2(DecimalDtype):
 
     @classmethod
     def construct_array_type(cls):
+        """
+        Return the array type associated with this dtype.
+
+        Returns
+        -------
+        type
+        """
         return DecimalArray2
 
 
diff --git a/pandas/tests/extension/arrow/arrays.py b/pandas/tests/extension/arrow/arrays.py
@@ -37,6 +37,13 @@ def construct_from_string(cls, string):
 
     @classmethod
     def construct_array_type(cls):
+        """
+        Return the array type associated with this dtype.
+
+        Returns
+        -------
+        type
+        """
         return ArrowBoolArray
 
     def _is_boolean(self):
@@ -60,6 +67,13 @@ def construct_from_string(cls, string):
 
     @classmethod
     def construct_array_type(cls):
+        """
+        Return the array type associated with this dtype.
+
+        Returns
+        -------
+        type
+        """
         return ArrowStringArray
 
 
diff --git a/pandas/tests/extension/decimal/array.py b/pandas/tests/extension/decimal/array.py
@@ -27,7 +27,8 @@ def __repr__(self) -> str:
 
     @classmethod
     def construct_array_type(cls):
-        """Return the array type associated with this dtype
+        """
+        Return the array type associated with this dtype.
 
         Returns
         -------
diff --git a/pandas/tests/extension/json/array.py b/pandas/tests/extension/json/array.py
@@ -31,7 +31,8 @@ class JSONDtype(ExtensionDtype):
 
     @classmethod
     def construct_array_type(cls):
-        """Return the array type associated with this dtype
+        """
+        Return the array type associated with this dtype.
 
         Returns
         -------
