Migrate Figure.basemap tests to dvc

[seisman]

Description of proposed changes

• Remove three tests that only test the aliases
• Rewrite two tests using mpl_image_compare
• Rewrite tests using long-form parameters, SI units and small figure dimensions
• Migrate images to dvc

It would be easier to review each commit separately.

Address #1131.

Reminders

• Run make format and make check to make sure the code follows the style guide.
• Add tests for new features or tests that would have caught the bug that you're fixing.
• Add new public functions/methods/classes to doc/api/index.rst.
• Write detailed docstrings for all functions/methods.
• If adding new functionality, add an example to docstrings or tutorials.

Slash Commands

You can write slash commands (/command) in the first line of a comment to perform
specific operations. Supported slash commands are:

• /format: automatically format and lint the code
• /test-gmt-dev: run full tests on the latest GMT development version

[willschlitzer]

@seisman Do you want us to begin updating the current tests to use dvc? I spoke to @weiji14 about it and his guidance was to wait until GMT 6.2 has been released to avoid having to update the reference images.

[seisman]

@seisman Do you want us to begin updating the current tests to use dvc? I spoke to @weiji14 about it and his guidance was to wait until GMT 6.2 has been released to avoid having to update the reference images.

See my comment #963 (comment):

Do we want to do the migration before GMT 6.2.0 or after? I prefer to do the migration before v6.2.0, although it means more work for us. After bumping GMT to 6.2.0, most tests will fail due to the changes in GMT 6.2.0, but I feel it's also a good opportunity for us to learn the GMT changes and find potential bugs by comparing the images generated by GMT 6.1.1 and 6.2.0.

FYI, one week ago, I built the PyGMT documentation using GMT 6.2.0, and found several issues with the GMT dev version (GenericMappingTools/gmt#4955), and they were all fixed in less than one week!

[github-actions]

Summary of changed images

This is an auto-generated report of images that have changed on the DVC remote

Status	Path
added	pygmt/tests/baseline/test_basemap.png
added	pygmt/tests/baseline/test_basemap_compass.png
added	pygmt/tests/baseline/test_basemap_loglog.png
added	pygmt/tests/baseline/test_basemap_map_scale.png
added	pygmt/tests/baseline/test_basemap_polar.png
added	pygmt/tests/baseline/test_basemap_power_axis.png
added	pygmt/tests/baseline/test_basemap_rose.png
added	pygmt/tests/baseline/test_basemap_winkel_tripel.png

Image diff(s)

• pygmt/tests/baseline/test_basemap.png

• pygmt/tests/baseline/test_basemap_compass.png

• pygmt/tests/baseline/test_basemap_loglog.png

• pygmt/tests/baseline/test_basemap_map_scale.png

• pygmt/tests/baseline/test_basemap_polar.png

• pygmt/tests/baseline/test_basemap_power_axis.png

• pygmt/tests/baseline/test_basemap_rose.png

• pygmt/tests/baseline/test_basemap_winkel_tripel.png

Report last updated at commit 33f61b6

[weiji14]

Still need to run make format.

[weiji14]

/format

[seisman]

/format

[seisman]

So typing /format in the review box doesn't work.