Add new tests to the NCML handler to cover the Aggregation/Renaming bug fix


Two things to do here:

  1. Check the tests that currently fail when make check is run. The handler should be producing correct output - check that and update the baselines if the handler is working correctly.

  2. Look at test 405 which I’ve marked as expected fail because there is an issue with forming an aggregation on the outer dimensions of variables where the sizes don’t match.

More info about #2: I thought that using joinNew might be the problem - since the the two arrays were not the same exact shape ([31]‌‌‌[x]‌‌‌[y] versus [28]‌‌‌[x]‌‌‌[y]) maybe that was the reason for the failure. But these arrays work in a joinExisting aggregation just fine. However, when the Grids are renamed, there is a fail when reading data from the second of the two files. The problem is that the constraint using to access data from the second file uses size information gleaned from the first file. This is only the case when the Grids are renamed.

This is on the branch ncml_rename_test.

I think we might want to label this as a known bug, add a separate ticket for it, leave test 405 marked xfail and move on with merging this branch, since the issue reported regarding aggregations and renaming has been fixed for the kinds of data the reporters use, we should get that fix out to them.


James Gallagher
August 13, 2018, 11:08 PM

These tests may have the correct values even though they 'fail': 87 93 97 101 105 109 113 117 405 406

Check the values using getdap and then update the baselines if the correct values are being returned.

I used 'besstandalone -c bes.conf -i testsuite.dir/406/test.bescmd | getdap -M -' to look at the values after first running 'make check' where 406 is the number of the test that failed.

The names to feed to getnerate_baselines if these tests are passing:

  • 87: var_orgName.ncml

  • 93: var_orgName_Structure.ncml

  • 97: var_orgName_Structure_constraint_1.ncml

  • 101: var_orgName_Structure_cons_2.ncml

  • 105:var_orgName_cons_1.ncml

  • 109: var_orgName_cons_2.ncml

  • 113: var_orgName_cons_3.ncml

  • 117:var_orgName_cons_4.ncml

  • 405: gdal/h02v10_3Files_tif.ncml

  • 406: gdal/h02v10_3Files_tif.ncml

Your pinned fields
Click on the next to a field label to start pinning.


Slav Korolev


James Gallagher