-
-
Notifications
You must be signed in to change notification settings - Fork 18.4k
Question regarding Series.argsort documentation #24816
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
Labels
Milestone
Comments
I believe axis and order are needed to match the signature of `np.argsort`,
which dispatches to objects with a `.argsort` method,
…On Thu, Jan 17, 2019 at 7:19 AM Daniel Saxton ***@***.***> wrote:
Series.argsort has three arguments, axis, kind and order, but only kind
is actually used. Should the other two be removed from the documentation?
(I'm guessing they were once used and are left in the code for
compatibility reasons?)
https://pandas.pydata.org/pandas-docs/stable/generated/pandas.Series.argsort.html
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
<#24816>, or mute the thread
<https://github.com/notifications/unsubscribe-auth/ABQHIsD8fJTSlj6d2y9CPEQ9qgJ50heAks5vEHh_gaJpZM4aFLm5>
.
|
Could probably update the documentation similar to |
Happy to look at updating the documentation to reflect his. |
4 tasks
jschendel
pushed a commit
that referenced
this issue
Jan 19, 2019
Pingviinituutti
pushed a commit
to Pingviinituutti/pandas
that referenced
this issue
Feb 28, 2019
Pingviinituutti
pushed a commit
to Pingviinituutti/pandas
that referenced
this issue
Feb 28, 2019
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Series.argsort
has three arguments,axis
,kind
andorder
, but onlykind
is actually used. Should the other two be removed from the documentation? (I'm guessing they were once used and are left in the code for compatibility reasons?)https://pandas.pydata.org/pandas-docs/stable/generated/pandas.Series.argsort.html
The text was updated successfully, but these errors were encountered: