Figure.text: Allow passing a sequence of offset to the 'offset' parameter#4652
Figure.text: Allow passing a sequence of offset to the 'offset' parameter#4652seisman wants to merge 3 commits into
Conversation
seisman
added
enhancement
seisman
added
final review call
| y=[latA, latB], | ||
| text=["A", "B"], | ||
| offset="0c/0.3c", # Move text 0.2 centimeters up (y-direction) | ||
| offset=(0, 0.3), # Move text 0.3 centimeters up (y-direction) |
There was a problem hiding this comment.
Should we keep the "c" unit specifier? The gmt.conf docs say:
It is recommended that users specify the units for distances and lengths by appending c (cm), i (inch), or p (points) when changing parameters using any of these methods. By default, when no unit is specified the value will be assumed to be cm for parameters not related to fonts or pen thicknesses and will be assumed to be points for parameters related to fonts or pen thicknesses.
So recommended is to append the unit, but no unit means centimeter in some but not all cases (I guess offset counts as non-font/pen thickness?). Maybe keep it as ("0c", "0.3c") to be clearer?
There was a problem hiding this comment.
Yes, appending the unit specifier is recommended in the GMT CLI. However, in PyGMT, I feel it is more natural to use plain numbers for quantities such as offsets, widths, and sizes. They are much shorter to type and can also be used directly in calculations.
2 participants
Passing a sequence of offsets to
Figure.textis frequently used but not supported yet.This PR supports it and also migrates
offsetto the new alias system.