[ET Trac] [Einstein Toolkit] #1050: [PATCH]CarpetIOHDF5: make WriteLargeAttribute write a string, not an array of ints

[Einstein Toolkit]

#1050: [PATCH]CarpetIOHDF5: make WriteLargeAttribute write a string, not an array
of ints
--------------------------------+-------------------------------------------
  Reporter:  anton@…            |       Owner:  eschnett
      Type:  defect             |      Status:  new     
  Priority:  minor              |   Milestone:          
 Component:  Carpet             |     Version:          
Resolution:                     |    Keywords:          
--------------------------------+-------------------------------------------

Comment (by eschnett):

 No need to be sorry! That's what a review is for, and I thank you for
 improving our code.

 I am not overly familiar with strings in HDF5. I notice that the
 documentation of H5D_get_storage_size says that it is generally not
 recommended to use this function. H5T_get_size (the converse of the
 function used to create the string type) may be more appropriate. Could
 you update your patch?

 Did you run the test cases after applying your patch? I assume that (a)
 this patch indeed makes it simpler for Python to read the attributes, and
 (b) that in particular the recovery test cases succeed.

 To answer an earlier question: Yes, these attributes are how the grid
 structure is stored and retrieved. We opted for a human-readable (i.e.
 ASCII) format to simplify debugging.

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