Skip to content

plot forces ordering on Datetimeindex #3642

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

Closed
ghost opened this issue May 18, 2013 · 13 comments
Closed

plot forces ordering on Datetimeindex #3642

ghost opened this issue May 18, 2013 · 13 comments
Labels
Regression Functionality that used to work in a prior pandas version Visualization plotting
Milestone

Comments

@ghost
Copy link

ghost commented May 18, 2013

introduced by #3619, opening this to track the issue for 0.11.1.

While the user should be able to get sorted ticks when plotting, it should not
be forced on him. so either opt-in or opt-out but not "one size fits all".

@jreback
Copy link
Contributor

jreback commented May 20, 2013

@y-p whats the deal on this?

@ghost
Copy link
Author

ghost commented May 20, 2013

huh?

@jreback
Copy link
Contributor

jreback commented May 20, 2013

just wanted to see if should bump to 0.12

@jreback
Copy link
Contributor

jreback commented May 20, 2013

why don't we just add a keyword, maybe sort_index=True which is the current behavior
False is prior, and if want to get really fancy allow a sorter function (via the same kw)?

@jreback
Copy link
Contributor

jreback commented May 20, 2013

I don't have an easy way of testing these kind of things (and that is why I don't really get involved in plotting)....I am just a user here :)

for some reason I can't import the dev version (the one I am working on) into my ipython notebook.....how do you do it?

@ghost
Copy link
Author

ghost commented May 20, 2013

I think you want python setup.py develop.

@jreback
Copy link
Contributor

jreback commented May 20, 2013

yep...thankks

@wesm
Copy link
Member

wesm commented Jun 2, 2013

Shelving for the release candidate, maybe push to 0.12.

@ghost
Copy link
Author

ghost commented Jun 3, 2013

This is about a change introduced for 0.11.1, so moving back the decision
to 0.12 seems like a bad move.

Seems it's either:

  1. revert BUG: Reindex data if plotting time/period index (GH3601) #3619 (and replace by an option somewhere)
  2. close this.

I seem to be in the minority in thinking this was the wrong solution, so 2 is fine by me.

@cpcloud
Copy link
Member

cpcloud commented Jun 16, 2013

@y-p @wesm any decision(s) on this?

@ghost
Copy link
Author

ghost commented Jun 16, 2013

@dieterv77, I'd like to revert #3619 and convert into a new issue to make this
an option in 0.12. I'm +1 on making this behavior the default, but the user should
be allowed to opt out.

Does that sound reasonable?

@wesm
Copy link
Member

wesm commented Jun 19, 2013

are there cases where the old behavior is what you want?

@ghost
Copy link
Author

ghost commented Jun 19, 2013

not really. I can imagine that in some RTL locales, ascending vs descending order of sorting
would be desired. Up till now that was in the hands of the user.

enough. I'm not championing this anymore.

@ghost ghost closed this as completed Jun 19, 2013
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Regression Functionality that used to work in a prior pandas version Visualization plotting
Projects
None yet
Development

No branches or pull requests

3 participants