Skip to content

Subclass axioms analysis: Minor - "top 10" tweaks #259

@joeflack4

Description

@joeflack4

I customized these outputs (they eventually make their way into stats.md), where if there are too many bars in the upset plot, it makes a variation of the upset plot that only shows the top 10 bars (by intersection size), and adds ", top 10" to the title.

Results after running

### SC axioms: All sources, with CompLOINC disaggregations, top 10

![Upset plot](upset-plot_all-sources-with-comploinc-disaggregations.png)

![Upset plot](upset-plot_all-sources-with-comploinc-disaggregations-untruncated.png)

Bugs:

  • This subheader ### is dynamically generated, and should not also say "top 10"
  • It also renders a table below these plots. That table should show all of the kinds of intersections, not just the top 10. But it currently only shows the top 10.
  • Also minor, but I would prefer that the "non top 10" plot variations should not have a suffix, e.g. when all bars are included, it should say upset-plot_all-sources-with-comploinc-disaggregations.png. And for any ones where it only shows the top 10, it should add the suffix at the end e.g. -truncated.png. But right now it is the opposite.

Metadata

Metadata

Assignees

No one assigned

    Type

    Projects

    Status

    1. Backlog

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions