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

Reestablish CPT hinge for master tables #8111

Merged
merged 3 commits into from
Nov 30, 2023
Merged

Reestablish CPT hinge for master tables #8111

merged 3 commits into from
Nov 30, 2023

Conversation

PaulWessel
Copy link
Member

We (I) somehow lost a line or two in gmt_support.c. Added a test as well so that we are warned if this tests fails again. The CPT polar has a soft hinge at 0 and that is only activated via the +h[hinge] modifier at the end of the selected CPT. Output is below. Note that

  1. top plot sets no hinge so the white center is at the mid-point -1368.5.
  2. Middle panel uses a hinge of 2000 and creates a new cpt via makecpt. Now the white point is at z = 2000.
  3. Bottom panel activates the soft-hinge at 0 so that is where the white point falls.

cpt_hinge

All tests pass (well, that use to pass).

We (I) somehow lost a line or two in gmt_support.c.  Added a test as well so that we are warned if this tests fails again.  Output is like this:
Add test script cpt_hinge.sh.
@PaulWessel PaulWessel added the bug Something isn't working label Nov 30, 2023
@PaulWessel PaulWessel added this to the 6.5.0 milestone Nov 30, 2023
@PaulWessel PaulWessel self-assigned this Nov 30, 2023
Copy link
Member

@Esteban82 Esteban82 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Closes #8102

@Esteban82 Esteban82 merged commit 5b617fe into master Nov 30, 2023
@Esteban82 Esteban82 deleted the cpt-hinge-fix branch November 30, 2023 14:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants