[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: ForecastModelRun aggregation with explicit files



Hi Michael:

The forecast model aggregation is still experimental, so im not surprised you 
see these problems.
Thanks you for the patch, I will incorporate it ASAP.

Godin, Michael wrote:


There appears to be an omission in the code for handling "forecastModelRun" aggregations when the files being aggregated are explicit files (specified with a "netcdf" element) as opposed to the contents of scanned directories (specified with a "scan" element). Esentially, the function checkOK is not being called for the explicit files.

A minor modification to the Aggregation#finish method appears to resolve the issue:
starting at line 338 or 339:
    //nestedDatasets.addAll(explicitDatasets);
    for (int i = 0; i < explicitDatasets.size(); i++) {
        Dataset dataset = (Dataset) explicitDatasets.get(i);
        if (dataset.checkOK(cancelTask))
                nestedDatasets.add(dataset);
    }

A workaround is to use <scan> elements instead of <netcdf> elements for explicit file, specifying the parent path in the "location" attribute and the file name in the "suffix" attribute, but this is a little hokey.

Regards, Mike