geoapi matrix requires public clone method

Description

The org.opengis.referencing.operation.Matrix class in the geoapi jar declaires a public clone method.

The matrix class in org.geotools.referencing.operation does not (nore does the GMatrix class in vecmath which it extends)

This is preventing compilation of this class and of ct.MatrixParameters and pt.Matrix

Environment

None

Activity

Show:
codehaus
April 10, 2015, 3:18 PM

CodeHaus Comment From: desruisseaux - Time: Mon, 31 May 2004 06:49:43 -0500
---------------------
This problem is already discuss in <a href="https://jira.codehaus.org/browse/GEOT-22" title="replace use of vecmath with an open alternative" class="issue-link" data-issue-key="GEOT-22">GEOT-22</a> (&quot;replace &#39;vecmath&#39; by an open source alternative&quot. The problem is that the free version of &#39;vecmath&#39; is Java3D 1.2 compliant, while the latest version of Sun&#39;s vecmath is Java3D 1.3 compliant. GMatrix has a &#39;clone()&#39; method, but only since 1.3.

The problem with the free version seems much more serious than just lacking the &#39;clone()&#39; method. For some test I did, it appears that the free and Sun&#39;s implementation of &#39;vecmath&#39; produces completly different numbers for some operations. Rueben reported that when using the free version of &#39;vecmath&#39;, a lot of test cases (which were used to pass with Sun&#39;s implementation) now fail.

I downloaded the free vecmath source code, added the missing &#39;clone()&#39; method and recreated the JAR. We should put this JAR instead of the current one in ibiblio as a fix for this issue. But the remaining issue (producing different results) still open.

codehaus
April 10, 2015, 3:18 PM

CodeHaus Comment From: desruisseaux - Time: Mon, 7 Jun 2004 18:07:01 -0500
---------------------
The code for the free version has been commited to the CVS here:

<a href="http://svn.geotools.org/geotools/trunk/vecmath/">http://svn.geotools.org/geotools/trunk/vecmath/</a>

The &#39;clone()&#39; method has been added, but the other numerical problems has not be resolved. For now, we are switching back to Sun&#39;s implementation. If distribution is a problem, please fix the numerical issues in the free version.

codehaus
April 10, 2015, 3:18 PM

CodeHaus Comment From: desruisseaux - Time: Thu, 1 Jul 2004 05:44:18 -0500
---------------------
We are now allowed to distribute the Sun&#39;s vecmath implementation under the Java Distribution License. See <a href="http://vecmath.dev.java.net/">http://vecmath.dev.java.net/</a>

Concequently, the free implementation (vecmath module) has been deleted from SVN and the clone() method is no longer an issue.

Assignee

Unassigned

Reporter

codehaus

Triage

None

Components

Fix versions

Priority

Highest
Configure