#1533: checkpoint cctk_delta_time in PUGH ------------------------------------+--------------------------------------- Reporter: rhaas | Owner: Type: defect | Status: review Priority: major | Milestone: Component: EinsteinToolkit thorn | Version: development version Resolution: | Keywords: IOHDF5Util ------------------------------------+---------------------------------------
Comment (by eschnett):
You write cctk_time and cctk_delta_time into the same attribute. Is there a particular reason for this? Otherwise I find it cleaner to use a separate attribute named "cctk_delta"time". This would also be automatically ignored when read from older versions of PUGH. You would read this attribute with a construct equivalent to {{{ bool file_exists; H5E_BEGIN_TRY { file_exists = H5Fis_hdf5(name.c_str()) > 0; } H5E_END_TRY; if (not file_exists) { }}} that ensures that HDF5 will not abort on errors.