[Git][debian-gis-team/proj][experimental] 2 commits: Add patch by Charles Karney to relax some of the tests in geodtest.

Bas Couwenberg gitlab at salsa.debian.org
Sat Feb 24 10:33:01 UTC 2018


Bas Couwenberg pushed to branch experimental at Debian GIS Project / proj


Commits:
d9e705d6 by Bas Couwenberg at 2018-02-24T11:15:07+01:00
Add patch by Charles Karney to relax some of the tests in geodtest.

- - - - -
9a0c0dd9 by Bas Couwenberg at 2018-02-24T11:16:03+01:00
Set distribution to experimental.

- - - - -


3 changed files:

- debian/changelog
- + debian/patches/Relax-some-the-tests-in-geodtest.patch
- debian/patches/series


Changes:

=====================================
debian/changelog
=====================================
--- a/debian/changelog
+++ b/debian/changelog
@@ -1,3 +1,9 @@
+proj (5.0.0~rc5-1~exp2) experimental; urgency=medium
+
+  * Add patch by Charles Karney to relax some of the tests in geodtest.
+
+ -- Bas Couwenberg <sebastic at debian.org>  Sat, 24 Feb 2018 11:15:11 +0100
+
 proj (5.0.0~rc5-1~exp1) experimental; urgency=medium
 
   * New upstream release candidate.


=====================================
debian/patches/Relax-some-the-tests-in-geodtest.patch
=====================================
--- /dev/null
+++ b/debian/patches/Relax-some-the-tests-in-geodtest.patch
@@ -0,0 +1,97 @@
+Description: Relax some the tests in geodtest.
+ This attempts to address the geodtest failures noted in #819.
+Author: Charles Karney <charles at karney.com>
+Origin: https://github.com/cffk/proj.4/commit/7ff60ee494b225f3988648c64bd70f6ae4176257
+
+--- a/src/geodtest.c
++++ b/src/geodtest.c
+@@ -698,35 +698,35 @@ static int GeodSolve80() {
+   int result = 0;
+   geod_init(&g, wgs84_a, wgs84_f);
+   geod_geninverse(&g, 0, 0, 0, 90, 0, 0, 0, 0, &M12, &M21, 0);
+-  result += assertEquals(M12, -0.0052842753, 0.5e-10);
+-  result += assertEquals(M21, -0.0052842753, 0.5e-10);
++  result += assertEquals(M12, -0.00528427534, 0.5e-10);
++  result += assertEquals(M21, -0.00528427534, 0.5e-10);
+   geod_geninverse(&g, 0, 0, 1e-6, 1e-6, 0, 0, 0, 0, &M12, &M21, 0);
+   result += assertEquals(M12, 1, 0.5e-10);
+   result += assertEquals(M21, 1, 0.5e-10);
+   a12 = geod_geninverse(&g, 20.001, 0, 20.001, 0,
+                         &s12, &azi1, &azi2, &m12, &M12, &M21, &S12);
+-  result += a12 == 0 ? 0 : 1;
+-  result += s12 == 0 ? 0 : 1;
+-  result += azi1 == 180 ? 0 : 1;
+-  result += azi2 == 180 ? 0 : 1;
+-  result += m12 == 0 ? 0 : 1;
++  result += assertEquals(a12, 0, 1e-13);
++  result += assertEquals(s12, 0, 1e-8);
++  result += assertEquals(azi1, 180, 1e-13);
++  result += assertEquals(azi2, 180, 1e-13);
++  result += assertEquals(m12, 0,  1e-8);
+   result += assertEquals(M12, 1, 1e-15);
+   result += assertEquals(M21, 1, 1e-15);
+-  result += S12 == 0 ? 0 : 1;
++  result += assertEquals(S12, 0, 1e-10);
+   a12 = geod_geninverse(&g, 90, 0, 90, 180,
+                         &s12, &azi1, &azi2, &m12, &M12, &M21, &S12);
+-  result += a12 == 0 ? 0 : 1;
+-  result += s12 == 0 ? 0 : 1;
+-  result += azi1 == 0 ? 0 : 1;
+-  result += azi2 == 180 ? 0 : 1;
+-  result += m12 == 0 ? 0 : 1;
++  result += assertEquals(a12, 0, 1e-13);
++  result += assertEquals(s12, 0, 1e-8);
++  result += assertEquals(azi1, 0, 1e-13);
++  result += assertEquals(azi2, 180, 1e-13);
++  result += assertEquals(m12, 0, 1e-8);
+   result += assertEquals(M12, 1, 1e-15);
+   result += assertEquals(M21, 1, 1e-15);
+   result += assertEquals(S12, 127516405431022, 0.5);
+   /* An incapable line which can't take distance as input */
+   geod_lineinit(&l, &g, 1, 2, 90, GEOD_LATITUDE);
+   a12 = geod_genposition(&l, 0, 1000, 0, 0, 0, 0, 0, 0, 0, 0);
+-  result += a12 != a12 ? 0 : 1;
++  result += a12 == a12 ? 1 : 0;
+   return result;
+ }
+ 
+@@ -890,8 +890,8 @@ static int Planimeter19() {
+   result += area == 0 ? 0 : 1;
+   result += perim == 0 ? 0 : 1;
+   geod_polygon_testedge(&g, &p, 90, 1000, 0, 1, &area, &perim);
+-  result += area != area ? 0 : 1;
+-  result += perim != perim ? 0 : 1;
++  result += area == area ? 1 : 0;
++  result += perim == perim ? 1 : 0;
+   geod_polygon_addpoint(&g, &p, 1, 1);
+   geod_polygon_compute(&g, &p, 0, 1, &area, &perim);
+   result += area == 0 ? 0 : 1;
+@@ -902,7 +902,7 @@ static int Planimeter19() {
+   geod_polygon_testpoint(&g, &p, 1, 1, 0, 1, 0, &perim);
+   result += perim == 0 ? 0 : 1;
+   geod_polygon_testedge(&g, &p, 90, 1000, 0, 1, 0, &perim);
+-  result += perim != perim ? 0 : 1;
++  result += perim == perim ? 1 : 0;
+   geod_polygon_addpoint(&g, &p, 1, 1);
+   geod_polygon_compute(&g, &p, 0, 1, 0, &perim);
+   result += perim == 0 ? 0 : 1;
+@@ -991,8 +991,8 @@ static int EmptyPoly() {
+   result += area == 0 ? 0 : 1;
+   result += perim == 0 ? 0 : 1;
+   geod_polygon_testedge(&g, &p, 90, 1000, 0, 1, &area, &perim);
+-  result += area != area ? 0 : 1;
+-  result += perim != perim ? 0 : 1;
++  result += area == area ? 1 : 0;
++  result += perim == perim ? 1 : 0;
+   geod_polygon_compute(&g, &p, 0, 1, &area, &perim);
+   result += area == 0 ? 0 : 1;
+   result += perim == 0 ? 0 : 1;
+@@ -1000,7 +1000,7 @@ static int EmptyPoly() {
+   geod_polygon_testpoint(&g, &p, 1, 1, 0, 1, 0, &perim);
+   result += perim == 0 ? 0 : 1;
+   geod_polygon_testedge(&g, &p, 90, 1000, 0, 1, 0, &perim);
+-  result += perim != perim ? 0 : 1;
++  result += perim == perim ? 1 : 0;
+   geod_polygon_compute(&g, &p, 0, 1, 0, &perim);
+   result += perim == 0 ? 0 : 1;
+   geod_polygon_addpoint(&g, &p, 1, 1);


=====================================
debian/patches/series
=====================================
--- a/debian/patches/series
+++ b/debian/patches/series
@@ -1 +1,2 @@
 Dont-reset-errno-when-skipping-tests.patch
+Relax-some-the-tests-in-geodtest.patch



View it on GitLab: https://salsa.debian.org/debian-gis-team/proj/compare/a32b3fc1feca4d6b39bb377def2001a5d3b24754...9a0c0dd9f440240f846ef2638c1b84c43263b9c5

---
View it on GitLab: https://salsa.debian.org/debian-gis-team/proj/compare/a32b3fc1feca4d6b39bb377def2001a5d3b24754...9a0c0dd9f440240f846ef2638c1b84c43263b9c5
You're receiving this email because of your account on salsa.debian.org.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.alioth.debian.org/pipermail/pkg-grass-devel/attachments/20180224/33b137df/attachment-0001.html>


More information about the Pkg-grass-devel mailing list