Skip to content

Spherical signal derivatives tutorial - #402

Merged
matt-graham merged 23 commits into
astro-informatics:mainfrom
i-tanvir:Spherical-signal-derivatives-tutorial
Oct 29, 2026
Merged

matt-graham merged 23 commits into
astro-informatics:mainfrom
i-tanvir:Spherical-signal-derivatives-tutorial

Conversation

@i-tanvir

Copy link
Copy Markdown
Contributor

Retargetting of i-tanvir#2

@matt-graham matt-graham self-assigned this Sep 11, 2026
@matt-graham
matt-graham marked this pull request as ready for review October 19, 2026 11:59

@claude claude Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Claude Code Review

This pull request is from a fork — automated review is disabled. A repository maintainer can comment @claude review to run a one-time review.

Comment on lines +120 to +131
fig, ax = plt.subplots(
figsize=(6, 3),
subplot_kw={"projection": ccrs.Mollweide()},
)

ax.pcolormesh(
longitude_grid,
latitude_grid,
signal,
transform=ccrs.PlateCarree(),
cmap="viridis",
)

@matt-graham matt-graham Oct 19, 2026 •

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

From a quick check of the rendered tutorial after the Markdown to reStructuredText conversion, I noticed that the visualised spherical signals as in this code snippet have an artifact / discontinuity on the meridian / line of constant longitude running around the right section of the boundary of this projection

Image

Tanvir and I discussed this previously and I think this is due to the wrapping of the data around the longitudinal direction not being correctly handled by cartopy / matplotlib by default and can be fixed as suggested there using add_cyclic helper.

@mscroggs tagging you in case you can look in to this as part of review, thanks!

@matt-graham matt-graham added the documentation Improvements or additions to documentation label Oct 19, 2026

@mscroggs mscroggs left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Made some minor comments. Otherwise is a good tutorial

Co-authored-by: Matthew Scroggs <matthew.w.scroggs@gmail.com>
matt-graham and others added 2 commits October 23, 2026 17:32
@matt-graham
matt-graham merged commit 2130c41 into astro-informatics:main Oct 29, 2026
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Tutorial example on computing spatial derivatives of a spherical signal

3 participants