CLN: Prune unnecessary internals (#27685)

Author: jbrockmendel

pandas/core/groupby/generic.py

@@ -671,7 +671,7 @@ def _transform_item_by_item(self, obj, wrapper):
             except Exception:
                 pass
 
-        if len(output) == 0:  # pragma: no cover
+        if len(output) == 0:
             raise TypeError("Transform function invalid for data types")
 
         columns = obj.columns

pandas/core/groupby/groupby.py

@@ -1206,7 +1206,7 @@ def mean(self, *args, **kwargs):
             )
         except GroupByError:
             raise
-        except Exception:  # pragma: no cover
+        except Exception:
             with _group_selection_context(self):
                 f = lambda x: x.mean(axis=self.axis, **kwargs)
                 return self._python_agg_general(f)
@@ -1232,7 +1232,7 @@ def median(self, **kwargs):
             )
         except GroupByError:
             raise
-        except Exception:  # pragma: no cover
+        except Exception:
 
             def f(x):
                 if isinstance(x, np.ndarray):
@@ -2470,7 +2470,7 @@ def groupby(obj, by, **kwds):
         from pandas.core.groupby.generic import DataFrameGroupBy
 
         klass = DataFrameGroupBy
-    else:  # pragma: no cover
+    else:
         raise TypeError("invalid type: {}".format(obj))
 
     return klass(obj, by, **kwds)

pandas/core/internals/blocks.py

@@ -760,7 +760,7 @@ def to_native_types(self, slicer=None, na_rep="nan", quoting=None, **kwargs):
         values[mask] = na_rep
         return values
 
-    # block actions ####
+    # block actions #
     def copy(self, deep=True):
         """ copy constructor """
         values = self.values
@@ -1538,16 +1538,14 @@ def quantile(self, qs, interpolation="linear", axis=0):
             ).reshape(len(values), len(qs))
         else:
             # asarray needed for Sparse, see GH#24600
-            # Note: we use self.values below instead of values because the
-            #  `asi8` conversion above will behave differently under `isna`
-            mask = np.asarray(isna(self.values))
+            mask = np.asarray(isna(values))
             result = nanpercentile(
                 values,
                 np.array(qs) * 100,
                 axis=axis,
                 na_value=self.fill_value,
                 mask=mask,
-                ndim=self.ndim,
+                ndim=values.ndim,
                 interpolation=interpolation,
             )
 

pandas/core/internals/managers.py

