Skip to content

Add requirement for a 1-dimensional ndarray in the pd.qcut docstring #18211

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

Merged
merged 4 commits into from
Nov 10, 2017

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.

@@ -78,7 +78,7 @@ Performance Improvements
Documentation Changes
~~~~~~~~~~~~~~~~~~~~~

-
- 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

@@ -148,7 +148,7 @@ def qcut(x, q, labels=None, retbins=False, precision=3, duplicates='raise'):

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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

BUG: pd.qcut doesn't seem to support ndarray
3 participants