NOAA-GFDL / MDTF-diagnostics

Analysis framework and collection of process-oriented diagnostics for weather and climate simulations
https://mdtf-diagnostics.readthedocs.io/en/main/
Other
64 stars 100 forks source link

"save_pp_data" flag doesn't delete data if a plot fails #702

Closed jtmims closed 3 days ago

jtmims commented 3 weeks ago

Bug Severity

Describe the bug Currently, the "save_pp_data" flag in the runtime_config json file doesn't matter if the POD has a faulty plot for PODs such as Wheeler_Kiladis or MJO_suite. Since we are trying to implement the MDTF into the workflow of GFDL model development, having lingering pp_data in dirs could become problematic especially when wanting to run a large suite of PODs.

Environment Describe the system environment:

jtmims commented 3 days ago

After digging further into output_manager.py, it seems as though the framework was agnostic to the files in the WORK_DIR and would only cleanup .nc files in each POD dir. #710 makes the framework look into the WORK_DIR/OUTPUT_DIR as well.