Skip to content
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

Reduce chance of PostScript clipping if exceeding canvas in modern mode #197

Merged
merged 1 commit into from
Dec 2, 2018

Conversation

PaulWessel
Copy link
Member

Modern mode allows the max 32767x32767 point canvase (~11.6 x 11.6 m) but we only set the initial origin at (5 inch, 5 inch). If users are not careful and use -X -Y with negative values they may be plotting outside the canvas and that is lost. We now move to (40,40) inches instead.

Fixes #195.

Modern mode allows the max 32767x32767 point canvase (~11.6 x  11.6 m) but we only set the initial origin at (5 inch, 5 inch).  If users are not careful and use -X -Y with negative values they may be plotting outside the canvas and that is lost.  We now move to (40,40) inches instead.
@PaulWessel PaulWessel requested review from seisman and a team December 2, 2018 03:37
Copy link
Member

@seisman seisman left a comment

Choose a reason for hiding this comment

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

Works as expected.

@PaulWessel PaulWessel merged commit 97b57b0 into master Dec 2, 2018
@PaulWessel PaulWessel deleted the westsouthclipping branch December 2, 2018 05:08
obaney pushed a commit to obaney/gmt that referenced this pull request Aug 18, 2021
Implements support for `xarray.DataArray` grids in `Figure.grdimage`.
Not much changed here but this required a lot of changes to the
GMT API itself.
Lacking a bit in documentation but I'm working on a refactor of the
library which will make these things easier.

Fixes GenericMappingTools#124
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants