Skip to content

Fix issues with Aki-Richards convention - #9114

Merged
Esteban82 merged 1 commit into
masterfrom
fix_6840_Aki-Richards
Jul 27, 2026
Merged

Fix issues with Aki-Richards convention#9114
Esteban82 merged 1 commit into
masterfrom
fix_6840_Aki-Richards

Conversation

@Esteban82

Copy link
Copy Markdown
Member

Done with Claude 5.0

This PR fixes the issues reported in #6840 for the Aki-Richards convention.

I use the following input data:

112 32 25  30  90   0  4.5  112 33 Strike-slip
115 34 15  30  60  90  5    115 32 Reverse
118 32 45  30  60 -90  5.4  118 34 Normal

The fix was tested using the following scripts:

Issue: -S+l may not work as expected

The script doesn't plot any beachballs with +l.

gmt begin 6840 png
    gmt basemap -R110/120/30/35 -JM10c -Baf -B+t"Don't plot any beachball"
    gmt meca meca.dat -Sa1c+l -T0
gmt end

Output with this fix:
6840

Issue: Sometimes the nodal plane lines are not drawn correctly:

gmt begin beachball5_T0 png
    gmt basemap -R110/120/30/35 -JM10c -Baf -B+t"Nodal planes lines not drawn (T0)"
    gmt meca meca.dat -Sa1c -T0
gmt end

Output with this fix:

beachball5_T0

Issue: -T0/red doesn't work

gmt begin beachball5_T0red png
    gmt basemap -R110/120/30/35 -JM10c -Baf -B+t"-T0/red doesn't work but"
    gmt meca meca.dat -Sa1c -T0/red
gmt end

Output with this fix:
beachball5_T0red

Issue: -Lblue doesn't work

gmt begin beachball5_Lblue png
    gmt basemap -R110/120/30/35 -JM10c -Baf -B+t"-Lblue doesn't work"
    gmt meca meca.dat -Sa1c -Lblue
gmt end

Output with this fix:

beachball5_Lblue

Issue: -A+pred also doesn't work,

Output with this fix:

gmt begin beachball5_A+p png
    gmt basemap -R110/120/30/35 -JM10c -Baf -B+t"-A+pred also doesn't work"
    gmt meca meca.dat -Sa1c -A+pred
gmt end

Output with this fix:

beachball5_A+p

Fix #6840

@Esteban82
Esteban82 requested review from joa-quim and seisman July 27, 2026 01:48
@Esteban82 Esteban82 added add-changelog Add PR to the changelog AI-assisted All (or most) of the code was written by Artificial Intelligence. labels Jul 27, 2026

@joa-quim joa-quim left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

But we must stop submitting new PRs before 6.7.0 is created

@Esteban82

Copy link
Copy Markdown
Member Author

But we must stop submitting new PRs before 6.7.0 is created

Do I merge it? or should I wait after 6.7.0?

@joa-quim

Copy link
Copy Markdown
Member

Go ahead and merge.
True that we could make branches that hold until after release but we know how that works.

@Esteban82
Esteban82 merged commit 7fed33a into master Jul 27, 2026
11 of 14 checks passed
@Esteban82
Esteban82 deleted the fix_6840_Aki-Richards branch July 27, 2026 15:31
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 AI-assisted All (or most) of the code was written by Artificial Intelligence.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

meca: Possible bugs or improvements

2 participants