#1118: Incorrect header info for 2d CarpetIOASCII when "compact_format=yes" -----------------------------------------+---------------------------------- Reporter: reisswig@… | Owner: eschnett Type: defect | Status: review Priority: minor | Milestone: Component: Carpet | Version: Resolution: | Keywords: -----------------------------------------+----------------------------------
Comment (by eschnett):
Can you try this patch instead? {{{ diff --git a/Carpet/CarpetIOASCII/src/ioascii.cc b/Carpet/CarpetIOASCII/src/ioas index 0a966d5..74557e5 100644 --- a/Carpet/CarpetIOASCII/src/ioascii.cc +++ b/Carpet/CarpetIOASCII/src/ioascii.cc @@ -1377,7 +1377,7 @@ namespace CarpetIOASCII { } } os << "\t" << col++ << ":time"; - if (not compact_format) { + if (not compact_format or grouptype == CCTK_GF) { for (int d=0; d<dim; ++d) { os << (d==0 ? "\t" : " ") << col++ << ":" << coords[d]; } }}}
It is simpler, and uses the same logic while outputting headers as when outputting the coordinates.
The check dim vs. groupdim is not necessary since CCTK_GF always have dim dimensions.