Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

CLN: remove pandas.core.categorical #25655

Merged
merged 1 commit into from
Mar 13, 2019

Conversation

simonjayhawkins
Copy link
Member

xref #19269

# TODO: Remove after 0.23.x

@codecov
Copy link

codecov bot commented Mar 11, 2019

Codecov Report

Merging #25655 into master will decrease coverage by 49.55%.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff             @@
##           master   #25655       +/-   ##
===========================================
- Coverage   91.29%   41.74%   -49.56%     
===========================================
  Files         173      172        -1     
  Lines       52961    52957        -4     
===========================================
- Hits        48350    22106    -26244     
- Misses       4611    30851    +26240
Flag Coverage Δ
#multiple ?
#single 41.74% <ø> (ø) ⬆️
Impacted Files Coverage Δ
pandas/io/formats/latex.py 0% <0%> (-100%) ⬇️
pandas/io/sas/sas_constants.py 0% <0%> (-100%) ⬇️
pandas/tseries/plotting.py 0% <0%> (-100%) ⬇️
pandas/tseries/converter.py 0% <0%> (-100%) ⬇️
pandas/io/formats/html.py 0% <0%> (-99.36%) ⬇️
pandas/core/groupby/categorical.py 0% <0%> (-95.46%) ⬇️
pandas/io/sas/sas7bdat.py 0% <0%> (-91.17%) ⬇️
pandas/io/sas/sas_xport.py 0% <0%> (-90.15%) ⬇️
pandas/core/tools/numeric.py 10.44% <0%> (-89.56%) ⬇️
pandas/io/json/normalize.py 8.16% <0%> (-88.78%) ⬇️
... and 129 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update de52d0b...924b60a. Read the comment docs.

@codecov
Copy link

codecov bot commented Mar 11, 2019

Codecov Report

Merging #25655 into master will decrease coverage by <.01%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master   #25655      +/-   ##
==========================================
- Coverage   91.29%   91.29%   -0.01%     
==========================================
  Files         173      172       -1     
  Lines       52961    52957       -4     
==========================================
- Hits        48350    48345       -5     
- Misses       4611     4612       +1
Flag Coverage Δ
#multiple 89.86% <ø> (-0.01%) ⬇️
#single 41.74% <ø> (ø) ⬆️
Impacted Files Coverage Δ
pandas/util/testing.py 88.98% <0%> (-0.1%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update de52d0b...924b60a. Read the comment docs.

@jbrockmendel
Copy link
Member

LGTM. Did you grep for all the “remove after ...” comments?

@simonjayhawkins
Copy link
Member Author

a quick grep yields:

pandas/compat/pickle_compat.py:    # TODO: When FrozenNDArray is removed, add
pandas/core/arrays/datetimelike.py:        # TODO(GH-20300): remove this
pandas/core/arrays/numpy_.py:# TODO(NumPy1.13): remove this
pandas/core/arrays/numpy_.py:        # TODO(_values_for_fillna): remove this
pandas/core/arrays/period.py:        # TODO: remove
pandas/core/arrays/sparse.py:        # TODO(SparseArray.__setitem__): remove special cases in
pandas/core/indexes/api.py:                    # TODO: remove once pd.concat sort default changes
pandas/core/indexes/base.py:        # TODO(EA): remove index types as they become extension arrays
pandas/core/indexes/period.py:        # TODO(DatetimeArray): remove
pandas/core/internals/blocks.py:            # TODO(SparseArray.__setitem__): remove this if condition
pandas/core/internals/blocks.py:    # TODO: Refactor when convert_objects is removed since there will be 1 path
pandas/core/internals/blocks.py:                    # TODO(ExtensionArray): remove is_extension_type
pandas/core/series.py:            # TODO: remove when the default Categorical.take behavior changes
pandas/core/sparse/series.py:        # TODO: remove?
pandas/io/formats/html.py:                # TODO: Refactor to remove code duplication with code
pandas/io/formats/html.py:            # TODO: Refactor to remove code duplication with code block
pandas/plotting/_core.py:    # TODO(mpl_converter): remove once converter is explicit
pandas/tests/extension/test_categorical.py:        # TODO remove this once Categorical.take is fixed
pandas/tests/extension/test_numpy.py:    @pytest.mark.skip(reason="TODO: remove?")
pandas/tests/frame/test_to_csv.py:            # TODO: remove renaming when GH 10875 is solved
pandas/tests/sparse/series/test_series.py:        # TODO: sp_zero is not used anywhere...remove?
pandas/tests/sparse/series/test_series.py:            # TODO: expected is not used anywhere...remove?
pandas/util/_validators.py:    # TODO(PY3): Change to keyword-only args and remove all this

nothing jumps out as being related.

@WillAyd WillAyd added the Clean label Mar 11, 2019
@jreback jreback added this to the 0.25.0 milestone Mar 13, 2019
@jreback jreback merged commit 486472a into pandas-dev:master Mar 13, 2019
@jreback
Copy link
Contributor

jreback commented Mar 13, 2019

thanks @simonjayhawkins

@simonjayhawkins simonjayhawkins deleted the core-categorical branch March 13, 2019 13:34
sighingnow added a commit to sighingnow/pandas that referenced this pull request Mar 14, 2019
* master: (22 commits)
  Fixturize tests/frame/test_operators.py (pandas-dev#25641)
  Update ValueError message in corr (pandas-dev#25729)
  DOC: fix some grammar and inconsistency issues in the User Guide (pandas-dev#25728)
  ENH: Add public start, stop, and step attributes to RangeIndex (pandas-dev#25720)
  Make Rolling.apply documentation clearer (pandas-dev#25712)
  pandas-dev#25707 - Fixed flakiness in stata write test (pandas-dev#25714)
  Json normalize nan support (pandas-dev#25619)
  TST: resolve issues with test_constructor_dtype_datetime64 (pandas-dev#24868)
  DEPR: Deprecate box kwarg for to_timedelta and to_datetime (pandas-dev#24486)
  BUG: Preserve name in DatetimeIndex.snap (pandas-dev#25585)
  Fix concat not respecting order of OrderedDict (pandas-dev#25224)
  CLN: remove pandas.core.categorical (pandas-dev#25655)
  TST/CLN: Remove more Panel tests (pandas-dev#25675)
  Pinned pycodestyle (pandas-dev#25701)
  DOC: update date of 0.24.2 release notes (pandas-dev#25699)
  BUG: Fix error in replace with strings that are large numbers (pandas-dev#25616) (pandas-dev#25644)
  BUG: fix usage of na_sentinel with sort=True in factorize() (pandas-dev#25592)
  BUG: Fix to_string output when using header (pandas-dev#16718) (pandas-dev#25602)
  CLN: Remove unused test code (pandas-dev#25670)
  CLN: remove Panel from concat error message (pandas-dev#25676)
  ...

# Conflicts:
#	doc/source/whatsnew/v0.25.0.rst
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants