<div dir="ltr"><div dir="ltr"><div dir="ltr"><br><br>On Wed, Mar 6, 2019 at 8:38 PM Greg Troxel <<a href="mailto:gdt@lexort.com">gdt@lexort.com</a>> wrote:<br>><br>> Markus Metz <<a href="mailto:markus.metz.giswork@gmail.com">markus.metz.giswork@gmail.com</a>> writes:<br>><br>> > On Wed, Mar 6, 2019 at 6:57 PM Greg Troxel <<a href="mailto:gdt@lexort.com">gdt@lexort.com</a>> wrote:<br>> >><br>> >> Given that the grid files are intended to get better results, it would<br>> >> be really nice if the tests passed with them, either via fuzz (which<br>> >> makes sense if both answers are correct) or via having two recorded<br>> >> acceptable values.<br>> ><br>> > I guess the tests are designed to test the numerical stability of the proj<br>> > code, for given input data. Therefore it does not matter how accurate the<br>> > input data are, what matters is that for the expected input data the<br>> > expected result must be provided. If the input data are not as expected,<br>> > the test fails, obviously.<br>><br>> yes, but<br>><br>> > The output line<br>> > --><br>> > Try with NTv2 and NTv1 together ... falls back to NTv1<br>> > <--<br>> > looks suspicious, also causing the test failure.<br>> ><br>> > README:<br>> > "The test suite requires that the proj-datumgrid package is installed<br>> > in `PROJ_LIB`."<br>> ><br>> > This should be expanded to note which datumgrid packages should *not* be<br>> > installed for the tests, or that only the package proj-datumgrid-<X>.<Y><br>> > should be installed, but no other datumgrid packages.<br>><br>> This is what's awkward.  Various packaging systems will have different<br>> approaches.  One is to include all the grids (space is cheap, simple),<br>> another is to make them separate packages, and another is to not package<br>> them at all.  Telling people that you can't run proj tests if the other<br>> package is installed, when really you should have it installed, just<br><div>> makes it hard and will make the tests be ignored.</div><div><br></div><div>I was reporting as a user compiling proj from source, not as a packager. I agree that tests must succeed if only the minimum required datum grids are installed (why are these not included in the proj source code?) and also if all other datum grids available from <a href="https://proj4.org/download.html">https://proj4.org/download.html</a> are installed. Additional grids e.g. mentioned in <a href="https://proj4.org/resource_files.html#external-resources">https://proj4.org/resource_files.html#external-resources</a> should not be considered by the tests.</div><div><br></div><div>In short, tests must use expected input and ignore unexpected input in order to succeed. That's what you suggest, right?</div><div><br></div><div>Markus M<br></div></div></div></div>