Skip to content

Conversation

@jdoepfert
Copy link
Contributor

@jdoepfert jdoepfert commented Nov 10, 2017

pd.qcut only accepts 1-dimensional numpy arrays, which is not mentioned in the docstring. This PR updates the docstring accordingly.

@codecov
Copy link

codecov bot commented Nov 10, 2017

Codecov Report

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

Impacted file tree graph

@@ Coverage Diff @@ ## master #18211 +/- ## ========================================== + Coverage 91.4% 91.4% +<.01%  ========================================== Files 163 163 Lines 50064 50064 ========================================== + Hits 45760 45763 +3  + Misses 4304 4301 -3
Flag Coverage Δ
#multiple 89.21% <ø> (+0.02%) ⬆️
#single 40.36% <ø> (-0.06%) ⬇️
Impacted Files Coverage Δ
pandas/core/reshape/tile.py 90.25% <ø> (ø) ⬆️
pandas/io/gbq.py 25% <0%> (-58.34%) ⬇️
pandas/core/frame.py 97.8% <0%> (-0.1%) ⬇️
pandas/plotting/_converter.py 65.2% <0%> (+1.81%) ⬆️

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 324ac85...890e010. Read the comment docs.

~~~~~~~~~~~~~~~~~~~~~

-
- Add requirement for 1-dimensional ndarrays in ``pd.qcut`` docstring (:issue:`18173`)
Copy link
Contributor

Choose a reason for hiding this comment

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

remove this

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done

Parameters
----------
x : ndarray or Series
x : ndarray (1-dimensional) or Series
Copy link
Contributor

Choose a reason for hiding this comment

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

1d ndarray or Series

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done

@jreback jreback added the Docs label Nov 10, 2017
@jreback jreback added this to the 0.21.1 milestone Nov 10, 2017
@jreback jreback merged commit a6345c7 into pandas-dev:master Nov 10, 2017
@jreback
Copy link
Contributor

jreback commented Nov 10, 2017

thanks @jdoepfert

@jdoepfert
Copy link
Contributor Author

thx @jreback !

TomAugspurger pushed a commit to TomAugspurger/pandas that referenced this pull request Dec 8, 2017
TomAugspurger pushed a commit that referenced this pull request Dec 11, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3 participants