-
Notifications
You must be signed in to change notification settings - Fork 229
Closed
Labels
bugSomething isn't workingSomething isn't working
Milestone
Description
Description of the problem
The "PRICIPAL_AXIS" convention has 10 parameters: the value (in 10*exponent dynes-cm), azimuth, and plunge of N, T, and P axis, and also the exponent of the values. See https://docs.generic-mapping-tools.org/latest/supplements/seis/meca.html#s for the official documentation. Currently, the meca
function defines the following parameters:
Lines 255 to 266 in 4b80f3c
"PRINCIPAL_AXIS": [ | |
"t_exponent", | |
"t_azimuth", | |
"t_plunge", | |
"n_exponent", | |
"n_azimuth", | |
"n_plunge", | |
"p_exponent", | |
"p_azimuth", | |
"p_plunge", | |
"exponent", | |
], |
The parameter name t_exponent
, n_exponent
, and p_exponent
is inaccurate and confusing, and should be renamed to t_value
, n_value
, and p_value
.
I think we should make the change in a backward-compatible way and it should go through the deprecation cycle.
Metadata
Metadata
Assignees
Labels
bugSomething isn't workingSomething isn't working