/matplotlib

More precise choice of axes limits. #6192

Merged
merged 1 commit into from Mar 20, 2016

Conversation

Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
  v2.0.0
2 participants
@tacaswell @anntzer
@tacaswell
Owner

tacaswell commented Mar 20, 2016

plt.plot([-.1, .2]) used to pick (in round numbers mode) [-.1, .25] as
ylims due to floating point inaccuracies; change it to pick [-.1, .2](up to floating point inaccuracies).

Support for the (unused and deprecated-in-comment) "trim" keyword
argument has been dropped as the way ticks are picked has changed.

Many test images have changed!

Implementation notes:

  • A bug in numpy's implementation of divmod (numpy/numpy#6127) is worked
    around.
  • The implementation of scale_range has also been cleaned.

See #5767, #5738.

Replaces #5768
@anntzer @jenshnielsen anntzer More precise choice of axes limits. 
plt.plot([-.1, .2]) used to pick (in round numbers mode) [-.1, .25] as
ylims due to floating point inaccuracies; change it to pick [-.1, .2]
(up to floating point inaccuracies).

Support for the (unused and deprecated-in-comment) "trim" keyword
argument has been dropped as the way ticks are picked has changed.

Many test images have changed!

Implementation notes:
- A bug in numpy's implementation of divmod (numpy/numpy#6127) is worked
around.
- The implementation of scale_range has also been cleaned.

See #5767, #5738.

All checks have passed

2 successful checks
continuous-integration/appveyor/pr — AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr — The Travis CI build passed
Details
6d23c8e

tacaswell added the needs_review label Mar 20, 2016

tacaswell added this to the 2.0 (style change major release) milestone Mar 20, 2016

@tacaswell tacaswell added a commit that referenced this pull request Mar 20, 2016

@tacaswell tacaswell Merge pull request #6192 from jenshnielsen/tighter-axes-limits 
More precise choice of axes limits.
 996f3ec

@tacaswell tacaswell merged commit 996f3ec into matplotlib:master Mar 20, 2016
2 checks passed

2 checks passed

continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details

tacaswell removed the needs_review label Mar 20, 2016

@tacaswell tacaswell added a commit that referenced this pull request Mar 20, 2016

@tacaswell tacaswell Merge pull request #6192 from jenshnielsen/tighter-axes-limits 
More precise choice of axes limits.
 cd7a02e
@tacaswell
Owner

tacaswell commented Mar 20, 2016

Backported to v2.x as cd7a02e

This seems too disruptive to push back to 1.5.x, but could be convinced other wise.

jenshnielsen deleted the jenshnielsen:tighter-axes-limits branch Mar 21, 2016

This was referenced Apr 2, 2016

Open
Closed
Merged
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment