Skip to content

Wrap GMT's standard data type GMT_CUBE for cubes and support reading/writing cubes - #3150

Open
seisman wants to merge 36 commits into
mainfrom
datatypes/gmtcube
Open

Wrap GMT's standard data type GMT_CUBE for cubes and support reading/writing cubes#3150
seisman wants to merge 36 commits into
mainfrom
datatypes/gmtcube

Conversation

@seisman

@seisman seisman commented Mar 31, 2024

Copy link
Copy Markdown
Member

Similar to #3338 and #3128, but for wrapping GMT_CUBE.

Main changes in this PR:

  • Wrap GMT's standard data type GMT_CUBE
  • Implement the to_xarray method
  • Let low-level functions (e.g., read_data/write_data) support cubes
  • Extend the xarray backend to support cubes (i.e., xr.open_dataarray("xxx.cube", engine="gmt", kind="cube")

Known issues:

  • It seems we can't use lib.call_module("read", "@cube.nc out.nc -Tu") for cubes. The similar command works well for grids and images. It's likely an upstream bug.

Base automatically changed from gmtgrid/parser to main April 1, 2024 05:56
@seisman
seisman force-pushed the datatypes/gmtcube branch from fe976e4 to 1c5df8c Compare April 1, 2024 06:00
Comment thread pygmt/datatypes/cube.py Outdated
@seisman
seisman force-pushed the datatypes/gmtcube branch from 499c519 to b59efe0 Compare July 19, 2024 08:51
@seisman seisman added the feature Brand new feature label Jul 27, 2024
@seisman seisman removed this from the 0.14.0 milestone Sep 5, 2024
@seisman seisman self-assigned this Oct 30, 2024
@seisman seisman changed the title Wrap GMT's standard data type GMT_CUBE for cubes Wrap GMT's standard data type GMT_CUBE for cubes and support reading/writing cubes Aug 25, 2026
@seisman seisman added this to the 0.20.0 milestone Aug 25, 2026
@seisman
seisman marked this pull request as ready for review August 25, 2026 16:54
@seisman seisman added the needs review This PR has higher priority and needs review. label Aug 25, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feature Brand new feature needs review This PR has higher priority and needs review.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants