Skip to content

Remove overspecified latex geometry. #17847

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
Jul 6, 2020
Merged

Conversation

anntzer
Copy link
Contributor

@anntzer anntzer commented Jul 6, 2020

The geometry package derives body from papersize and margin. Even
though the value given for body is consistent (papersize - 2*margin), giving all three of them yields a warning in the tex log
(which can be seen e.g. when processing an invalid tex string in usetex
mode (e.g. figtext(.5, .5, r"\foo")):

Package geometry Warning: Over-specification in `h'-direction.
    `width' (5058.9pt) is ignored.

Package geometry Warning: Over-specification in `v'-direction.
    `height' (5058.9pt) is ignored.

Just get rid of the overspecification.

PR Summary

PR Checklist

  • Has Pytest style unit tests
  • Code is Flake 8 compliant
  • New features are documented, with examples if plot related
  • Documentation is sphinx and numpydoc compliant
  • Added an entry to doc/users/next_whats_new/ if major new feature (follow instructions in README.rst there)
  • Documented in doc/api/next_api_changes/* if API changed in a backward-incompatible way

The geometry package derives `body` from `papersize` and `margin`.  Even
though the value given for `body` is consistent (`papersize -
2*margin`), giving all three of them yields a warning in the tex log
(which can be seen e.g. when processing an invalid tex string in usetex
mode (e.g. `figtext(.5, .5, r"\foo")`):
```
Package geometry Warning: Over-specification in `h'-direction.
    `width' (5058.9pt) is ignored.

Package geometry Warning: Over-specification in `v'-direction.
    `height' (5058.9pt) is ignored.
```
Just get rid of the overspecification.
@anntzer anntzer added this to the v3.4.0 milestone Jul 6, 2020
@dstansby dstansby merged commit 1664420 into matplotlib:master Jul 6, 2020
@anntzer anntzer deleted the geom branch July 6, 2020 12:38
QuLogic added a commit to QuLogic/matplotlib that referenced this pull request Jul 17, 2020
This is a corollary to matplotlib#17847; specifying paper size, body and margin
causes a warning from the geometry package.
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.

3 participants