Repository navigation
Conversation
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 32 out of 32 changed files in this pull request and generated no new comments.
Suppressed comments (2)
pygmt/src/plot.py:63
- The updated type annotation for
perspectivedropsstr, butperspectiveis still accepted as a raw GMT-pstring by the alias system (AliasSystem.add_commonusesAlias(value, sep="/", size={2, 3}), which treats strings as scalars). Droppingstrmakes the public typing inconsistent with runtime behavior and removes the only typed way to pass advanced-pmodifiers not covered byPerspectiveyet (e.g.,+w/+v).
panel: int | Sequence[int] | bool = False,
incols: int | str | Sequence[int | str] | None = None,
perspective: Perspective | float | Sequence[float] | bool = False,
transparency: float | Sequence[float] | bool | None = None,
pygmt/helpers/decorators.py:283
- The
perspectivedocstring no longer mentions that callers can still pass a raw string with the full GMT-psyntax (including+w/+vmodifiers). Since the alias system still accepts strings forp(seepygmt/alias.py:374-375), this makes the documentation misleading and removes the only documented path for features not yet covered byPerspective.
Accepts a :class:`pygmt.params.Perspective` object, a single value
*azimuth*, a sequence of two values (*azimuth*, *elevation*), a sequence of
three values (*azimuth*, *elevation*, *level*).
- *azimuth*: Azimuth angle of the viewpoint in degrees [Default is 180,
Co-authored-by: Yvonne Fröhlich <94163266+yvonnefroehlich@users.noreply.github.com>
I personally like (I) Rotate around the center of the plot, as we do it for the PyGMT logo: Line 44 in 6f49e84 (II) Rotate plotting symbols perspective_receiver = f"{angle_receiver}+w{x_receiver}/{y_receiver}"
|
That's a good use case. Please see my comments at #4176 (comment). |


Implement the Perspective class for the
-poption. See #4176 for related discussions.Notes:
levelis used rather thanzlevel, because strictly speaking it can set the level for x-, y-, or z-planes.+w/+vare implemented asrefpoint+cstypePreview: