Opened 15 years ago
Last modified 11 years ago
#5 reopened defect
Fix Datum conversion
| Reported by: | mdavis | Owned by: | |
|---|---|---|---|
| Priority: | critical | Milestone: | Initial Release |
| Component: | core library | Version: | 0.1 |
| Keywords: | Cc: |
Description
There are known issues with datum conversion
Attachments (1)
Change History (4)
by , 14 years ago
| Attachment: | Datum.java.diff added |
|---|
comment:1 by , 14 years ago
Proj4 (C library) has a conversion for the rotation and scale parameters used by the 7 parameter datum conversion. The conversion is done in pj_datum_set.c. This conversion is missing in the current Proj4j (Java library) version. I attached a a patch (Datum.java.diff) to fix this.
comment:3 by , 11 years ago
| Resolution: | fixed |
|---|---|
| Status: | closed → reopened |
I'm reopening this issue because the suggested patch conflicts with Proj4Parser.parseToWGS84(). This leads to twice-converting arcsec->rad. Good news is this happen only in one place - it is called only in Proj4Parser.parse(), so the error appears only when CRS is parsed from string, but not when programatically constructed. I think the addition in patch is mutually exclusive with the converting fragment in Proj4Parser.parseToWGS84().
Patch to fix 7 parameters datum conversion