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

Filenames with deployment field, closes #541 #614

Closed ocehugo closed 4 years ago

ocehugo commented 4 years ago

This move the filename conventions to use the time_deployment fields isntead of time_coverage fields.

@ggalibert , please check if this is what you had in mind in #541

ocehugo commented 4 years ago

@ggalibert, I think @lbesnard is waiting your :+1: on that to merge

ocehugo commented 4 years ago

@lbesnard , this became a very simple commit since I removed some refactorign not needed.

ocehugo commented 4 years ago

@ggalibert - there is only a case here that may break a workflow - for very simple processing streams (no ddb and no PP/QC runs), the filenames only get creation dates since time_deployment is empty (but not time_coverage).

I'm just wondering if this is someone workflow outside of us simply testing the export to netcdf functionality.