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)

Datum.java.diff (3.6 KB ) - added by idema 14 years ago.
Patch to fix 7 parameters datum conversion

Download all attachments as: .zip

Change History (4)

by idema, 14 years ago

Attachment: Datum.java.diff added

Patch to fix 7 parameters datum conversion

comment:1 by idema, 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:2 by mdavis, 12 years ago

Resolution: fixed
Status: newclosed

Fixed

comment:3 by svetlio, 11 years ago

Resolution: fixed
Status: closedreopened

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().

Note: See TracTickets for help on using tickets.