Skip to content

Conversation

ctb
Copy link
Contributor

@ctb ctb commented Aug 14, 2025

Adds taxburst JSON input support for sankey and treemap commands.

Also fixes a bug with sankey & tax annotate input that eliminated all but the last source -> dest link (I think?) At any rate, Sankey and taxburst now produce output that is consistent between the two.

Fixes #86.

  • check @ctb comment about csv_summary special casing
  • refactor JSON argument name/bool flag for consistency
  • clean up code a bit
  • add tests into workflow?

@ctb ctb changed the title WIP: read taxburst JSON for sankey & treemap MRG: read taxburst JSON for sankey & treemap Aug 19, 2025
@ctb
Copy link
Contributor Author

ctb commented Aug 19, 2025

@bluegenes ready for review & merge!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

consider supporting ingest of taxburst-like JSON format
1 participant