Skip to content

Let colorbar plot rectangles if possible#6360

Merged
PaulWessel merged 8 commits intomasterfrom
colorbar-rect-default
Feb 23, 2022
Merged

Let colorbar plot rectangles if possible#6360
PaulWessel merged 8 commits intomasterfrom
colorbar-rect-default

Conversation

@PaulWessel
Copy link
Member

See #6350 for background. Because Apple Preview (and possibly others) will resample coarse PDF images, any colormap based on a discrete CPT will look very blurred and continuous. This PR lets -Np be the default if it is possible.
Closes #6350. Note: A handfull of PostScript plot originals will need updating due to hairline differences between the approximate images and the exact polygon boundaries.

Because Apple Preview (and possibly others) will resample coarse PDF images, any colormap based on a discrete CPT will look very blurred and continuous.  This PR lets -Np be the default if it is possible.
@PaulWessel PaulWessel added documentation Improve documentation enhancement Improving an existing feature labels Feb 21, 2022
@PaulWessel PaulWessel added this to the 6.4.0 milestone Feb 21, 2022
@PaulWessel PaulWessel requested a review from maxrjones February 21, 2022 03:24
@PaulWessel PaulWessel self-assigned this Feb 21, 2022
@maxrjones
Copy link
Member

@PaulWessel, should I push the test image updates to this branch?

@PaulWessel
Copy link
Member Author

yes, please. I looked at CONTRIBUTING and I probably missed it but I did not find a section on how to revise plots already in dvc?

@maxrjones
Copy link
Member

I looked at CONTRIBUTING and I probably missed it but I did not find a section on how to revise plots already in dvc?

I will take a look and revise as needed in a separate PR.

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

Labels

documentation Improve documentation enhancement Improving an existing feature

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add an informational message to grdimage and colorbar regarding Apple Preview

2 participants