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

Convert hex-binning test to a doc figure instead #7969

Merged
merged 4 commits into from
Oct 24, 2023
Merged

Conversation

PaulWessel
Copy link
Member

Refer to #7911. Basically remove test/gmtbinstat/hexbinning.sh and build doc/scripts/GMT_hexbinning.sh from it but make it horizontal, add a) b) tags, and a color bar.

GMT_hexbinning

DVC updated with removal of old hexbinning.ps under test/baseline/gmtbinstat and adding GMT_hexbinning.ps to doc/scripts/images.

Basically remote test/gmtbinstat/hexbinning.sh and build doc/scripts/GMT_hexbinning.sh from it but make it horizontal and add a) b).
@PaulWessel PaulWessel added the documentation Improve documentation label Oct 23, 2023
@PaulWessel PaulWessel added this to the 6.5.0 milestone Oct 23, 2023
@PaulWessel PaulWessel requested a review from a team October 23, 2023 15:39
@PaulWessel PaulWessel self-assigned this Oct 23, 2023
@github-actions
Copy link
Contributor

Summary of changed images

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

Status Path
added doc/scripts/images/GMT_hexbinning.ps
deleted test/baseline/gmtbinstats/hexbinning.ps
modified doc/scripts/images/
modified test/baseline/gmtbinstats/

Image diff(s)

Added images

  • doc/scripts/images/GMT_hexbinning.png

Modified images

Path Old New

Report last updated at commit a5e215f

@seisman seisman merged commit aa79c9a into master Oct 24, 2023
7 checks passed
@seisman seisman deleted the binstat-doc branch October 24, 2023 13:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improve documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants