You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@sis.apache.org by "Martin Desruisseaux (JIRA)" <ji...@apache.org> on 2017/04/26 13:34:04 UTC
[jira] [Assigned] (SIS-327) Defer loading of datum shift grid files
[ https://issues.apache.org/jira/browse/SIS-327?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Martin Desruisseaux reassigned SIS-327:
---------------------------------------
Assignee: Martin Desruisseaux
> Defer loading of datum shift grid files
> ---------------------------------------
>
> Key: SIS-327
> URL: https://issues.apache.org/jira/browse/SIS-327
> Project: Spatial Information Systems
> Issue Type: Improvement
> Components: Referencing
> Affects Versions: 0.7
> Reporter: Martin Desruisseaux
> Assignee: Martin Desruisseaux
> Priority: Minor
> Fix For: 0.8
>
>
> When a {{CoordinateOperation}} is created, its {{MathTransform}} is created immediately. This is usually not so costly, except in the case of datum shift based on a grid files where the grid files is loaded immediately. This is a waste of resources when the caller needs create many such operations, compare their {{domainOfValidity}} and retain only one operation at the end.
> The proposal is to load the datum shift grid files only the first time that {{getMathTransform()}} is invoked. In addition to reduce the amount of resources to load, another effect of this would be to reduce the amount of warnings emitted because of grid files not present.
> *How to reproduce:* the snippet below produces a lot of warnings if the grid files are not present. After this issue is fixed, we should have at most 1 warning.
> {code:java}
> CoordinateReferenceSystem sourceCRS = CRS.forCode("EPSG:4267");
> CoordinateReferenceSystem targetCRS = CRS.forCode("EPSG:4326");
> CoordinateOperation op = CRS.findOperation(sourceCRS, targetCRS, null);
> {code}
> *Note:* we could defer the creation of all {{MathTransform}} objects, but this is advantageous only for the cases where many operations exist between the same pair of CRS. This is the case of datum shift mostly.
--
This message was sent by Atlassian JIRA
(v6.3.15#6346)