[ https://issues.apache.org/jira/browse/LUCENE-1930?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Erick Erickson resolved LUCENE-1930. ------------------------------------ Resolution: Won't Fix 2013 Old JIRA cleanup > Scale moderator not in line with sinusoidal projector > ----------------------------------------------------- > > Key: LUCENE-1930 > URL: https://issues.apache.org/jira/browse/LUCENE-1930 > Project: Lucene - Core > Issue Type: Bug > Components: modules/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 = projector.coords(latitude, longitude); > double id = getBoxId(coords[0]) + (getBoxId(coords[1]) / > tierVerticalPosDivider); > return id ; > } > private double getBoxId (double coord){ > return Math.floor(coord / (idd / this.tierLength)); > } > with > Double idd = new Double(180); > in the CartesianTierPlotter constructor. > But current Sinusoidal Projector set and used in initialisation of > CartesianTierPlotter does : > public double[] coords(double latitude, double longitude) { > double rlat = Math.toRadians(latitude); > double rlong = Math.toRadians(longitude); > double nlat = rlong * Math.cos(rlat); > double r[] = {nlat, rlong}; > return r; > } > Thus we moderate with idd = 180 a coord that is in a Radian space. > Things to do : > 1°) fix idd to : double idd= PI; > 2°) Move idd definition to IProjector interface : The coord space should > belong to the projector doing the job. Change the code from CTP to use that > new interface. -- This message was sent by Atlassian JIRA (v6.1#6144) --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@lucene.apache.org For additional commands, e-mail: dev-h...@lucene.apache.org