apjanke
commented
5 years ago Ah: to make this work with prettyprint(), define totalcolumns and flatDispstrs methods. totalcolumns counts the total columns across all variables, recursively expanding nested tables. flatDispstrs calculates display strings for both headers and cell contents:
For an k rows × n totalcolumns table:
- Headers is an m-by-n cellstr of header columns to display in a grid above rows
- Where m is the total nesting depth of tables, maybe +1 for each table level that has VariableUnits, and maybe +1 for each table level that has Variable
- Bodycells is a k-by-n cellstr of dispstrs from the VariableValues (just the bodies for nested tables), flattened out
- Bump n by +1 for each table (including nested tables) that have RowNames present.
Then you can just format the headers + bodycells as a grid, and not have to worry about doing recursive stuff at the display step; just the string gathering step. The string gathering step can be recursive, using dispstrs() for regular variables and flatDispstrs() for table-valued variables.
Should RowNames be included in totalcolumns? Probably not; it should probably just show up as an extra display column in prettyprint, making n = totalcolumns + 1 for tables with RowNames. RowNames aren't really a column; they're a dimension label (I think).
Need to support nested tables. That is, tables, with table-valued variables. These behave somewhat differently than the regular polymorphic array behavior. For example, they have special display formats in the prettyprint output (See the
splitvarsdoco and thesummarydoco).Related Issues