Wei Ji

This has come up again and again in different forms, so I've started a project at https://github.com/GenericMappingTools/pygmt/projects/6 to track the various issues and PRs tackling this. Let's make this issue...

> I agree that the classes option is good. I have a couple further opinions about the implementation of long GMT arguments: > > 1. Based on this discussion, we...

Just wanted to surface a point I made in https://github.com/GenericMappingTools/pygmt/pull/1418#discussion_r700746488. Please try and refrain from using `grdinfo` while writing unit tests for new wrapper functions (otherwise we'll have lots of...

> I would be pretty excited to see this in the next release, so I will get started on refactoring the tests. Here is a list of modules in `pygmt/tests/`...

I had a similar idea at https://github.com/GenericMappingTools/pygmt/pull/1695#discussion_r814250521 with the `RidgeTest.shp` file to store it using `dvc` in `pygmt/tests/data`. In that case, some of the files were binary and not plain-text...

> I did not find a conversation that settled on this convention in a quick search of past issues (apologies if I missed this). Yes, things are a bit inconsistent,...

> @weiji14 Would there be a way to just have the test suite run on one machine (presumably Ubuntu, since that typically is completed in the shortest period of time)...

Try setting your remote using ssh (`[email protected]:GenericMappingTools/pygmt.git`) instead of https (`https://github.com/GenericMappingTools/pygmt.git`) and see if it works.

You also need to add your ssh key to your GitHub account https://docs.github.com/en/github/authenticating-to-github/adding-a-new-ssh-key-to-your-github-account.

> It may be a GMTDataArrayAccessor bug, or maybe we should update `grdcut()` to work with GMTDataArrayAccessor. The problem is that the temporary netCDF file is already deleted when GMTDataArrayAccessor...