Uploaded image for project: 'Commons Math'
  1. Commons Math
  2. MATH-880

Polygon difference produces erronious results in some cases

    XMLWordPrintableJSON

Details

    • Bug
    • Status: Closed
    • Major
    • Resolution: Fixed
    • 3.0
    • 3.1
    • None
    • None

    Description

      The 2D polygon difference method is returning incorrect
      results. Below is a test case of subtracting two polygons (Sorry,
      this is the simplest case that I could find that duplicates the
      problem).

      There are three problems with the result. The first is that the first
      point of the first set of vertices is null (and the first point of the
      second set is also null). The second is that, even if the first null
      points are ignored, the returned polygon is not the correct result.
      The first and last points are way off, and the remaining points do not
      match the original polygon boundaries. Additionally, there are two
      holes that are returned in the results. This subtraction case should
      not have holes.

      "Complex Polygon Difference Test"
      public void testComplexDifference() {
              Vector2D[][] vertices1 = new Vector2D[][] {
                  new Vector2D[] {
                          new Vector2D( 90.08714908223715,  38.370299337260235),
                          new Vector2D( 90.08709517675004,  38.3702895991413),
                          new Vector2D( 90.08401538704919,  38.368849330127944),
                          new Vector2D( 90.08258210430711,  38.367634558585564),
                          new Vector2D( 90.08251455106665,  38.36763409247078),
                          new Vector2D( 90.08106599752608,  38.36761621664249),
                          new Vector2D( 90.08249585300035,  38.36753627557965),
                          new Vector2D( 90.09075743352184,  38.35914647644972),
                          new Vector2D( 90.09099945896571,  38.35896264724079),
                          new Vector2D( 90.09269383800086,  38.34595756121246),
                          new Vector2D( 90.09638631543191,  38.3457988093121),
                          new Vector2D( 90.09666417351019,  38.34523360999418),
                          new Vector2D( 90.1297082145872,  38.337670454923625),
                          new Vector2D( 90.12971687748956,  38.337669827794684),
                          new Vector2D( 90.1240820219179,  38.34328502001131),
                          new Vector2D( 90.13084259656404,  38.34017811765017),
                          new Vector2D( 90.13378567942857,  38.33860579180606),
                          new Vector2D( 90.13519557833206,  38.33621054663689),
                          new Vector2D( 90.13545616732307,  38.33614965452864),
                          new Vector2D( 90.13553111202748,  38.33613962818305),
                          new Vector2D( 90.1356903436448,  38.33610227127048),
                          new Vector2D( 90.13576283227428,  38.33609255422783),
                          new Vector2D( 90.13595870833188,  38.33604606376991),
                          new Vector2D( 90.1361556630693,  38.3360024198866),
                          new Vector2D( 90.13622408795709,  38.335987048115726),
                          new Vector2D( 90.13696189099994,  38.33581914328681),
                          new Vector2D( 90.13746655304897,  38.33616706665265),
                          new Vector2D( 90.13845973716064,  38.33650776167099),
                          new Vector2D( 90.13950901827667,  38.3368469456463),
                          new Vector2D( 90.14393814424852,  38.337591835857495),
                          new Vector2D( 90.14483839716831,  38.337076122362475),
                          new Vector2D( 90.14565474433601,  38.33769000964429),
                          new Vector2D( 90.14569421179482,  38.3377117256905),
                          new Vector2D( 90.14577067124333,  38.33770883625908),
                          new Vector2D( 90.14600350631684,  38.337714326520995),
                          new Vector2D( 90.14600355139731,  38.33771435193319),
                          new Vector2D( 90.14600369112401,  38.33771443882085),
                          new Vector2D( 90.14600382486884,  38.33771453466096),
                          new Vector2D( 90.14600395205912,  38.33771463904344),
                          new Vector2D( 90.14600407214999,  38.337714751520764),
                          new Vector2D( 90.14600418462749,  38.337714871611695),
                          new Vector2D( 90.14600422249327,  38.337714915811034),
                          new Vector2D( 90.14867838361471,  38.34113888210675),
                          new Vector2D( 90.14923750157374,  38.341582537502575),
                          new Vector2D( 90.14877083250991,  38.34160685841391),
                          new Vector2D( 90.14816667319519,  38.34244232585684),
                          new Vector2D( 90.14797696744586,  38.34248455284745),
                          new Vector2D( 90.14484318014337,  38.34385573215269),
                          new Vector2D( 90.14477919958296,  38.3453797747614),
                          new Vector2D( 90.14202393306448,  38.34464324839456),
                          new Vector2D( 90.14198920640195,  38.344651155237216),
                          new Vector2D( 90.14155207025175,  38.34486424263724),
                          new Vector2D( 90.1415196143314,  38.344871730519),
                          new Vector2D( 90.14128611910814,  38.34500196593859),
                          new Vector2D( 90.14047850603913,  38.34600084496253),
                          new Vector2D( 90.14045907000337,  38.34601860032171),
                          new Vector2D( 90.14039496493928,  38.346223030432384),
                          new Vector2D( 90.14037626063737,  38.346240203360026),
                          new Vector2D( 90.14030005823724,  38.34646920000705),
                          new Vector2D( 90.13799164754806,  38.34903093011013),
                          new Vector2D( 90.11045289492762,  38.36801537312368),
                          new Vector2D( 90.10871471476526,  38.36878044144294),
                          new Vector2D( 90.10424901707671,  38.374300101757),
                          new Vector2D( 90.10263482039932,  38.37310041316073),
                          new Vector2D( 90.09834601753448,  38.373615053823414),
                          new Vector2D( 90.0979455456843,  38.373578376172475),
                          new Vector2D( 90.09086514328669,  38.37527884194668),
                          new Vector2D( 90.09084931407364,  38.37590801712463),
                          new Vector2D( 90.09081227075944,  38.37526295920463),
                          new Vector2D( 90.09081378927135,  38.375193883266434)
                  }
              };
              PolygonsSet set1 = buildSet(vertices1);
      
              Vector2D[][] vertices2 = new Vector2D[][] {
                  new Vector2D[] {
                          new Vector2D( 90.13067558880044,  38.36977255037573),
                          new Vector2D( 90.12907570488,  38.36817308242706),
                          new Vector2D( 90.1342774136516,  38.356886880294724),
                          new Vector2D( 90.13090330629757,  38.34664392676211),
                          new Vector2D( 90.13078571364593,  38.344904617518466),
                          new Vector2D( 90.1315602208914,  38.3447185040846),
                          new Vector2D( 90.1316336226821,  38.34470643148342),
                          new Vector2D( 90.134020944832,  38.340936644972885),
                          new Vector2D( 90.13912536387306,  38.335497255122334),
                          new Vector2D( 90.1396178806582,  38.334878075552126),
                          new Vector2D( 90.14083049696671,  38.33316530644106),
                          new Vector2D( 90.14145252901329,  38.33152722916191),
                          new Vector2D( 90.1404779335565,  38.32863516047786),
                          new Vector2D( 90.14282712131586,  38.327504432532066),
                          new Vector2D( 90.14616669875488,  38.3237354115015),
                          new Vector2D( 90.14860976050608,  38.315714862457924),
                          new Vector2D( 90.14999277782437,  38.3164932507504),
                          new Vector2D( 90.15005207194997,  38.316534677663356),
                          new Vector2D( 90.15508513859612,  38.31878731691609),
                          new Vector2D( 90.15919938519221,  38.31852743183782),
                          new Vector2D( 90.16093758658837,  38.31880662005153),
                          new Vector2D( 90.16099420184912,  38.318825953291594),
                          new Vector2D( 90.1665411125756,  38.31859497874757),
                          new Vector2D( 90.16999653861313,  38.32505772048029),
                          new Vector2D( 90.17475243391698,  38.32594398441148),
                          new Vector2D( 90.17940844844992,  38.327427213761325),
                          new Vector2D( 90.20951909541378,  38.330616833491774),
                          new Vector2D( 90.2155400467941,  38.331746223670336),
                          new Vector2D( 90.21559881391778,  38.33175551425302),
                          new Vector2D( 90.21916646426041,  38.332584299620805),
                          new Vector2D( 90.23863749852285,  38.34778978875795),
                          new Vector2D( 90.25459855175802,  38.357790570608984),
                          new Vector2D( 90.25964298227257,  38.356918010203174),
                          new Vector2D( 90.26024593994703,  38.361692743151366),
                          new Vector2D( 90.26146187570015,  38.36311080550837),
                          new Vector2D( 90.26614159359622,  38.36510808579902),
                          new Vector2D( 90.26621342936448,  38.36507942500333),
                          new Vector2D( 90.26652190211962,  38.36494042196722),
                          new Vector2D( 90.26621240678867,  38.365113172030874),
                          new Vector2D( 90.26614057102057,  38.365141832826794),
                          new Vector2D( 90.26380080055299,  38.3660381760273),
                          new Vector2D( 90.26315345241,  38.36670658276421),
                          new Vector2D( 90.26251574942881,  38.367490323488084),
                          new Vector2D( 90.26247873448426,  38.36755266444749),
                          new Vector2D( 90.26234628016698,  38.36787989125406),
                          new Vector2D( 90.26214559424784,  38.36945909356126),
                          new Vector2D( 90.25861728442555,  38.37200753430875),
                          new Vector2D( 90.23905557537864,  38.375405314295904),
                          new Vector2D( 90.22517251874075,  38.38984691662256),
                          new Vector2D( 90.22549955153215,  38.3911564273979),
                          new Vector2D( 90.22434386063355,  38.391476432092134),
                          new Vector2D( 90.22147729457276,  38.39134652252034),
                          new Vector2D( 90.22142070120117,  38.391349167741964),
                          new Vector2D( 90.20665060751588,  38.39475580900313),
                          new Vector2D( 90.20042268367109,  38.39842558622888),
                          new Vector2D( 90.17423771242085,  38.402727751805344),
                          new Vector2D( 90.16756796257476,  38.40913898597597),
                          new Vector2D( 90.16728283954308,  38.411255399912875),
                          new Vector2D( 90.16703538220418,  38.41136059866693),
                          new Vector2D( 90.16725865657685,  38.41013618805954),
                          new Vector2D( 90.16746107640665,  38.40902614307544),
                          new Vector2D( 90.16122795307462,  38.39773101873203)
                  }
              };
              PolygonsSet set2 = buildSet(vertices2);
              PolygonsSet set  = (PolygonsSet) new
      RegionFactory<Euclidean2D>().difference(set1.copySelf(),
      
                    set2.copySelf());
      
              Vector2D[][] verticies = set.getVertices();
              Assert.assertTrue(verticies[0][0] != null);
              Assert.assertEquals(1, verticies.length);
          }
      

      Attachments

        1. PolygonDiffAll.png
          55 kB
          Curtis Jensen
        2. PolygonDiffResults.png
          38 kB
          Curtis Jensen
        3. PolygonInputs.png
          47 kB
          Curtis Jensen

        Activity

          People

            Unassigned Unassigned
            curtis Curtis Jensen
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: