Skip to content
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

Add the PyGMT ecosystem page #3475

Open
wants to merge 30 commits into
base: main
Choose a base branch
from
Open

Add the PyGMT ecosystem page #3475

wants to merge 30 commits into from

Conversation

seisman
Copy link
Member

@seisman seisman commented Oct 1, 2024

@seisman seisman added documentation Improvements or additions to documentation help wanted Helping hands are appreciated labels Oct 1, 2024
@seisman seisman changed the title DOC: Add the PyGMT ecosystem page POC + WIP + DOC: Add the PyGMT ecosystem page Oct 1, 2024
@seisman seisman marked this pull request as ready for review October 7, 2024 14:48
@seisman seisman changed the title POC + WIP + DOC: Add the PyGMT ecosystem page Add the PyGMT ecosystem page Oct 7, 2024
@seisman seisman added needs review This PR has higher priority and needs review. and removed help wanted Helping hands are appreciated labels Oct 7, 2024
@seisman seisman added this to the 0.14.0 milestone Oct 7, 2024
doc/ecosystem.md Outdated Show resolved Hide resolved
doc/ecosystem.md Outdated Show resolved Hide resolved
doc/ecosystem.md Outdated Show resolved Hide resolved
@michaelgrund
Copy link
Member

Do you plan to also add the packages that rely on PyGMT as mentioned in the bottom of the page in this PR?

@seisman
Copy link
Member Author

seisman commented Oct 9, 2024

Do you plan to also add the packages that rely on PyGMT as mentioned in the bottom of the page in this PR?

I feel we shouldn't add any packages without asking for permission from the package maintainers.

Similar to what geopandas is doing (https://geopandas.org/en/stable/community/ecosystem.html), we may add a note like:

If your package relies on PyGMT, please let us know or add it by yourself.

doc/ecosystem.md Show resolved Hide resolved
doc/ecosystem.md Outdated Show resolved Hide resolved
doc/ecosystem.md Outdated Show resolved Hide resolved
doc/ecosystem.md Outdated Show resolved Hide resolved
doc/ecosystem.md Outdated Show resolved Hide resolved

#### NumPy

[NumPy][] is the fundamental package for scientific computing in Python. It is a Python
Copy link
Member Author

Choose a reason for hiding this comment

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


#### pandas

[pandas][] is a Python package providing fast, flexible, and expressive data structures
Copy link
Member Author

Choose a reason for hiding this comment

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

doc/ecosystem.md Outdated Show resolved Hide resolved

#### Xarray

[Xarray][] is an open source project and Python package that introduces labels in the
Copy link
Member Author

Choose a reason for hiding this comment

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

This paragraph comes from https://xarray.dev/.

doc/ecosystem.md Outdated Show resolved Hide resolved
doc/ecosystem.md Outdated

#### rioxarray

[rioxarray][] is a geospatial [Xarray][] extension powered by rasterio. Built on top of
Copy link
Member Author

Choose a reason for hiding this comment

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

I can't remember where I found the rioxarray description. Maybe it's written by ChatGPT.


#### PyArrow

[Apache Arrow][] is a development platform for in-memory analytics. It contains a set of
Copy link
Member Author

Choose a reason for hiding this comment

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

(also named "PyArrow") have first-class integration with NumPy, pandas, and built-in
Python objects. They are based on the C++ implementation of Arrow.

```{note}
Copy link
Member Author

Choose a reason for hiding this comment

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

This note is copied from https://www.pygmt.org/dev/install.html with updates from PR #2933.

The plan is to remove the note from https://www.pygmt.org/dev/install.html. See PR #3506.

Copy link
Member Author

Choose a reason for hiding this comment

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

It's likely this PR will be merged before PR #2933, so I've reverted the changes in PR #2933 (770968e) and use the original notes from the installation guides.


Various packages rely on PyGMT for geospatial data processing, analysis, and visualization.
Below is an incomplete list (in no particular order) of tools which form the PyGMT-related
ecosystem.
Copy link
Member Author

Choose a reason for hiding this comment

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

For a package that rely on PyGMT, I think a short description about the package itself and how PyGMT is used in the package is enough.

For example, I know SHTOOLS maintained by MarkWieczorek uses PyGMT. I expect to see it listed here like below:

### SHTOOLS

pyshtools is an archive of Python software that can be used for spherical harmonic transforms, multitaper spectral analyses, expansions of gridded data into Slepian basis functions, and standard operations on global gravitational and magnetic field data.

It uses PyGMT for the creation of publication quality maps.

I don't plan to add SHTOOLS myself. Instead, after merging this PR, I prefer to ping MarkWieczorek to see if he is willing to add SHTOOLS himself.

Copy link
Member

Choose a reason for hiding this comment

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

Sounds good to me.

@seisman
Copy link
Member Author

seisman commented Oct 15, 2024

@GenericMappingTools/pygmt-maintainers I think this PR is ready for final reviews.

doc/ecosystem.md Outdated Show resolved Hide resolved
doc/index.md Outdated Show resolved Hide resolved
@seisman
Copy link
Member Author

seisman commented Oct 18, 2024

Ping @GenericMappingTools/pygmt-maintainers for final reviews. Will merge in 48 hours if no further comments.

@seisman seisman added final review call This PR requires final review and approval from a second reviewer and removed needs review This PR has higher priority and needs review. labels Oct 18, 2024
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 final review call This PR requires final review and approval from a second reviewer
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants