[ET Trac] [Einstein Toolkit] #1533: checkpoint cctk_delta_time in PUGH

[Einstein Toolkit]

#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.

-- 
Ticket URL: <https://trac.einsteintoolkit.org/ticket/1533#comment:2>
Einstein Toolkit <http://einsteintoolkit.org>
The Einstein Toolkit