@@ -975,8 +975,6 @@ def iget(self, i):
         """
         block = self.blocks[self._blknos[i]]
         values = block.iget(self._blklocs[i])
-        if values.ndim != 1:
-            return values
 
         # shortcut for select a single-dim from a 2-dim BM
         return SingleBlockManager(

pandas/io/clipboards.py

@@ -121,7 +121,7 @@ def to_clipboard(obj, excel=True, sep=None, **kwargs):  # pragma: no cover
             return
         except TypeError:
             warnings.warn(
-                "to_clipboard in excel mode requires a single " "character separator."
+                "to_clipboard in excel mode requires a single character separator."
             )
     elif sep is not None:
         warnings.warn("to_clipboard with excel=False ignores the sep argument")

pandas/io/excel/_base.py

@@ -297,7 +297,7 @@ def read_excel(
     for arg in ("sheet", "sheetname", "parse_cols"):
         if arg in kwds:
             raise TypeError(
-                "read_excel() got an unexpected keyword argument " "`{}`".format(arg)
+                "read_excel() got an unexpected keyword argument `{}`".format(arg)
             )
 
     if not isinstance(io, ExcelFile):
@@ -353,7 +353,7 @@ def __init__(self, filepath_or_buffer):
             self.book = self.load_workbook(filepath_or_buffer)
         else:
             raise ValueError(
-                "Must explicitly set engine if not passing in" " buffer or path for io."
+                "Must explicitly set engine if not passing in buffer or path for io."
             )
 
     @property
@@ -713,9 +713,7 @@ def _get_sheet_name(self, sheet_name):
         if sheet_name is None:
             sheet_name = self.cur_sheet
         if sheet_name is None:  # pragma: no cover
-            raise ValueError(
-                "Must pass explicit sheet_name or set " "cur_sheet property"
-            )
+            raise ValueError("Must pass explicit sheet_name or set cur_sheet property")
         return sheet_name
 
     def _value_with_fmt(self, val):
@@ -851,7 +849,7 @@ def parse(
         """
         if "chunksize" in kwds:
             raise NotImplementedError(
-                "chunksize keyword of read_excel " "is not implemented"
+                "chunksize keyword of read_excel is not implemented"
             )
 
         return self._reader.parse(

pandas/io/feather_format.py

@@ -53,7 +53,7 @@ def to_feather(df, path):
 
     if df.index.name is not None:
         raise ValueError(
-            "feather does not serialize index meta-data on a " "default index"
+            "feather does not serialize index meta-data on a default index"
         )
 
     # validate columns

pandas/io/formats/csvs.py

@@ -96,9 +96,7 @@ def __init__(
         # validate mi options
         if self.has_mi_columns:
             if cols is not None:
-                raise TypeError(
-                    "cannot specify cols with a MultiIndex on the " "columns"
-                )
+                raise TypeError("cannot specify cols with a MultiIndex on the columns")
 
         if cols is not None:
             if isinstance(cols, ABCIndexClass):
@@ -158,7 +156,7 @@ def save(self):
         """
         # GH21227 internal compression is not used when file-like passed.
         if self.compression and hasattr(self.path_or_buf, "write"):
-            msg = "compression has no effect when passing file-like " "object as input."
+            msg = "compression has no effect when passing file-like object as input."
             warnings.warn(msg, RuntimeWarning, stacklevel=2)
 
         # when zip compression is called.

pandas/io/formats/format.py

@@ -2,9 +2,10 @@
 Internal module for formatting output data in csv, html,
 and latex files. This module also applies to display formatting.
 """
-
+import decimal
 from functools import partial
 from io import StringIO
+import math
 import re
 from shutil import get_terminal_size
 from typing import (
@@ -862,7 +863,7 @@ def to_latex(
             with codecs.open(self.buf, "w", encoding=encoding) as f:
                 latex_renderer.write_result(f)
         else:
-            raise TypeError("buf is not a file name and it has no write " "method")
+            raise TypeError("buf is not a file name and it has no write method")
 
     def _format_col(self, i: int) -> List[str]:
         frame = self.tr_frame
@@ -907,7 +908,7 @@ def to_html(
             with open(self.buf, "w") as f:
                 buffer_put_lines(f, html)
         else:
-            raise TypeError("buf is not a file name and it has no write " " method")
+            raise TypeError("buf is not a file name and it has no write  method")
 
     def _get_formatted_column_labels(self, frame: "DataFrame") -> List[List[str]]:
         from pandas.core.index import _sparsify
@@ -1782,9 +1783,6 @@ def __call__(self, num: Union[int, float]) -> str:
 
         @return: engineering formatted string
         """
-        import decimal
-        import math
-
         dnum = decimal.Decimal(str(num))
 
         if decimal.Decimal.is_nan(dnum):

pandas/io/parsers.py

@@ -687,7 +687,7 @@ def parser_f(
 read_csv = Appender(
     _doc_read_csv_and_table.format(
         func_name="read_csv",
-        summary=("Read a comma-separated values (csv) file " "into DataFrame."),
+        summary=("Read a comma-separated values (csv) file into DataFrame."),
         _default_sep="','",
     )
 )(read_csv)
@@ -770,7 +770,7 @@ def read_fwf(
     if colspecs is None and widths is None:
         raise ValueError("Must specify either colspecs or widths")
     elif colspecs not in (None, "infer") and widths is not None:
-        raise ValueError("You must specify only one of 'widths' and " "'colspecs'")
+        raise ValueError("You must specify only one of 'widths' and 'colspecs'")
 
     # Compute 'colspecs' from 'widths', if specified.
     if widths is not None:
@@ -901,9 +901,7 @@ def _get_options_with_defaults(self, engine):
 
             # see gh-12935
             if argname == "mangle_dupe_cols" and not value:
-                raise ValueError(
-                    "Setting mangle_dupe_cols=False is " "not supported yet"
-                )
+                raise ValueError("Setting mangle_dupe_cols=False is not supported yet")
             else:
                 options[argname] = value
 
@@ -942,7 +940,7 @@ def _check_file_or_buffer(self, f, engine):
             # needs to have that attribute ("next" for Python 2.x, "__next__"
             # for Python 3.x)
             if engine != "c" and not hasattr(f, next_attr):
-                msg = "The 'python' engine cannot iterate " "through this file buffer."
+                msg = "The 'python' engine cannot iterate through this file buffer."
                 raise ValueError(msg)
 
         return engine
@@ -959,7 +957,7 @@ def _clean_options(self, options, engine):
         # C engine not supported yet
         if engine == "c":
             if options["skipfooter"] > 0:
-                fallback_reason = "the 'c' engine does not support" " skipfooter"
+                fallback_reason = "the 'c' engine does not support skipfooter"
                 engine = "python"
 
         encoding = sys.getfilesystemencoding() or "utf-8"
@@ -1397,11 +1395,11 @@ def __init__(self, kwds):
                 raise ValueError("header must be integer or list of integers")
             if kwds.get("usecols"):
                 raise ValueError(
-                    "cannot specify usecols when " "specifying a multi-index header"
+                    "cannot specify usecols when specifying a multi-index header"
                 )
             if kwds.get("names"):
                 raise ValueError(
-                    "cannot specify names when " "specifying a multi-index header"
+                    "cannot specify names when specifying a multi-index header"
                 )
 
             # validate index_col that only contains integers
@@ -1611,7 +1609,7 @@ def _get_name(icol):
 
             if col_names is None:
                 raise ValueError(
-                    ("Must supply column order to use {icol!s} " "as index").format(
+                    ("Must supply column order to use {icol!s} as index").format(
                         icol=icol
                     )
                 )
@@ -2379,7 +2377,7 @@ def _make_reader(self, f):
         if sep is None or len(sep) == 1:
             if self.lineterminator:
                 raise ValueError(
-                    "Custom line terminators not supported in " "python parser (yet)"
+                    "Custom line terminators not supported in python parser (yet)"
                 )
 
             class MyDialect(csv.Dialect):
@@ -2662,7 +2660,7 @@ def _infer_columns(self):
                         "number of header fields in the file"
                     )
                 if len(columns) > 1:
-                    raise TypeError("Cannot pass names with multi-index " "columns")
+                    raise TypeError("Cannot pass names with multi-index columns")
 
                 if self.usecols is not None:
                     # Set _use_cols. We don't store columns because they are
@@ -2727,7 +2725,7 @@ def _handle_usecols(self, columns, usecols_key):
             elif any(isinstance(u, str) for u in self.usecols):
                 if len(columns) > 1:
                     raise ValueError(
-                        "If using multiple headers, usecols must " "be integers."
+                        "If using multiple headers, usecols must be integers."
                     )
                 col_indices = []