aodn / imos-toolbox

Graphical tool for QC'ing and NetCDF'ing oceanographic datasets
GNU General Public License v3.0
46 stars 31 forks source link

Insert a latitude/longitude range check before PP routines #634

Open BecCowley opened 4 years ago

BecCowley commented 4 years ago

When running the magneticDeclinationPP, the geomag70 code failed because I had lat and long in the wrong fields in the database csv files (longitude in latitude field and latitude in longitude field). There is no error message when the geomag70 code fails, the toolbox just hangs forever.

It would be nice to check the logic of the input arguments before the geomag70 code is run.

ocehugo commented 4 years ago

Thanks.

I think the right way is to validate the database when reading but argument validation before a mex function call should also be done to avoid the lock/hang.

Requirements: