Merge branch 'main' into pylint-48855-C-type-disallowed-name

Author: WillAyd

doc/source/user_guide/options.rst

@@ -249,7 +249,7 @@ displayed when calling :meth:`~pandas.DataFrame.info`.
 ``display.max_info_rows``: :meth:`~pandas.DataFrame.info` will usually show null-counts for each column.
 For a large :class:`DataFrame`, this can be quite slow. ``max_info_rows`` and ``max_info_cols``
 limit this null check to the specified rows and columns respectively. The :meth:`~pandas.DataFrame.info`
-keyword argument ``null_counts=True`` will override this.
+keyword argument ``show_counts=True`` will override this.
 
 .. ipython:: python
 

doc/source/whatsnew/v2.0.0.rst

@@ -181,6 +181,7 @@ Removal of prior version deprecations/changes
 - Removed deprecated :meth:`.Styler.where` (:issue:`49397`)
 - Removed deprecated :meth:`.Styler.render` (:issue:`49397`)
 - Removed deprecated argument ``null_color`` in :meth:`.Styler.highlight_null` (:issue:`49397`)
+- Removed deprecated ``null_counts`` argument in :meth:`DataFrame.info`. Use ``show_counts`` instead (:issue:`37999`)
 - Enforced deprecation disallowing passing a timezone-aware :class:`Timestamp` and ``dtype="datetime64[ns]"`` to :class:`Series` or :class:`DataFrame` constructors (:issue:`41555`)
 - Enforced deprecation disallowing passing a sequence of timezone-aware values and ``dtype="datetime64[ns]"`` to to :class:`Series` or :class:`DataFrame` constructors (:issue:`41555`)
 - Enforced deprecation disallowing unit-less "datetime64" dtype in :meth:`Series.astype` and :meth:`DataFrame.astype` (:issue:`47844`)

pandas/core/computation/engines.py

@@ -102,7 +102,6 @@ def _evaluate(self):
         -----
         Must be implemented by subclasses.
         """
-        pass
 
 
 class NumExprEngine(AbstractEngine):

pandas/core/dtypes/dtypes.py

@@ -116,8 +116,6 @@ class CategoricalDtypeType(type):
     the type of CategoricalDtype, this metaclass determines subclass ability
     """
 
-    pass
-
 
 @register_extension_dtype
 class CategoricalDtype(PandasExtensionDtype, ExtensionDtype):

pandas/core/frame.py

@@ -3398,17 +3398,7 @@ def info(
         max_cols: int | None = None,
         memory_usage: bool | str | None = None,
         show_counts: bool | None = None,
-        null_counts: bool | None = None,
     ) -> None:
-        if null_counts is not None:
-            if show_counts is not None:
-                raise ValueError("null_counts used with show_counts. Use show_counts.")
-            warnings.warn(
-                "null_counts is deprecated. Use show_counts instead",
-                FutureWarning,
-                stacklevel=find_stack_level(),
-            )
-            show_counts = null_counts
         info = DataFrameInfo(
             data=self,
             memory_usage=memory_usage,

pandas/core/indexes/multi.py

@@ -3610,7 +3610,6 @@ def _union(self, other, sort) -> MultiIndex:
                         RuntimeWarning,
                         stacklevel=find_stack_level(),
                     )
-                    pass
             return result
 
     def _is_comparable_dtype(self, dtype: DtypeObj) -> bool:

pandas/core/interchange/dataframe_protocol.py

@@ -134,15 +134,13 @@ def bufsize(self) -> int:
         """
         Buffer size in bytes.
         """
-        pass
 
     @property
     @abstractmethod
     def ptr(self) -> int:
         """
         Pointer to start of the buffer as an integer.
         """
-        pass
 
     @abstractmethod
     def __dlpack__(self):
@@ -166,7 +164,6 @@ def __dlpack_device__(self) -> tuple[DlpackDeviceType, int | None]:
         Uses device type codes matching DLPack.
         Note: must be implemented even if ``__dlpack__`` is not.
         """
-        pass
 
 
 class Column(ABC):
@@ -222,7 +219,6 @@ def size(self) -> int:
         Corresponds to DataFrame.num_rows() if column is a single chunk;
         equal to size of this current chunk otherwise.
         """
-        pass
 
     @property
     @abstractmethod
