Mailing-List: contact java-dev-help@lucene.apache.org; run by ezmlm
Precedence: bulk
Reply-To: java-dev@lucene.apache.org
Message-ID: <1519887693.688791270480647411.JavaMail.jira@brutus.apache.org>
Date: Mon, 5 Apr 2010 15:17:27 +0000 (UTC)
From: "Nicolas Helleringer (JIRA)" <jira@apache.org>
To: java-dev@lucene.apache.org
Subject: [jira] Commented: (LUCENE-1930) Scale moderator not in line with
 sinusoidal projector
In-Reply-To: <1415403468.1254130456101.JavaMail.jira@brutus>
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: quoted-printable


    [ https://issues.apache.org/jira/browse/LUCENE-1930?page=3Dcom.atlassia=
n.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=3D128=
53399#action_12853399 ]=20

Nicolas Helleringer commented on LUCENE-1930:
---------------------------------------------

No it is not.

I ll make a updated one against current code base ASAP

> Scale moderator not in line with sinusoidal projector
> -----------------------------------------------------
>
>                 Key: LUCENE-1930
>                 URL: https://issues.apache.org/jira/browse/LUCENE-1930
>             Project: Lucene - Java
>          Issue Type: Bug
>          Components: contrib/spatial
>    Affects Versions: 2.9, 2.9.1, 3.0
>            Reporter: Nicolas Helleringer
>            Assignee: Chris Male
>         Attachments: LUCENE-1930.patch
>
>
> Current implementation in spatial Lucene does :
> public double getTierBoxId (double latitude, double longitude) {
>   double[] coords =3D projector.coords(latitude, longitude);
>   double id =3D getBoxId(coords[0]) + (getBoxId(coords[1]) / tierVertical=
PosDivider);
>   return id ;
> }
> private double getBoxId (double coord){
>   return Math.floor(coord / (idd / this.tierLength));
> }
> with
> Double idd =3D new Double(180);
> in the CartesianTierPlotter constructor.
> But current Sinusoidal Projector set and used in initialisation of Cartes=
ianTierPlotter does :
> public double[] coords(double latitude, double longitude) {
>   double rlat =3D Math.toRadians(latitude);
>   double rlong =3D Math.toRadians(longitude);
>   double nlat =3D rlong * Math.cos(rlat);
>   double r[] =3D {nlat, rlong};
> return r;
> }
> Thus we moderate with idd =3D 180 a coord that is in a Radian space.
> Things to do :
> 1=C2=B0) fix idd to : double idd=3D PI;
> 2=C2=B0) Move idd definition to IProjector interface : The coord space sh=
ould belong to the projector doing the job. Change the code from CTP to use=
 that new interface.

--=20
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


---------------------------------------------------------------------
To unsubscribe, e-mail: java-dev-unsubscribe@lucene.apache.org
For additional commands, e-mail: java-dev-help@lucene.apache.org