Skip to content

Remove bar line #9122

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 1 commit into from
Jan 19, 2015
Merged

Conversation

TomAugspurger
Copy link
Contributor

Closes #9088
Closes #7128

Votes on whether we should

  1. Remove the line entirely
  2. Add a parameter to .plot control whether the line should be added.

I vote for 1, but I can go either way.

@jreback
Copy link
Contributor

jreback commented Dec 21, 2014

-1 in adding a parameter - this just looks like a bug
and could be done if the user really wants via matplotlib (the default is what u r doing as it seems more common)

@shoyer
Copy link
Member

shoyer commented Dec 21, 2014

Agreed, let's kill it. It's easy enough to add in manually afterwards.

@jreback jreback added this to the 0.16.0 milestone Dec 22, 2014
@@ -29,6 +29,9 @@ Backwards incompatible API changes

- ``Index.duplicated`` now returns `np.array(dtype=bool)` rather than `Index(dtype=object)` containing `bool` values. (:issue:`8875`)

- Bar and horizontal bar plots no longer add a dashed line along the info axis. The prior style can be achieved with matplotlib's `axhline` or `axvline`
methods (:issue:`9088`).
Copy link
Contributor

Choose a reason for hiding this comment

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

no spaces needed as line breaks

@jreback
Copy link
Contributor

jreback commented Jan 2, 2015

minor doc issue. looks good.

@jreback
Copy link
Contributor

jreback commented Jan 18, 2015

@TomAugspurger I think you can merge this

TomAugspurger pushed a commit that referenced this pull request Jan 19, 2015
@TomAugspurger TomAugspurger merged commit 93141eb into pandas-dev:master Jan 19, 2015
@TomAugspurger TomAugspurger deleted the remove-bar-line branch April 5, 2017 02:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants