Contributors Guide — GMT 6.3.0 documentation

# Contributors Guide

This is a community driven project and everyone is welcome to contribute. The project is hosted at the GMT GitHub repository.

The goal is to maintain a diverse community that’s pleasant for everyone. Please be considerate and respectful of others. Everyone must abide by our Code of Conduct and we encourage all to read it carefully.

## Ways to Contribute

### Ways to Contribute Documentation and/or Code

- Tackle any issue that you wish! Please leave a comment on the issue indicating that you want to work on it. Some issues are labeled as “good first issues” to indicate that they are beginner friendly, meaning that they don’t require extensive knowledge of the project.

- Make a tutorial or example of how to do something.

- Improve the API documentation.

- Contribute code you already have. It doesn’t need to be perfect! We will help you clean things up, test it, etc.

### Ways to Contribute Feedback

- Provide feedback about how we can improve the project or about your particular use case. Open an issue with feature requests of bug fixes, or post general comments/questions on the forum.

- Help triage issues, or give a “thumbs up” on issues that others reported which are relevant to you.

### Ways to Contribute to Community Building

- Participate and answer questions on the GMT Community Forum.

- Participate in discussions at the monthly GMT Community Meetings, which are announced on the forum governance page.

- Cite GMT when using the project.

- Spread the word about GMT or start the project!

## Providing Feedback

### Reporting a Bug

- Find the Issues tab on the top of the GitHub repository and click New Issue.

- Click on Get started next to Bug report.

- Please try to fill out the template with as much detail as you can.

- After submitting your bug report, try to answer any follow up questions about the bug as best as you can.

### Submitting a Feature Request

- Find the Issues tab on the top of the GitHub repository and click New Issue.

- Click on Get started next to Feature request.

- Please try to fill out the template with as much detail as you can.

- After submitting your feature request, try to answer any follow up questions as best as you can.

### Submitting General Comments/Questions

There are several pages on the Community Forum where you can submit general comments and/or questions:

- For questions about using GMT, select New Topic from the Q&A Page.

- For general comments, select New Topic from the Lounge Page.

- To share your work, select New Topic from the Showcase Page.

## General Guidelines

### Resources for New Contributors

Please take a look at these resources to learn about Git and pull requests (don’t hesitate to ask for help):

- How to Contribute to Open Source.

- Git Workflow Tutorial.

- How to Contribute to an Open Source Project on GitHub.

### Getting Help

Discussion often happens in the issues and pull requests. For general questions, you can post on the GMT Community Forum. We also host community meetings roughly monthly to discuss GMT development, which are announced on the GMT Community Forum.

### Pull Request Workflow

We follow the git pull request workflow to make changes to our codebase. Every change made goes through a pull request, even our own, so that our continuous integration services have a change to check that the code is up to standards and passes all our tests. This way, the master branch is always stable.

#### General Guidelines for Making a Pull Request (PR):

- Open an issue first describing what you want to do. If there is already an issue that matches your PR, leave a comment there instead to let us know what you plan to do.

- Each pull request should consist of a small and logical collection of changes.

- Larger changes should be broken down into smaller components and integrated separately.

- Bug fixes should be submitted in separate PRs.

- Describe what your PR changes and why this is a good thing. Be as specific as you can. The PR description is how we keep track of the changes made to the project over time.

- Do not commit changes to files that are irrelevant to your feature or bugfix (eg: .gitignore, IDE project files, etc).

- Write descriptive commit messages. Chris Beams has written a guide on how to write good commit messages.

- Be willing to accept criticism and work on improving your code; we don’t want to break other users’ code, so care must be taken to not introduce bugs.

- Be aware that the pull request review process is not immediate, and is generally proportional to the size of the pull request.

#### General Process for Pull Request Review:

After you’ve submitted a pull request, you should expect to hear at least a comment within a couple of days. We may suggest some changes or improvements or alternatives.

Some things that will increase the chance that your pull request is accepted quickly:

- Write a good and detailed description of what the PR does.

- Write tests for the code you wrote/modified.

- Readable code is better than clever code (even with comments).

- Write documentation for your code and leave comments explaining the reason behind non-obvious things.

- Include an example of new features in the gallery or tutorials.

Pull requests will automatically have tests run by GitHub Actions. Github will show the status of these checks on the pull request. Try to get them all passing (green). If you have any trouble, leave a comment in the PR or get in touch.

## Setting up your environment

For editing the documentation locally and contributing code, you will need a development environment. To set up a development environment, follow the steps for building GMT paying attention to several “Note for developers” that provide helpful or necessary information for running tests and building the documentation.

To enable testing, you need to uncomment the following lines in your `ConfigUserAdvanced.cmake` when configuring GMT:

    enable_testing()
    set (DO_EXAMPLES TRUE)
    set (DO_TESTS TRUE)

    set (SUPPORT_EXEC_IN_BINARY_DIR TRUE)

    set (DO_API_TESTS ON)

Optionally, uncomment the following line and change `4` to the number of ctest jobs to run simultaneously:

    set (N_TEST_JOBS 4)

### Updating the development source codes

Assuming you did not delete the build directory, this is just as simple as:

    cd <path-to-gmt>
    git pull
    cd build
    cmake --build .
    cmake --build . --target install

CMake will detect any changes to the source files and will automatically reconfigure. If you deleted all files inside the build directory you have to run CMake again manually.

### Using build and test aliases

The file `/admin/bashrc_for_gmt` contains useful aliases for building and testing GMT that some developers chose to use. New pull requests with other aliases that you find helpful are welcome. This file is version controlled, so you should copy the file to a different location in order to edit and use it. For example, use these commands to copy it to your home directory:

    cd <path-to-gmt>
    cp admin/bashrc_for_gmt ~/.bashrc_for_gmt

Here are the steps for setting up `bashrc_for_gmt` after copying it to a new location:

- If you do not have ninja installed, you will need to change `builder=ninja` to `builder=make` and `Bname="Ninja"` to `Bname="Unix Makefiles"`. Ninja is recommended for speeding up build times.

- You may need to update `pngview=open` and `pdfview=open` depending on your preferred program for viewing files.

- Optionally, change `ncores=4` to the number of cores to use for building and running tests.

- Change `MATLAB=/Applications/MATLAB_R2019a.app` to the path for your version of the MATLAB app.

- Set `REPO_DIR` to the path that contains the local `git clone` copy of the GMT repository.

- Set `DATA_DIR` to the path that contains the folders `dcw-gmt-2.0.0/` and `gshhg-gmt-2.3.7/` for the dcw and gshhg datasets respectively. If these folders are not located in the same path, you can instead delete the line (`DATA_DIR=<path to directory containing GSHHG and DCW>`) and set the individual paths to the GSHHG and DCW source by changing (`export GMT_GSHHG_SOURCE=${DATA_DIR}/gshhg-gmt-2.3.7`) and (`export GMT_DCW_SOURCE=${DATA_DIR}/dcw-gmt-2.0.0`).

- Edit the file `~/.bashrc` to include the line `source <path>/bashrc_for_gmt`. If you set up `bashrc_for_gmt` as a hidden file in your home directory, this line should be `source ~/.bashrc_for_gmt`.

Here are some of the shortcuts included in `bashrc_for_gmt`:

- `gmt6` and `gtop` can be used to quickly `cd` to the top of the GMT source directory and repository base respectively.

- `gmtfind` can be used to list all source, docs, scripts, and text files where a string appears in the file (e.g., `gmtfind "Grid increment is"` returns all files that contain the string ‘Grid increment is’). This includes all files recursively from the current working directory; `gtop` or `gmt6` can be used prior to this command to get to the source directory or repository base.

- `cmakegmtd`, `cmakegmtr`, and `cmakegmtx` configures cmake for debug, release, and XCode debug respectively.

- `dlog` and `rlog` can be used to open the debug and release build check error logs respectively.

- There are several aliases with various combinations of pulling new changes, deleting the build directories, configuring cmake, and building the source code. Each of these are documented with comments in `bashrc_for_gmt`.

- `checkdbuild` and `checkrbuild` can be used to run the tests for the debug and release builds respectively.

- `vpngdbuild` and `vpdfdbuild` can be used to open the results from all failing image-based tests.

- `view_png_failures_r` and `view_pdf_failures_r` can be used for view failures of the release build with a lag between opening each file.

## Contributing Documentation

If you’re browsing the documentation and notice a typo or something that could be improved, please consider letting us know. You can either create an issue on GitHub, or click the “Edit on GitHub” button at the top right corner of the documentation, and submit a pull request.

The GMT documentation is written in the plaintext markup language reStructuredText (reST) and built by documentation generator Sphinx. The reST plaintext files for the GMT documentation are located in the doc/rst/source folder. You may need to know some basic reST syntax before making changes. Please refer to our reStructuredText Cheatsheet for details.

### Building the documentation

Usually you don’t need to build the documentation locally for small changes. To build the GMT documentation you need to install the Sphinx documentation builder. After configuring and building GMT, you can build GMT documentation using the following commands within the build directory:

    cmake --build . --target docs_depends     # Generate images included in the documentation
    cmake --build . --target optimize_images  # Optimize PNG images for documentation [optional]
    cmake --build . --target docs_man         # UNIX manual pages
    cmake --build . --target docs_html        # HTML manual, tutorial, cookbook, and API reference

Note

- Refer to the file `admin/bashrc_for_gmt` for useful aliases for building the documentation.

- pngquant is needed for optimizing images.

### Contributing an animation

The animations are built from the scripts in `doc/examples/anim*/`. To add a new animation:

- Open an issue with your idea for a new animation. It is best to get some feedback on your idea before starting work on the animation. If you do have an animation already made, you can share it as part of the new issue.

- Create a new script `doc/examples/anim??/anim??.sh`, where ?? is the number of the new example. Be sure to follow the style of the existing animations, including using `#!/usr/bin/env bash` and including the purpose, list of modules and unix programs used, and any relevant notes. Use enough comments in your script to make it easily interpretable.

- Create a new ReStructured Text document `doc/rst/source/animations/anim??.rst`, where ?? is the number of the new example. Follow the same format as the other anim??.rst files, including the ReST target `.. _anim??:` at the top, a title, and a description of the animation.

- Add a directive that will include the source code in the built documentation in `doc/rst/source/animations/anim??.rst`:

      .. literalinclude:: /_verbatim/anim??.txt
        :language: bash

- Add a placeholder `.. youtube::` directive to the `doc/rst/source/animations/anim??.rst` file:

      ..  youtube:: Pvvc4vb8G4Y
        :width: 100%

- Add a placeholder gallery item to the end of the list of animations in `doc/rst/source/animations.rst`:

      .. youtube:: Pvvc4vb8G4Y
        :width: 100%

        :doc:`/animations/anim??`

- Submit a pull request with your new animation. Please be sure to follow the pull request template and include the built animation in the pull request or provide a link to the built animation.

- If the pull request is approved, one of the GMT maintainers will build the animation, upload it to the Generic Mapping Tools YouTube channel, and update the links to the YouTube video in `doc/rst/source/animations/anim??.rst` and `doc/rst/source/animations.rst`.

## Contributing Code

The source code for GMT is locating in the src/ directory. When contributing code, be sure to follow the general guidelines in the pull request workflow section.

### Code Style

When modifying or submitting new source code, make sure that your code follows the GMT code style. Use the other functions/files in the src/ directory as a basis. Here are some specific guidelines:

- Use tabs, rather than spaces, for indentation.

- Try to split lines at ~120 characters.

### Testing GMT

GMT ships with more than 1000 tests to make sure that any changes won’t break its functionality. In addition to the tests located in the `/test` directory, GMT tests all the plots included in its documentation. The documentation tests are located in the `/doc/scripts` directory. The majority of GMT tests are plot-based, with each test requiring a bash script for generating the plot and a reference PostScript file. These tests pass if the difference between a new plot generated using the test script and the reference PostScript file is less than a defined threshold. Other tests compute grids, tables, or other output, with the test passing if a suitable comparison is made against a reference case.

Tests that are known to fail are excluded by adding `# GMT_KNOWN_FAILURE` anywhere in the test script.

#### Running tests

After configuring CMake and building GMT, you can run all the tests by running this command in the build directory:

    cmake --build . --target check

You can also run `ctest` commands in the build directory. Below are some common used ctest commands.

- Run all tests in 4 parallel jobs:

      ctest -j 4

- Re-run all failing tests in previous run in 4 parallel jobs:

      ctest -j 4 --rerun-failed

- Select individual tests using regexp with ctest:

      ctest --output-on-failure -R ex2[3-6]

Note

Refer to the file `admin/bashrc_for_gmt` for useful aliases for running the tests.

#### Reviewing test failures

There are several tests that are “known to fail” for GMT. Unless the `GMT_ENABLE_KNOWN2FAIL` variable is set when configuring CMake or setting up `ConfigUserAdvanced.cmake`, these tests are excluded when running ctest using the instructions provided in the Running tests section. Therefore, you should expect all tests to pass unless something new is broken.

Information about failing tests is produced in `test/fail_count.txt` inside the build directory. For plot-based tests, the subdirectories `test/` and `doc/scripts/` inside the build directory contain folders for each failing test. For plot-based tests, the directory associated with each failing tests contains a `gmtest.sh` script, a `gmt.conf` file, an alias to the test script, a PostScript file and PDF document generated by the test script, and a PNG image that shows differences between the reference plot and new plot in magenta. In addition to these files, running the failing tests with verbose output can be helpful for evaluating failures:

    ctest --rerun-failed --verbose

#### Updating reference plots for tests

Pull requests should avoid needing to change PostScript files in the `/test` and `doc/scripts/` directories. However, if this is unavoidable, new PostScript reference files can be generated by running `ctest -R <test-script-name>` in the build directory after following the building guide, the setting up your environment instructions, and the running tests instructions. The new PostScript file can then be copied from the appropriate subdirectory within `build/test/` or `build/doc/scripts/` to `/test` or `/doc/scripts` respectively.

#### Adding new tests

If you are fixing a bug or adding a new feature, you should add a test with your pull request. Most of the tests are image based and compare a result against a reference PostScript file using GraphicsMagick.

To add a PostScript based test (e.g., box.sh):

- Create a new shell script in the subdirectory under `test/` that corresponds to the module you are testing. The name of the shell script should be descriptive and unique.

- Include `#!/usr/bin/env bash` and a short description of the test at the top of the script.

- Add the content of the script that will create a PostScript file. Some general guidelines:

  - Use as small a dataset as possible. See the GMT server cache for some example datasets that can be used.

  - Keep the script as simple as possible, with as few commands and options as needed to test the feature, enhancement, or bug fix.

  - Minimize the size of the resultant PostScript file as much as possible.

- Run the tests using the instructions in the running tests section.

- Check that the new PostScript file in `build/test/<module>` or `build/doc/scripts/` is as-expected.

- Copy the new PostScript file from the appropriate subdirectory within `build/test/` or `build/doc/scripts/` to `test/<module>` or `doc/scripts` respectively.

- Check that your new test working by rerunning the failing tests.

To add a non-PostScript based test (e.g., gmean.sh):

- Create a new shell script in the subdirectory under `test/` that corresponds to the module you are testing. The name of the shell script should be descriptive and unique.

- Include `#!/usr/bin/env bash` and a short description of the test at the top of the script.

- Structure the test so that it produced both a reference file with the expected output (e.g., using `echo` or `cat`) in a file `answer.txt` (for a text-based case) and the test output from gmt in a file `result.txt`.

- Add a `diff` command that will compare the ``result.txt` and `answer.txt` files and create a file `fail` if the files do not match (e.g., `diff -q --strip-trailing-cr answer.txt result.txt > fail`).

- Check that your new test works using the instructions in the running tests section.

### Debugging GMT

Guides for debugging GMT are provided in the Debugging GMT section of the GMT documentation.