@@ -234,7 +230,6 @@ def offset(self) -> int:
         equal size M (only the last chunk may be shorter),
         ``offset = n * M``, ``n = 0 .. N-1``.
         """
-        pass
 
     @property
     @abstractmethod
@@ -266,7 +261,6 @@ def dtype(self) -> tuple[DtypeKind, int, str, str]:
             - Data types not included: complex, Arrow-style null, binary, decimal,
               and nested (list, struct, map, union) dtypes.
         """
-        pass
 
     @property
     @abstractmethod
@@ -289,7 +283,6 @@ def describe_categorical(self) -> CategoricalDescription:
 
         TBD: are there any other in-memory representations that are needed?
         """
-        pass
 
     @property
     @abstractmethod
@@ -302,7 +295,6 @@ def describe_null(self) -> tuple[ColumnNullType, Any]:
         mask or a byte mask, the value (0 or 1) indicating a missing value. None
         otherwise.
         """
-        pass
 
     @property
     @abstractmethod
@@ -312,22 +304,19 @@ def null_count(self) -> int | None:
 
         Note: Arrow uses -1 to indicate "unknown", but None seems cleaner.
         """
-        pass
 
     @property
     @abstractmethod
     def metadata(self) -> dict[str, Any]:
         """
         The metadata for the column. See `DataFrame.metadata` for more details.
         """
-        pass
 
     @abstractmethod
     def num_chunks(self) -> int:
         """
         Return the number of chunks the column consists of.
         """
-        pass
 
     @abstractmethod
     def get_chunks(self, n_chunks: int | None = None) -> Iterable[Column]:
@@ -336,7 +325,6 @@ def get_chunks(self, n_chunks: int | None = None) -> Iterable[Column]:
 
         See `DataFrame.get_chunks` for details on ``n_chunks``.
         """
-        pass
 
     @abstractmethod
     def get_buffers(self) -> ColumnBuffers:
@@ -360,7 +348,6 @@ def get_buffers(self) -> ColumnBuffers:
                          if the data buffer does not have an associated offsets
                          buffer.
         """
-        pass
 
 
 #    def get_children(self) -> Iterable[Column]:
@@ -391,7 +378,6 @@ class DataFrame(ABC):
     @abstractmethod
     def __dataframe__(self, nan_as_null: bool = False, allow_copy: bool = True):
         """Construct a new interchange object, potentially changing the parameters."""
-        pass
 
     @property
     @abstractmethod
@@ -405,14 +391,12 @@ def metadata(self) -> dict[str, Any]:
         entries, please add name the keys with the name of the library
         followed by a period and the desired name, e.g, ``pandas.indexcol``.
         """
-        pass
 
     @abstractmethod
     def num_columns(self) -> int:
         """
         Return the number of columns in the DataFrame.
         """
-        pass
 
     @abstractmethod
     def num_rows(self) -> int | None:
@@ -422,56 +406,48 @@ def num_rows(self) -> int | None:
         """
         Return the number of rows in the DataFrame, if available.
         """
-        pass
 
     @abstractmethod
     def num_chunks(self) -> int:
         """
         Return the number of chunks the DataFrame consists of.
         """
-        pass
 
     @abstractmethod
     def column_names(self) -> Iterable[str]:
         """
         Return an iterator yielding the column names.
         """
-        pass
 
     @abstractmethod
     def get_column(self, i: int) -> Column:
         """
         Return the column at the indicated position.
         """
-        pass
 
     @abstractmethod
     def get_column_by_name(self, name: str) -> Column:
         """
         Return the column whose name is the indicated name.
         """
-        pass
 
     @abstractmethod
     def get_columns(self) -> Iterable[Column]:
         """
         Return an iterator yielding the columns.
         """
-        pass
 
     @abstractmethod
     def select_columns(self, indices: Sequence[int]) -> DataFrame:
         """
         Create a new DataFrame by selecting a subset of columns by index.
         """
-        pass
 
     @abstractmethod
     def select_columns_by_name(self, names: Sequence[str]) -> DataFrame:
         """
         Create a new DataFrame by selecting a subset of columns by name.
         """
-        pass
 
     @abstractmethod
     def get_chunks(self, n_chunks: int | None = None) -> Iterable[DataFrame]:
@@ -483,4 +459,3 @@ def get_chunks(self, n_chunks: int | None = None) -> Iterable[DataFrame]:
         ``self.num_chunks()``, meaning the producer must subdivide each chunk
         before yielding it.
         """
-        pass

pandas/io/excel/_base.py

@@ -1142,7 +1142,6 @@ def engine(self) -> str:
     @abc.abstractmethod
     def sheets(self) -> dict[str, Any]:
         """Mapping of sheet names to sheet objects."""
-        pass
 
     @property
     @abc.abstractmethod
@@ -1152,15 +1151,13 @@ def book(self):
 
         This attribute can be used to access engine-specific features.
         """
-        pass
 
     @book.setter
     @abc.abstractmethod
     def book(self, other) -> None:
         """
         Set book instance. Class type will depend on the engine used.
         """
-        pass
 
     def write_cells(
         self,
@@ -1212,7 +1209,6 @@ def _write_cells(
         freeze_panes: int tuple of length 2
             contains the bottom-most row and right-most column to freeze
         """
-        pass
 
     def save(self) -> None:
         """
@@ -1228,7 +1224,6 @@ def _save(self) -> None:
         """
         Save workbook to disk.
         """
-        pass
 
     def __init__(
         self,

pandas/io/formats/info.py

@@ -52,13 +52,6 @@
         shows the counts, and False never shows the counts."""
 )
 
-null_counts_sub = dedent(
-    """
-    null_counts : bool, optional
-        .. deprecated:: 1.2.0
-            Use show_counts instead."""
-)
-
 
 frame_examples_sub = dedent(
     """\
@@ -159,7 +152,6 @@
     "type_sub": " and columns",
     "max_cols_sub": frame_max_cols_sub,
     "show_counts_sub": show_counts_sub,
-    "null_counts_sub": null_counts_sub,
     "examples_sub": frame_examples_sub,
     "see_also_sub": frame_see_also_sub,
     "version_added_sub": "",
@@ -240,7 +232,6 @@
     "type_sub": "",
     "max_cols_sub": "",
     "show_counts_sub": show_counts_sub,
-    "null_counts_sub": "",
     "examples_sub": series_examples_sub,
     "see_also_sub": series_see_also_sub,
     "version_added_sub": "\n.. versionadded:: 1.4.0\n",
@@ -280,7 +271,7 @@
         at the cost of computational resources. See the
         :ref:`Frequently Asked Questions <df-memory-usage>` for more
         details.
-    {show_counts_sub}{null_counts_sub}
+    {show_counts_sub}
 
     Returns
     -------

pandas/io/json/_json.py

@@ -248,7 +248,6 @@ def write(self) -> str:
     @abstractmethod
     def obj_to_write(self) -> NDFrame | Mapping[IndexLabel, Any]:
         """Object to write in JSON format."""
-        pass
 
 
 class SeriesWriter(Writer):

pandas/io/pytables.py

@@ -2593,8 +2593,6 @@ def get_atom_timedelta64(cls, shape):
 class GenericDataIndexableCol(DataIndexableCol):
     """represent a generic pytables data column"""
 
-    pass
-
 
 class Fixed:
     """
@@ -2701,11 +2699,9 @@ def attrs(self):
 
     def set_attrs(self) -> None:
         """set our object attributes"""
-        pass
 
     def get_attrs(self) -> None:
         """get our object attributes"""
-        pass
 
     @property
     def storable(self):
@@ -2728,7 +2724,6 @@ def validate(self, other) -> Literal[True] | None:
 
     def validate_version(self, where=None) -> None:
         """are we trying to operate on an old version?"""
-        pass
 
     def infer_axes(self) -> bool:
         """

pandas/io/stata.py

@@ -2417,7 +2417,6 @@ def _replace_nans(self, data: DataFrame) -> DataFrame:
 
     def _update_strl_names(self) -> None:
         """No-op, forward compatibility"""
-        pass
 
     def _validate_variable_name(self, name: str) -> str:
         """
@@ -2701,19 +2700,15 @@ def _close(self) -> None:
 
     def _write_map(self) -> None:
         """No-op, future compatibility"""
-        pass
 
     def _write_file_close_tag(self) -> None:
         """No-op, future compatibility"""
-        pass
 
     def _write_characteristics(self) -> None:
         """No-op, future compatibility"""
-        pass
 
     def _write_strls(self) -> None:
         """No-op, future compatibility"""
-        pass
 
     def _write_expansion_fields(self) -> None:
         """Write 5 zeros for expansion fields"""
@@ -3438,7 +3433,6 @@ def _write_strls(self) -> None:
 
     def _write_expansion_fields(self) -> None:
         """No-op in dta 117+"""
-        pass
 
     def _write_value_labels(self) -> None:
         self._update_map("value_labels")