Lucene - Core
  1. Lucene - Core
  2. LUCENE-1930

Scale moderator not in line with sinusoidal projector

    Details

    • Type: Bug Bug
    • Status: Resolved
    • Priority: Major Major
    • Resolution: Won't Fix
    • Affects Version/s: 2.9, 2.9.1, 3.0
    • Fix Version/s: 4.0-BETA
    • Component/s: modules/spatial
    • Labels:
      None
    • Lucene Fields:
      Patch Available

      Description

      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.

      1. LUCENE-1930.patch
        4 kB
        Nicolas Helleringer

        Issue Links

          Activity

          Nicolas Helleringer created issue -
          Nicolas Helleringer made changes -
          Field Original Value New Value
          Attachment LUCENE-1930.patch [ 12420781 ]
          Chris Male made changes -
          Link This issue relates to LUCENE-2139 [ LUCENE-2139 ]
          Nicolas Helleringer made changes -
          Affects Version/s 3.0 [ 12312889 ]
          Affects Version/s 2.9.1 [ 12314295 ]
          Lucene Fields [New] [Patch Available]
          Chris Male made changes -
          Assignee Chris Male [ cmale ]
          Mark Thomas made changes -
          Workflow jira [ 12478099 ] Default workflow, editable Closed status [ 12563122 ]
          Mark Thomas made changes -
          Workflow Default workflow, editable Closed status [ 12563122 ] jira [ 12584074 ]
          Erick Erickson made changes -
          Status Open [ 1 ] Resolved [ 5 ]
          Resolution Won't Fix [ 2 ]
          Erick Erickson made changes -
          Resolution Won't Fix [ 2 ]
          Status Resolved [ 5 ] Reopened [ 4 ]
          David Smiley made changes -
          Status Reopened [ 4 ] Resolved [ 5 ]
          Fix Version/s 4.0-BETA [ 12322456 ]
          Resolution Won't Fix [ 2 ]

            People

            • Assignee:
              Chris Male
              Reporter:
              Nicolas Helleringer
            • Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development