Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Summary data display #26

Open
apjanke opened this issue Mar 2, 2019 · 0 comments
Open

Summary data display #26

apjanke opened this issue Mar 2, 2019 · 0 comments
Assignees
Labels
enhancement New feature or request tables The table stuff
Milestone

Comments

@apjanke
Copy link
Owner

apjanke commented Mar 2, 2019

Gonna need to do a whole type hierarchy for this.

  • Each variable needs a custom sub-display output based on its type.
  • Since per-variable summary types vary, we'll need a heterogeneous collection of custom per-variable summary objects.
  • Maybe we could collect those in a struct of sub-objects.
    • But we'll also need object-level summary data for object-level stuff like tbl.Description.
    • So, a table-level object that has a collection of table-level properties, and then a struct of heterogeneous per-variable summary objects?

I'm thinking:

  • octave.table.internal.TableSummary
    • field .TableProperties: struct of (name, string)
    • field .VariableSummaries: struct of (name, octave.table.internal.VariableSummary subclasses)
  • octave.table.internal.VariableSummary
    • abstract type with concrete subtypes for various variable types
    • method print(this, indent) for incorporation into indented table summary output
    • concrete subclasses for:
      • Numeric (with polymorphic handling of actual scalar numeric values for display of min/max/median/mean/etc)
      • logical
      • cellstr
      • table for nested tables, sheesh
        • maybe this is a wrapper around octave.table.internal.TableSummary?
        • do we recursively display the full indented table summary for the nested table, or just a top-level summary? See Nested table support #22 (“nested table support”).
      • datetime
        • harder because datetime itself is not implemented yet
      • categorical
        • harder because categorical itself is not implemented yet
    • will need a generic one that just contains a dumb single-string summary of the variable
@apjanke apjanke self-assigned this Mar 2, 2019
@apjanke apjanke added the enhancement New feature or request label Mar 2, 2019
@apjanke apjanke added this to the 1.0 milestone Mar 2, 2019
@apjanke apjanke mentioned this issue Mar 2, 2019
58 tasks
@apjanke apjanke changed the title Summary data Summary data and display Mar 2, 2019
@apjanke apjanke changed the title Summary data and display Summary data display Mar 2, 2019
@apjanke apjanke added this to Needs triage in Octave-Tablicious via automation May 5, 2019
@apjanke apjanke moved this from Needs triage to High priority in Octave-Tablicious May 5, 2019
@apjanke apjanke added the tables The table stuff label May 5, 2019
@apjanke apjanke modified the milestones: 0.3.0, 1.0.0 Jan 17, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request tables The table stuff
Projects
Status: High priority
Octave-Tablicious
  
High priority
Development

No branches or pull requests

1 participant