-
Notifications
You must be signed in to change notification settings - Fork 339
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
MAP_FRAME_PEN auto-scaling is not ideal in modern mode #6786
Comments
Yes, it is not ideal, but I guess the auto-scaling was set up to get reasonable annotation font sizes for reasonable sizes. 1 cm maps are not reasonable and ends up with too large annotations and pen thicknesses. Given our font size scaling
and overall scale for thinkgs like pen widths:
the 1 cm map only yields a scale reduction of 0.78. |
I agree that 1-cm-wide maps are not reasonable. I feel that the pen thickness is still a little too thick for the colorbar for the 5-cm-wide maps. Compared to the 10-cm-wide maps, the height of the colorbar has a scale reduction of 0.5, but the colorbar frame pen thickness has a scale reduction of 0.83. I think that's why I have to change the frame pen thickness for most colorbars in my own plots. |
I agree we need to do something about the colorbar pen. |
Because the thickness of the bar is usually very small compared to its length, it is not helpful to use the same pen width as for the map. Even if we reduced it by the fractional length to map with (e.g. 0.8) it is still to think. Perhaps it is reasonable to simply use a factor of 0.5 * framepen->width for colorbars where framepen is the width scaled for the size of the map? |
BTW, do you get this now after the cmocean merge?
|
Never mind, went away after rebuild. |
Description of the problem
The following script plots three basemaps with different sizes (10c, 5c and 1c).
Here is the output image. As you can see, the frame pen thickness is almost identical (with very small thickness changes). The default pen thickness looks good for a 15-cm-wide basemap, but is too thick for a 5-cm-wide or 1-cm-wide basemap.
System information
gmt --version
):6.4.0_a7493e9_2022.05.26
The text was updated successfully, but these errors were encountered: