Skip to content

Conversation

@max-sixty
Copy link
Contributor

closes #13108

Copy link
Contributor

Choose a reason for hiding this comment

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

1 will

shows a warning (not raises)

@max-sixty
Copy link
Contributor Author

@jreback updated

@jreback jreback added this to the 0.18.2 milestone May 7, 2016
@jreback jreback closed this in af7bdd3 May 8, 2016
@jreback
Copy link
Contributor

jreback commented May 8, 2016

change around a little bit. thanks for the PR!

Traceback (most recent call last)
...
KeyError: Key length (3) was greater than MultiIndex lexsort depth (2)

Copy link
Member

Choose a reason for hiding this comment

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

Is there a reason this is removed? I think it is still a nice example of what happens if the index is not sorted? (this error still occurs on master)

Copy link
Contributor

Choose a reason for hiding this comment

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

I added it back (in the context of the slightly changed example). lmk if more clear.
cc @MaximilianR

Copy link
Contributor

Choose a reason for hiding this comment

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

jreback added a commit that referenced this pull request May 10, 2016
@max-sixty max-sixty deleted the multiindex-sort branch December 22, 2016 05:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

DOC: MultiIndex sort docs

3 participants