Open singhsatyampratap opened 2 months ago
Yes, that's true, it probably should be changed to "trench plate ID" here. I don't think that'll cause a problem - if I don't hear from any @GPlates/gplately-dev then I'll change that next week.
Originally the subduction convergence script (previously in PlateTectonicTools) used to find the plate ID of the overriding plate, but was subsequently changed to the plate ID of the trench line for this reason.
Description:
In the
PlateReconstruction.tessellate_subduction_zones(return_geodataframe=True)
function, the returned DataFrame includes a column namedoverriding_plate_id
. However, this column should be namedtrench_plate_id
to accurately reflect the data it represents. The currentoverriding_plate_id
values differ significantly from the trench plate IDs, as shown in the attached figure.Steps to Reproduce:
PlateReconstruction.tessellate_subduction_zones(return_geodataframe=True)
function.Please refer to the attached figure for a visual comparison of the
overriding_plate_id
andtrench_plate_id
.Attachment: