Skip to content

Explain the coordinate systems in the user guide - #294

Merged
leouieda merged 3 commits into
fatiando:mainfrom
manduinca:docs/coordinate-systems
Oct 2, 2026
Merged

leouieda merged 3 commits into
fatiando:mainfrom
manduinca:docs/coordinate-systems

Conversation

@manduinca

Copy link
Copy Markdown
Contributor

Fixes #257

Adds a section at the top of the coordinate conversions page describing the four systems Boule converts between (geocentric Cartesian, geocentric spherical, geodetic and ellipsoidal harmonic) and what each coordinate means geometrically, before the examples.

The figure is generated in a jupyter-execute block with matplotlib, so there is no image file to keep in sync and no copyright to worry about. Every distance and angle in it is computed with Boule itself on an Ellipsoid with a flattening of 0.3; the text says so, since with a realistic flattening the systems are indistinguishable by eye. Left panel: Cartesian axes, r and φ′, h and φ. Right panel: the confocal ellipsoid through P with u, its foci, and β on the circumscribing circle.

Built the page locally with Sphinx 7.2 and jupyter-sphinx 0.5; the new section renders without warnings.

@leouieda

Copy link
Copy Markdown
Member

Hi @manduinca thank you for the submission! Could you let us know if AI was used to make this PR and if so, to what extent?

@leouieda

Copy link
Copy Markdown
Member

I'm also merging in the main branch to this PR so remember to git pull before doing further work on your branch.

It's better to compare them in a single figure and the symbols didn't
match the docstrings of the conversion methods.
@leouieda

Copy link
Copy Markdown
Member

I made some tweaks to the figure by combining it into a single one and making the maths consistent with the method docstrings. This is what the figure looks like:

image

@manduinca

Copy link
Copy Markdown
Contributor Author

@leouieda yes, and to a large extent. I used Claude Code (an AI coding agent) for most of this PR: it wrote the text of the coordinate systems section and the code that draws the figure, following my direction, and the docs build was checked locally before I opened it.

The AI policy came out after I submitted this, and under it this PR counts as mostly AI-made. If you'd rather close it, that's fine with me; your figure changes are worth keeping either way.

This reply was also drafted with AI help.

@leouieda

leouieda commented Oct 2, 2026

Copy link
Copy Markdown
Member

Hi @manduinca thanks for the disclosure. We can accept this one since we've already put in the time for reviewing and it was from before the new policy. Thank you! Going forward, we would prefer mainly human-authored PRs.

@leouieda
leouieda merged commit b610249 into fatiando:main Oct 2, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add a figure and section in docs explaining the coordinate systems

2 participants