Skip to content

Allow gmtmath trailing text to be kept for output#7759

Merged
PaulWessel merged 5 commits intomasterfrom
gmtmath-trailing-text
Aug 24, 2023
Merged

Allow gmtmath trailing text to be kept for output#7759
PaulWessel merged 5 commits intomasterfrom
gmtmath-trailing-text

Conversation

@PaulWessel
Copy link
Member

This came from this forum post. I see no reason to not keep the trailing text in gmtmath as it can always be ignored with -on. The change in this PR is to not force prevention of text output and also due the initial duplication from input to output text.

cay << EOF > in.txt
-317516 109061 high ms0334
-317517 109062 low ms0334
EOF

# Keep trailing text
gmt math -Ca in.txt 3600 DIV =
-88.1988888889	30.2947222222	high ms0334
-88.1991666667	30.295	low ms0334

# Discard trailing text
gmt math -Ca -on in.txt 3600 DIV =
-88.1988888889	30.2947222222
-88.1991666667	30.295

This came from this forum post.  No reason to not keep the trailing text as it can always be ignored with -on.
@PaulWessel PaulWessel added the enhancement Improving an existing feature label Aug 24, 2023
@PaulWessel PaulWessel added this to the 6.5.0 milestone Aug 24, 2023
@PaulWessel PaulWessel requested a review from a team August 24, 2023 08:48
@PaulWessel PaulWessel self-assigned this Aug 24, 2023
PaulWessel and others added 2 commits August 24, 2023 10:56
Co-authored-by: Dongdong Tian <seisman.info@gmail.com>
@PaulWessel PaulWessel merged commit 30661f2 into master Aug 24, 2023
@PaulWessel PaulWessel deleted the gmtmath-trailing-text branch August 24, 2023 09:35
@maxrjones maxrjones added the add-changelog Add PR to the changelog label Dec 8, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

add-changelog Add PR to the changelog enhancement Improving an existing feature

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants