Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Improve Scatter hist example #14059

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
Apr 27, 2019
Merged

Conversation

timhoffm
Copy link
Member

@timhoffm timhoffm commented Apr 26, 2019

PR Summary

  • move ticks to the inside. This gives a better visual impression for axes that are laid out close to each other.
  • Use tick_params() instead of setting a NullFormatter. This is the more high-level approach and IMHO more suited here.
  • some general cleanup.

link to doc build

@tacaswell tacaswell added this to the v3.2.0 milestone Apr 26, 2019
@timhoffm
Copy link
Member Author

@tacaswell I think example/docstring updates can safely go in to the next bugfix release. There's no need to hold them back till the next point release. Please let me know if you disagree.

@timhoffm timhoffm modified the milestones: v3.2.0, v3.1.1 Apr 27, 2019
Copy link
Member

@NelleV NelleV left a comment

Choose a reason for hiding this comment

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

Thanks!

@NelleV NelleV merged commit 118adfd into matplotlib:master Apr 27, 2019
meeseeksmachine pushed a commit to meeseeksmachine/matplotlib that referenced this pull request Apr 27, 2019
@timhoffm timhoffm deleted the example-scatter-hist branch April 27, 2019 20:30
NelleV added a commit that referenced this pull request Apr 27, 2019
…059-on-v3.1.x

Backport PR #14059 on branch v3.1.x (Improve Scatter hist example)
@QuLogic QuLogic modified the milestones: v3.1.1, v3.1.0 Apr 28, 2019
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.

4 participants