Skip to content

Conversation

@phofl
Copy link
Member

@phofl phofl commented Feb 9, 2024

  • closes #xxxx (Replace xxxx with the GitHub issue number)
  • Tests added and passed if fixing a bug or adding a new feature
  • All code checks passed.
  • Added type annotations to new arguments/methods/functions.
  • Added an entry in the latest doc/source/whatsnew/vX.X.X.rst file if fixing a bug or adding a new feature.
@phofl phofl marked this pull request as draft February 9, 2024 23:55
@phofl phofl marked this pull request as ready for review February 10, 2024 23:04
@simonjayhawkins simonjayhawkins added the Deprecate Functionality to remove in pandas label Feb 13, 2024
Copy link
Member

@mroeschke mroeschke left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A reminder that we'll need to deprecate future.no_silent_downcasting. Could you open a follow up issue?

@mroeschke mroeschke added this to the 3.0 milestone Feb 13, 2024
@mroeschke mroeschke added Clean and removed Deprecate Functionality to remove in pandas labels Feb 13, 2024
@mroeschke mroeschke merged commit 590d6ed into pandas-dev:main Feb 13, 2024
@mroeschke
Copy link
Member

Thanks @phofl

@phofl phofl deleted the silent_downcasting branch February 13, 2024 17:39
@phofl
Copy link
Member Author

phofl commented Feb 13, 2024

- :func:`read_excel`, :func:`read_json`, :func:`read_html`, and :func:`read_xml` no longer accept raw string or byte representation of the data. That type of data must be wrapped in a :py:class:`StringIO` or :py:class:`BytesIO` (:issue:`53767`)
- All arguments except the first ``path``-like argument in IO writers are now keyword only (:issue:`54229`)
- Changed the default value of ``observed`` in :meth:`DataFrame.groupby` and :meth:`Series.groupby` to ``True`` (:issue:`51811`)
- Enforced silent-downcasting deprecation for :ref:`all relevant methods <whatsnew_220.silent_downcasting>` (:issue:`54710`)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks like the option still exists in config_init and it is set in 3 tests

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The option will need to be deprecated in a follow up, but it can be removed in those tests

pmhatre1 pushed a commit to pmhatre1/pandas-pmhatre1 that referenced this pull request May 7, 2024
* Enforce silent downcasting deprecation * Update * Update * Add whatsnew * Fixup docstrings * Update
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

4 participants