DEPR: add shims for util + TST: test that we work in downstream packages
State: closed


Labels:
name: Testing
description: pandas testing functions or related to the test suite
name: Deprecate
description: Functionality to remove in pandas

Comments:
should update to have at least an example for each one.

@jorisvandenbossche @TomAugspurger 
# [Codecov](https://codecov.io/gh/pandas-dev/pandas/pull/16250?src=pr&el=h1) Report
> Merging [#16250](https://codecov.io/gh/pandas-dev/pandas/pull/16250?src=pr&el=desc) into [master](https://codecov.io/gh/pandas-dev/pandas/commit/a31c96d34d00dc757908b564dc93991e867d83e2?src=pr&el=desc) will **increase** coverage by `<.01%`.
> The diff coverage is `n/a`.

[![Impacted file tree graph](https://codecov.io/gh/pandas-dev/pandas/pull/16250/graphs/tree.svg?src=pr&height=150&width=650&token=eZ4WkYLtcO)](https://codecov.io/gh/pandas-dev/pandas/pull/16250?src=pr&el=tree)

```diff
@@            Coverage Diff             @@
##           master   #16250      +/-   ##
==========================================
+ Coverage   90.33%   90.33%   +<.01%     
==========================================
  Files         164      164              
  Lines       50890    50890              
==========================================
+ Hits        45972    45973       +1     
+ Misses       4918     4917       -1
```

| Flag | Coverage Δ | |
|---|---|---|
| #multiple | `88.12% <ø> (+0.01%)` | :arrow_up: |
| #single | `40.3% <ø> (-0.1%)` | :arrow_down: |

| [Impacted Files](https://codecov.io/gh/pandas-dev/pandas/pull/16250?src=pr&el=tree) | Coverage Δ | |
|---|---|---|
| [pandas/io/gbq.py](https://codecov.io/gh/pandas-dev/pandas/pull/16250?src=pr&el=tree#diff-cGFuZGFzL2lvL2dicS5weQ==) | `25% <0%> (-58.34%)` | :arrow_down: |
| [pandas/core/frame.py](https://codecov.io/gh/pandas-dev/pandas/pull/16250?src=pr&el=tree#diff-cGFuZGFzL2NvcmUvZnJhbWUucHk=) | `97.58% <0%> (-0.1%)` | :arrow_down: |
| [pandas/core/generic.py](https://codecov.io/gh/pandas-dev/pandas/pull/16250?src=pr&el=tree#diff-cGFuZGFzL2NvcmUvZ2VuZXJpYy5weQ==) | `91.94% <0%> (+0.21%)` | :arrow_up: |
| [pandas/core/common.py](https://codecov.io/gh/pandas-dev/pandas/pull/16250?src=pr&el=tree#diff-cGFuZGFzL2NvcmUvY29tbW9uLnB5) | `91.03% <0%> (+0.34%)` | :arrow_up: |
| [pandas/\_version.py](https://codecov.io/gh/pandas-dev/pandas/pull/16250?src=pr&el=tree#diff-cGFuZGFzL192ZXJzaW9uLnB5) | `44.65% <0%> (+1.9%)` | :arrow_up: |

------

[Continue to review full report at Codecov](https://codecov.io/gh/pandas-dev/pandas/pull/16250?src=pr&el=continue).
> **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta)
> `Δ = absolute <relative> (impact)`, `ø = not affected`, `? = missing data`
> Powered by [Codecov](https://codecov.io/gh/pandas-dev/pandas/pull/16250?src=pr&el=footer). Last update [a31c96d...de72be4](https://codecov.io/gh/pandas-dev/pandas/pull/16250?src=pr&el=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).

# [Codecov](https://codecov.io/gh/pandas-dev/pandas/pull/16250?src=pr&el=h1) Report
> Merging [#16250](https://codecov.io/gh/pandas-dev/pandas/pull/16250?src=pr&el=desc) into [master](https://codecov.io/gh/pandas-dev/pandas/commit/a31c96d34d00dc757908b564dc93991e867d83e2?src=pr&el=desc) will **decrease** coverage by `0.03%`.
> The diff coverage is `0%`.

[![Impacted file tree graph](https://codecov.io/gh/pandas-dev/pandas/pull/16250/graphs/tree.svg?token=eZ4WkYLtcO&width=650&height=150&src=pr)](https://codecov.io/gh/pandas-dev/pandas/pull/16250?src=pr&el=tree)

```diff
@@            Coverage Diff             @@
##           master   #16250      +/-   ##
==========================================
- Coverage   90.33%    90.3%   -0.04%     
==========================================
  Files         164      167       +3     
  Lines       50890    50907      +17     
==========================================
  Hits        45972    45972              
- Misses       4918     4935      +17
```

| Flag | Coverage Δ | |
|---|---|---|
| #multiple | `88.09% <0%> (-0.02%)` | :arrow_down: |
| #single | `40.29% <0%> (-0.11%)` | :arrow_down: |

| [Impacted Files](https://codecov.io/gh/pandas-dev/pandas/pull/16250?src=pr&el=tree) | Coverage Δ | |
|---|---|---|
| [pandas/util/hashing.py](https://codecov.io/gh/pandas-dev/pandas/pull/16250?src=pr&el=tree#diff-cGFuZGFzL3V0aWwvaGFzaGluZy5weQ==) | `0% <0%> (ø)` | |
| [pandas/util/decorators.py](https://codecov.io/gh/pandas-dev/pandas/pull/16250?src=pr&el=tree#diff-cGFuZGFzL3V0aWwvZGVjb3JhdG9ycy5weQ==) | `0% <0%> (ø)` | |
| [pandas/types/common.py](https://codecov.io/gh/pandas-dev/pandas/pull/16250?src=pr&el=tree#diff-cGFuZGFzL3R5cGVzL2NvbW1vbi5weQ==) | `0% <0%> (ø)` | |
| [pandas/io/gbq.py](https://codecov.io/gh/pandas-dev/pandas/pull/16250?src=pr&el=tree#diff-cGFuZGFzL2lvL2dicS5weQ==) | `25% <0%> (-58.34%)` | :arrow_down: |
| [pandas/core/frame.py](https://codecov.io/gh/pandas-dev/pandas/pull/16250?src=pr&el=tree#diff-cGFuZGFzL2NvcmUvZnJhbWUucHk=) | `97.58% <0%> (-0.1%)` | :arrow_down: |
| [pandas/core/generic.py](https://codecov.io/gh/pandas-dev/pandas/pull/16250?src=pr&el=tree#diff-cGFuZGFzL2NvcmUvZ2VuZXJpYy5weQ==) | `91.94% <0%> (+0.21%)` | :arrow_up: |
| [pandas/\_version.py](https://codecov.io/gh/pandas-dev/pandas/pull/16250?src=pr&el=tree#diff-cGFuZGFzL192ZXJzaW9uLnB5) | `44.65% <0%> (+1.9%)` | :arrow_up: |

------

[Continue to review full report at Codecov](https://codecov.io/gh/pandas-dev/pandas/pull/16250?src=pr&el=continue).
> **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta)
> `Δ = absolute <relative> (impact)`, `ø = not affected`, `? = missing data`
> Powered by [Codecov](https://codecov.io/gh/pandas-dev/pandas/pull/16250?src=pr&el=footer). Last update [a31c96d...085375f](https://codecov.io/gh/pandas-dev/pandas/pull/16250?src=pr&el=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).

@jreback I'm taking care of the release note for this in my doc updates (will have a PR shortly).
thanks @TomAugspurger yeah will merge when this is fixed.
@jreback https://travis-ci.org/pandas-dev/pandas/jobs/229097977 failed. Does that error message make sense to you?
@TomAugspurger still working on things.
turns out our build-test build was not actually testing the correct installation :<
lon it built correctly but still some issue
will investigate 
This all passed. Good to merge then?
The build one failed https://travis-ci.org/pandas-dev/pandas/jobs/229133755
not yet
Ah, yes, I didn't look at the allowed failures...
But, it is only the setting up of the test build env, not the actual code changes? Those could already be merged?
Ah, yes I think so. Although, those tests will be skipped if the deps aren't around. Do we we have dask and geopandas anywhere else in the matrix?
we can test those two locally
I checked out this PR, and can confirm it fixed the geopandas import for me
I think we're good then. I'll ensure they work after tagging but before
uploading.

On Fri, May 5, 2017 at 11:13 AM, Joris Van den Bossche <
notifications@github.com> wrote:

> I checked out this PR, and can confirm it fixed the geopandas import for me
>
> —
> You are receiving this because you were mentioned.
> Reply to this email directly, view it on GitHub
> <https://github.com/pandas-dev/pandas/pull/16250#issuecomment-299507947>,
> or mute the thread
> <https://github.com/notifications/unsubscribe-auth/ABQHIiIeA89ll0o3pVss0hh10UU14noHks5r20qWgaJpZM4NRuiT>
> .
>

@jreback OK to merge and fix the test later? 
(I can also split off just the code changes to merge only that)
yes that is fine